On Wed, 2015-09-09 at 20:25 +0200, Wolfgang Denk wrote:
> Dear Andreas,
> 
> In message <1441626234-16364-1-git-send-email-andreas.de...@googlemail.com> 
> you wrote:
> ...
> > --- a/include/linux/bitops.h
> > +++ b/include/linux/bitops.h
> > @@ -3,6 +3,8 @@
> >  
> >  #include <asm/types.h>
> >  
> > +#define BIT(nr)            (1UL << (nr))
> 
> What happens when someone decides to use this on a 64 bit register?
> 
> Also, this definition is inherently wrong for Power Architecture (TM)
> systems, where bit 0 is the most significant bit.

It's not "inherently wrong" to number bits from LSB on such systems -- it 
just doesn't match the unusual convention found in PPC documentation and the 
rotate instructions.

In any case, this would be part of the Linux driver compatibility layer 
rather than the placement in common.h that you previously objected to.

-Scott

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to