RE: [PATCH v6 1/2] can: rcar_canfd: Add Renesas R-Car CAN FD driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Ulrich,

> Thanks for the patch, this is a lot easier to handle.
> 
[...]
> > +#define RCANFD_RFPTR_RFTS(x)           (((x) >> 0) & 0xff)
> 
> That should be "(((x) >> 0) & 0xffff)".
> 
> > +
[...]

> > +/* RSCFDnCFDRMXXXq -> RCANFD_F_RMXXX(q) */
> > +#define RCANFD_F_RMID(q)               (0x2000 + (0x10 * (q)))
> > +#define RCANFD_F_RMPTR(q)              (0x2004 + (0x10 * (q)))
> > +#define RCANFD_F_RMFDSTS(q)            (0x2008 + (0x10 * (q)))
> 
> These should all be "+ (0x20 * (q))".
> 
[...]
> 
> Apart from that, all register addresses, offsets, bit positions and masks
> check out. With the two issues fixed:
> 
> Reviewed-by: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx>

Thank you for your time & review comments. I have fixed these typos and sent out a v7 patch set adding your Reviewed-by tag.

Thanks,
Ramesh




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux