[U-Boot] [PATCH v3 3/4] spi: Use BIT macro in spi/drivers/*

Jagan Teki jteki at openedev.com
Thu Oct 22 21:09:43 CEST 2015


On 23 October 2015 at 00:35, Marek Vasut <marex at denx.de> wrote:
> On Thursday, October 22, 2015 at 08:21:29 PM, Jagan Teki wrote:
>> On 22 October 2015 at 23:45, Marek Vasut <marex at denx.de> wrote:
>> > On Thursday, October 22, 2015 at 07:45:49 PM, Jagan Teki wrote:
>> >> BIT macro used on drivers/spi/*
>> >>
>> >> Replace (1 << nr) to BIT(nr) where nr = 0, 1, 2 .... 31
>> >>
>> >> Cc: Tom Rini <trini at konsulko.com>
>> >> Cc: Simon Glass <sjg at chromium.org>
>> >> Cc: Masahiro Yamada <yamada.m at jp.panasonic.com>
>> >> Cc: Peng Fan <Peng.Fan at freescale.com>
>> >> Cc: Stefan Roese <sr at denx.de>
>> >> Cc: Marek Vasut <marex at denx.de>
>> >> Cc: Stephen Warren <swarren at nvidia.com>
>> >> Cc: Tom Warren <twarren at nvidia.com>
>> >> Signed-off-by: Jagan Teki <jteki at openedev.com>
>> >
>> > So, you're sending this horrid blob of changes instead of splitting them
>> > into more managable parts, sigh. Moreover, I have a feeling this is a
>> > change for the sake of changing something, not great. Also, you did not
>> > describe how you performed this change -- was it done using some sort of
>> > sed script or by hand ?
>>
>> Yes this is performed using sed
>
> The script you used should be explained somewhere.

OK, add in next version patches.

>
>> and as this a common change I have
>> combined all spi drivers as I belongs to - did you see any wrong on
>> this?
>
> It's hard to review such a big blob, furthermore if you need to adjust some
> driver, you'd have to repost this whole thing instead of a single patch.

OK, then I will resend it separate patches, for individual drivers.

-- 
Jagan | openedev.


More information about the U-Boot mailing list