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

Jagan Teki jagannadh.teki at gmail.com
Thu Oct 22 20:21:29 CEST 2015


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 and as this a common change I have
combined all spi drivers as I belongs to - did you see any wrong on
this?

-- 
Jagan.


More information about the U-Boot mailing list