[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20110121.165409.200115988.davem@davemloft.net>
Date: Fri, 21 Jan 2011 16:54:09 -0800 (PST)
From: David Miller <davem@...emloft.net>
To: mkl@...gutronix.de
Cc: Socketcan-core@...ts.berlios.de, netdev@...r.kernel.org,
wg@...ndegger.com
Subject: Re: [PATCH v2 1/3] can: at91_can: clean up usage of
AT91_MB_RX_FIRST and AT91_MB_RX_NUM
From: Marc Kleine-Budde <mkl@...gutronix.de>
Date: Mon, 17 Jan 2011 20:50:36 +0100
> On 01/11/2011 02:21 PM, Marc Kleine-Budde wrote:
>> This patch cleans up the usage of two macros which specify the mailbox
>> usage. AT91_MB_RX_FIRST and AT91_MB_RX_NUM define the first and the
>> number of RX mailboxes. The current driver uses these variables in an
>> unclean way; assuming that AT91_MB_RX_FIRST is 0;
>>
>> This patch cleans up the usage of these macros, no longer assuming
>> AT91_MB_RX_FIRST == 0.
>>
>> Signed-off-by: Marc Kleine-Budde <mkl@...gutronix.de>
>
> Any comments on this?
I would also seriously like to see these changes get some feedback,
they've been rotting in patchwork for more than a week.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists