From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Date: Tue, 05 Nov 2013 21:59:36 +0000 Subject: Re: [PATCH v2] can: add Renesas R-Car CAN driver Message-Id: <5279784E.1010205@cogentembedded.com> List-Id: References: <201310250303.00695.sergei.shtylyov@cogentembedded.com> <526AC65B.5020203@grandegger.com> <52742A0F.7040707@cogentembedded.com> <5276A63E.3070505@grandegger.com> In-Reply-To: <5276A63E.3070505@grandegger.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Wolfgang Grandegger , netdev@vger.kernel.org, mkl@pengutronix.de, linux-can@vger.kernel.org Cc: linux-sh@vger.kernel.org, vksavl@gmail.com Hello. On 11/03/2013 10:38 PM, Wolfgang Grandegger wrote: >>>> Add support for the CAN controller found in Renesas R-Car SoCs. >>>> Signed-off-by: Sergei Shtylyov >> [...] >>>> Index: linux-can-next/drivers/net/can/rcar_can.c >>>> =================================>>>> --- /dev/null >>>> +++ linux-can-next/drivers/net/can/rcar_can.c >>>> @@ -0,0 +1,920 @@ >> [...] >>>> + rcar_can_writeb(priv, RCAR_CAN_ECSR, (u8)~ECSR_ADEF); >>> Please avoid casts here and below. >> These casts help avoid compiler warnings. > Well instead of using casts the declaration of ECSR_ADEF and otheres > should be fixed. I think the problem is that BIT is declared as shown below: > #define BIT(nr) (1UL << (nr)) > Using BIT seems not appropriate here. Well, I thought it was designed to replace explicit (1 << (nr)) #define's. Hopefully those are good for you then... > Wolfgang, WBR, Sergei