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

Jagan Teki jagannadh.teki at gmail.com
Tue Jan 23 16:21:29 UTC 2018


On Tue, Jan 23, 2018 at 8:42 PM, Jason Rush <jarush at gmail.com> wrote:
> 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?

yes along with tags from cover-letter patch because patchwork didn't pick those.


More information about the U-Boot mailing list