[U-Boot] Checkpatch warnings for "volatile"

Jason u-boot at lakedaemon.net
Fri Oct 14 22:22:24 CEST 2011


Wolfgang,

On Fri, Oct 14, 2011 at 09:58:44PM +0200, Wolfgang Denk wrote:
> Dear Jason,
> 
> In message <20111014131245.GF7102 at titan.lakedaemon.net> you wrote:
> > 
> > 	1.) checkpatch.pl complained about volatile in source.c:312.  This is
> >        a register we bitbang in a loop and we don't want the compiler
> >        optimizing it out.
> 
> This would, in almost all cases, trigger a NAK due to the fact that
> device register accesses should be done through I/O accesors, and
> never through volatile pointers.

very true, I was attempting to give a generic example.  Sorry if that
led to any confusion.

thx,

Jason.


More information about the U-Boot mailing list