linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Olof Johansson <olof@lixom.net>
To: John Stultz <john.stultz@linaro.org>
Cc: lkml <linux-kernel@vger.kernel.org>,
	arm@kernel.org, Michael Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@codeaurora.org>,
	Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
	Wei Xu <xuwei5@hisilicon.com>, Guodong Xu <guodong.xu@linaro.org>,
	Zhangfei Gao <zhangfei.gao@linaro.org>
Subject: Re: [PATCH 0/2 v3] Add pl031 RTC support for Hi6220
Date: Tue, 5 Jul 2016 22:22:20 -0700	[thread overview]
Message-ID: <20160706052220.GK23212@localhost> (raw)
In-Reply-To: <1467247725-3665-1-git-send-email-john.stultz@linaro.org>

On Wed, Jun 29, 2016 at 05:48:43PM -0700, John Stultz wrote:
> This patchset enables the pl031 RTC on the Hi6220 SoC.
> 
> I'd like to submit it to be merged.
> 
> Wei has acked the second patch (modulo a whitespace fix which
> I've included in this v3), so it seems like both could go
> through the clk tree.
> 
> But Wei also seemed open to pulling in a clk tree branch
> as it goes through arm-soc.
> 
> Michael/Stephen: If there's no other objections, could you
> queue the first patch and make it avilable via the branch for
> Wei, or just take both patches?

I happen to dread these kind of patchsets these days. There's added
dependencies across trees just because a defined name for the clock
number is added to a header file.

I much prefer to use numerical clocks for one release, and then once
everything is in, switch over to the defines in the DTS.

That way there are no dependencies, no need to setup a shared branch
for a simple 3-line patch, etc.

So, mind respinning the DTS piece?

Thanks!


-Olof

  parent reply	other threads:[~2016-07-06  6:07 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-30  0:48 [PATCH 0/2 v3] Add pl031 RTC support for Hi6220 John Stultz
2016-06-30  0:48 ` [PATCH 1/2 v3] clk: hi6220: Add RTC clock for pl031 John Stultz
2016-06-30 19:15   ` Stephen Boyd
2016-06-30 19:23     ` John Stultz
2016-06-30  0:48 ` [PATCH 2/2 v3] arm64: dts: hi6220: Add pl031 RTC support John Stultz
2016-06-30 15:12 ` [PATCH 0/2 v3] Add pl031 RTC support for Hi6220 Wei Xu
2016-07-06  5:22 ` Olof Johansson [this message]
2016-07-06  6:55   ` John Stultz
2016-07-06  7:04     ` Olof Johansson
2016-07-06  7:20       ` John Stultz
2016-07-06  7:38         ` Arnd Bergmann
2016-07-06  8:13           ` Wei Xu
2016-07-12  1:03             ` John Stultz
2016-07-07  0:19           ` Michael Turquette
2016-07-07  8:13             ` Arnd Bergmann
     [not found]               ` <146794382979.73491.3322475351079454720@resonance>
2016-07-11 20:21                 ` Arnd Bergmann
     [not found]                   ` <146827441381.73491.4865692343236492728@resonance>
2016-07-12  8:51                     ` Arnd Bergmann
     [not found]                       ` <146834751937.73491.12265160509757545340@resonance>
2016-07-14 14:30                         ` Arnd Bergmann
2016-07-07  0:58           ` John Stultz
2016-07-07  8:22             ` Arnd Bergmann
2016-07-08  2:21               ` Michael Turquette
2016-07-15  8:06                 ` Arnd Bergmann
2016-07-11  1:30             ` Guodong Xu

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=20160706052220.GK23212@localhost \
    --to=olof@lixom.net \
    --cc=arm@kernel.org \
    --cc=guodong.xu@linaro.org \
    --cc=john.stultz@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=pawel.moll@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=sboyd@codeaurora.org \
    --cc=xuwei5@hisilicon.com \
    --cc=zhangfei.gao@linaro.org \
    /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;
as well as URLs for NNTP newsgroup(s).