[U-Boot] [PATCH-OMAP3] OMAP3: Use I2C file coding style

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Sat Nov 8 20:09:08 CET 2008


On 17:32 Tue 04 Nov     , dirk.behme at googlemail.com wrote:
> Subject: [PATCH-OMAP3] OMAP3: Use I2C file coding style
> 
> From: Dirk Behme <dirk.behme at gmail.com>
> 
> Use file coding style for inx/outx instead of global coding style.
> 
> Signed-off-by: Dirk Behme <dirk.behme at gmail.com>
> 
> ---
> 
> Note: There was an additional review comment about this file:
> 
> -- cut --
> > +#define inb(a) __raw_readb(a)
> > +#define outb(a, v) __raw_writeb(a, v)
> >  #define inw(a) __raw_readw(a)
> >  #define outw(a,v) __raw_writew(a,v)
> 
> This 4 macro is supposed to be defined in io.h
> -- cut --
> 
> In ARM's io.h there are already inx/outx macros, but with different syntax. The correct fix for omap24xx_i2c.c will be to replace all inx/outx by readx/writex macros and remove above defines. But this can't be done on OMAP3 branch, as it would conflict with "no general coding style clean up in OMAP3 patches, only OMAP3 related changes, please". Thus, we have to do code style changes for this file at mainline once OMAP3 is merged. Until then we have to stay with consistent local style.

I desagree,

This fix is supposed to be done before appling of the OMAP3 patch set not
after.

please do not add code which need to fix just after.

NB : applied to the omap3 branch

Best Regards,
J.


More information about the U-Boot mailing list