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

Marek Vasut marex at denx.de
Thu Oct 22 21:05:27 CEST 2015


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.

> 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.

Best regards,
Marek Vasut


More information about the U-Boot mailing list