public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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?

  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