[U-Boot] [PATCH v4 1/4] bitops: introduce BIT() definition

Heiko Schocher hs at denx.de
Mon Sep 7 13:52:52 CEST 2015


Hello Andreas,

Am 07.09.2015 um 13:20 schrieb Andreas Bießmann:
> On 08/21/2015 07:01 PM, Heiko Schocher wrote:
>> introduce BIT() definition, used in at91_udc gadget
>> driver.
>>
>> Signed-off-by: Heiko Schocher <hs at denx.de>
>>
>
> NAK, this one breaks a lot of boards which already defined BIT()

Uhh... seems this BIT() macro is a big mess ...

Hmm Wolfgang Denk NACKed a similiar patch:
http://lists.denx.de/pipermail/u-boot/2014-February/173669.html

In drivers/usb/gadget/at91_udc.c BIT(x) is used only once...
So I fix it there and use (1 << x) there. Would be this OK?

bye,
Heiko
>
>> ---
>>
>> Changes in v4: None
>> Changes in v3:
>> - new in v3
>>
>> Changes in v2: None
>>
>>   include/linux/bitops.h | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/include/linux/bitops.h b/include/linux/bitops.h
>> index e724310..7d30ace 100644
>> --- a/include/linux/bitops.h
>> +++ b/include/linux/bitops.h
>> @@ -3,6 +3,8 @@
>>
>>   #include <asm/types.h>
>>
>> +#define BIT(nr)		(1UL << (nr))
>> +
>>   /*
>>    * ffs: find first bit set. This is defined the same way as
>>    * the libc and compiler builtin ffs routines, therefore
>>
>

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany


More information about the U-Boot mailing list