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

Jagan Teki
2018-01-23T16:21:29Z

On Tue, Jan 23, 2018 at 8:42 PM, Jason Rush <jarush@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@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@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@ti.com>
>>>>>
>>>>> Tested on a socfpga-cyclonev board:
>>>>> Tested-by: Simon Goldschmidt <sgoldschmidt@de.pepperl-fuchs.com>
>>>>>
>>>>> Signed-off-by: Jason Rush <jarush@gmail.com>
>>>>> Cc: Marek Vasut <marex@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.