[U-Boot] [PATCH v2 1/2] linux/bitops.h: Add BIT macro

Simon Glass sjg at chromium.org
Wed Apr 29 16:42:12 CEST 2015


Hi Tom,

On 29 April 2015 at 07:08, Tom Rini <trini at konsulko.com> wrote:
> On Wed, Apr 29, 2015 at 05:35:05PM +0530, Jagannadha Sutradharudu Teki wrote:
>
>> Replace (1 << nr) to BIT(nr) where nr = 0, 1, 2 .... 31
>>
>> Signed-off-by: Jagannadha Sutradharudu Teki <jagannadh.teki at gmail.com>
>> Cc: Masahiro Yamada <yamada.m at jp.panasonic.com>
>> Cc: Tom Rini <trini at konsulko.com>
>> Cc: Simon Glass <sjg at chromium.org>
>
> I think that conceptually this is a good idea.  We need to apply this
> fairly quickly however as it already throws out a few rejects with the
> PRs I've taken and doing some local testing on.

We have to date avoided this (I've been here before). But I think it
is useful. One concern I have is misuse, when someone does BIT(0) |
BIT(1))| BIT(2) | BIT(3) instead of 0xf, for example.

Anyway as you say, let's do it fast if we are going this way.

Regards,
Simon


More information about the U-Boot mailing list