[U-Boot] [PATCH v5 2/5] dts: cadence_spi: Sync DT bindings with Linux

Jason Rush jarush at gmail.com
Tue Jan 23 15:12:10 UTC 2018


On 1/23/2018 8:25 AM, Jagan Teki wrote:
> On Tue, Jan 23, 2018 at 7:45 PM, Jason Rush <jarush at gmail.com> wrote:
>> On 1/23/2018 3:16 AM, Jagan Teki wrote:
>>> On Sat, Jan 6, 2018 at 11:47 PM, Jason Rush <jarush at gmail.com> wrote:
>>>> Adopt the Linux DT bindings and clean-up duplicate
>>>> and unused values.
>>>>
>>>> Tested on TI K2G platform:
>>>> Tested-by: Vignesh R <vigneshr at ti.com>
>>>>
>>>> Tested on a socfpga-cyclonev board:
>>>> Tested-by: Simon Goldschmidt <sgoldschmidt at de.pepperl-fuchs.com>
>>>>
>>>> Signed-off-by: Jason Rush <jarush at gmail.com>
>>>> Cc: Marek Vasut <marex at denx.de>
>>>> ---
>>> checkpatch errors/warnings?
>> checkpatch complains about using spaces for indentation in the
>> arch/arm/dts/keystone-k2g-evm.dts file.  The entire flash node
>> was originally indented using spaces in this file before this patch,
>> so I kept the indentation as it was originally when I updated this
>> file in patch (2/5).  I then fixed all the indentation in this file in
>> commit 4/5 in this series.  Is this okay?
> Better to squash and update commit message.

Will do.  Do you want me to add your reviewed-by tag to the
commit messages?



More information about the U-Boot mailing list