All of lore.kernel.org
 help / color / mirror / Atom feed
From: Heiko Stuebner <heiko@sntech.de>
To: Brian Norris <briannorris@chromium.org>
Cc: linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org,
	Caesar Wang <wxt@rock-chips.com>,
	Doug Anderson <dianders@chromium.org>,
	devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
	Stephen Barber <smbarber@chromium.org>,
	linux-arm-kernel@lists.infradead.org,
	Chris Zhong <zyw@rock-chips.com>
Subject: Re: [PATCH 4/9] arm64: dts: rockchip: support dwc3 USB for rk3399
Date: Wed, 07 Dec 2016 20:09:35 +0100	[thread overview]
Message-ID: <5292364.Xa7yl762Dv@phil> (raw)
In-Reply-To: <20161207190301.GA107036@google.com>

Am Mittwoch, 7. Dezember 2016, 11:03:02 CET schrieb Brian Norris:
> On Wed, Dec 07, 2016 at 08:01:23PM +0100, Heiko Stuebner wrote:
> > Am Mittwoch, 7. Dezember 2016, 09:52:08 CET schrieb Brian Norris:
> > > But pcie@f8000000 is also out of order then. I guess maybe
> > > that's the only one then.
> > 
> > Yep, pcie is misplaced as sadly sometimes I miss those errors as well.
> 
> OK. Do you want me to patch that at the end of my series, or is that
> unnecessary churn?

I'm still pondering [and am investing way to much brain cells in trying to 
decide this ;-) ], but tend to want to move it.

That way the dtsi gives the correct sorting impression for future dt-patches 
of which I guess we'll see a lot in the time to come.

So yes, please move it as well.


Thanks
Heiko

WARNING: multiple messages have this Message-ID (diff)
From: heiko@sntech.de (Heiko Stuebner)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 4/9] arm64: dts: rockchip: support dwc3 USB for rk3399
Date: Wed, 07 Dec 2016 20:09:35 +0100	[thread overview]
Message-ID: <5292364.Xa7yl762Dv@phil> (raw)
In-Reply-To: <20161207190301.GA107036@google.com>

Am Mittwoch, 7. Dezember 2016, 11:03:02 CET schrieb Brian Norris:
> On Wed, Dec 07, 2016 at 08:01:23PM +0100, Heiko Stuebner wrote:
> > Am Mittwoch, 7. Dezember 2016, 09:52:08 CET schrieb Brian Norris:
> > > But pcie at f8000000 is also out of order then. I guess maybe
> > > that's the only one then.
> > 
> > Yep, pcie is misplaced as sadly sometimes I miss those errors as well.
> 
> OK. Do you want me to patch that at the end of my series, or is that
> unnecessary churn?

I'm still pondering [and am investing way to much brain cells in trying to 
decide this ;-) ], but tend to want to move it.

That way the dtsi gives the correct sorting impression for future dt-patches 
of which I guess we'll see a lot in the time to come.

So yes, please move it as well.


Thanks
Heiko

  reply	other threads:[~2016-12-07 19:09 UTC|newest]

Thread overview: 79+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-02  2:27 [PATCH 0/9] arm64: dts: rockchip: support Google Kevin Brian Norris
2016-12-02  2:27 ` Brian Norris
2016-12-02  2:27 ` Brian Norris
2016-12-02  2:27 ` [PATCH 1/9] arm64: dts: Add symlinks for cros-ec-keyboard and cros-ec-sbs Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27 ` [PATCH 2/9] arm64: dts: rockchip: add rk3399 thermal_zones phandle Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-07 16:55   ` Heiko Stuebner
2016-12-07 16:55     ` Heiko Stuebner
2016-12-02  2:27 ` [PATCH 3/9] arm64: dts: rockchip: add rk3399 eDP HPD pinctrl Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-07 16:56   ` Heiko Stuebner
2016-12-07 16:56     ` Heiko Stuebner
2016-12-02  2:27 ` [PATCH 4/9] arm64: dts: rockchip: support dwc3 USB for rk3399 Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-07 17:09   ` Heiko Stuebner
2016-12-07 17:09     ` Heiko Stuebner
2016-12-07 17:52     ` Brian Norris
2016-12-07 17:52       ` Brian Norris
2016-12-07 17:52       ` Brian Norris
2016-12-07 19:01       ` Heiko Stuebner
2016-12-07 19:01         ` Heiko Stuebner
2016-12-07 19:01         ` Heiko Stuebner
2016-12-07 19:03         ` Brian Norris
2016-12-07 19:03           ` Brian Norris
2016-12-07 19:03           ` Brian Norris
2016-12-07 19:09           ` Heiko Stuebner [this message]
2016-12-07 19:09             ` Heiko Stuebner
2016-12-02  2:27 ` [PATCH 5/9] arm64: dts: rockchip: add rk3399 OPPs Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27 ` [PATCH 6/9] dt-bindings: Document rk3399 Gru/Kevin Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-07 17:12   ` Heiko Stuebner
2016-12-07 17:12     ` Heiko Stuebner
2016-12-07 17:12     ` Heiko Stuebner
2016-12-07 17:41     ` Brian Norris
2016-12-07 17:41       ` Brian Norris
2016-12-07 17:41       ` Brian Norris
2016-12-07 19:15       ` Heiko Stuebner
2016-12-07 19:15         ` Heiko Stuebner
2016-12-09 17:54   ` Rob Herring
2016-12-09 17:54     ` Rob Herring
2016-12-09 18:28     ` Heiko Stuebner
2016-12-09 18:28       ` Heiko Stuebner
2016-12-09 18:28       ` Heiko Stuebner
2016-12-02  2:27 ` [PATCH 7/9] arm64: dts: rockchip: add Gru/Kevin DTS Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27 ` [PATCH 8/9] arm64: dts: rockchip: partially describe PWM regulators for Gru Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-07 16:48   ` Heiko Stuebner
2016-12-07 16:48     ` Heiko Stuebner
2016-12-07 16:48     ` Heiko Stuebner
2016-12-07 17:09     ` Brian Norris
2016-12-07 17:09       ` Brian Norris
2016-12-13 17:48       ` Heiko Stuebner
2016-12-13 17:48         ` Heiko Stuebner
2016-12-22 16:09         ` Heiko Stübner
2016-12-22 16:09           ` Heiko Stübner
2016-12-22 16:09           ` Heiko Stübner
2016-12-02  2:27 ` [PATCH 9/9] arm64: dts: rockchip: add regulator info for Kevin digitizer Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-02  2:27   ` Brian Norris
2016-12-13 17:36   ` Heiko Stuebner
2016-12-13 17:36     ` Heiko Stuebner
2016-12-13 17:36     ` Heiko Stuebner
2017-02-08  1:01 ` [PATCH 0/9] arm64: dts: rockchip: support Google Kevin Heiko Stuebner
2017-02-08  1:01   ` Heiko Stuebner
2017-02-08  1:01   ` Heiko Stuebner
2017-02-08  1:03   ` Brian Norris
2017-02-08  1:03     ` Brian Norris
2017-02-08  1:03     ` Brian Norris

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=5292364.Xa7yl762Dv@phil \
    --to=heiko@sntech.de \
    --cc=briannorris@chromium.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dianders@chromium.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=robh+dt@kernel.org \
    --cc=smbarber@chromium.org \
    --cc=wxt@rock-chips.com \
    --cc=zyw@rock-chips.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.