From: Jason Rush <jarush@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v5 2/5] dts: cadence_spi: Sync DT bindings with Linux
Date: Tue, 23 Jan 2018 09:12:10 -0600 [thread overview]
Message-ID: <fb54d6d6-877e-8a6e-9254-185da89be251@gmail.com> (raw)
In-Reply-To: <CAD6G_RQuuL3MHw5z5c_q=3U0cQoH1KEOgHzU-Y+LeJPSNQjyFg@mail.gmail.com>
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?
next prev parent reply other threads:[~2018-01-23 15:12 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-06 18:17 [U-Boot] [PATCH v5 0/5] spi: cadence_spi: Adopt Linux DT bindings Jason Rush
2018-01-06 18:17 ` [U-Boot] [PATCH v5 1/5] spi: cadence_spi: Sync DT bindings with Linux Jason Rush
2018-01-23 9:12 ` Jagan Teki
2018-01-06 18:17 ` [U-Boot] [PATCH v5 2/5] dts: " Jason Rush
2018-01-23 9:16 ` Jagan Teki
2018-01-23 14:15 ` Jason Rush
2018-01-23 14:25 ` Jagan Teki
2018-01-23 15:12 ` Jason Rush [this message]
2018-01-23 16:21 ` Jagan Teki
2018-01-06 18:18 ` [U-Boot] [PATCH v5 3/5] config: cadence_spi: Remove defines read from DT Jason Rush
2018-01-23 9:15 ` Jagan Teki
2018-01-06 18:18 ` [U-Boot] [PATCH v5 4/5] dts: k2g: Clean-up indentation Jason Rush
2018-01-23 9:17 ` Jagan Teki
2018-01-06 18:18 ` [U-Boot] [PATCH v5 5/5] dts: cadence_spi: Update documentation for DT bindings Jason Rush
2018-01-23 9:19 ` Jagan Teki
2018-01-07 11:41 ` [U-Boot] [PATCH v5 0/5] spi: cadence_spi: Adopt Linux " Marek Vasut
2018-01-15 12:03 ` Simon Goldschmidt
2018-01-23 8:13 ` Simon Goldschmidt
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=fb54d6d6-877e-8a6e-9254-185da89be251@gmail.com \
--to=jarush@gmail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox