linux-kernel - Re: [PATCH v2 3/3] ARM: shmobile: kzm9d: Use common clock framework

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20140106081030.GR24653@verge.net.au>
Date:	Mon, 6 Jan 2014 17:10:33 +0900
From:	Simon Horman <horms@...ge.net.au>
To:	Mike Turquette <mturquette@...aro.org>
Cc:	Magnus Damm <magnus.damm@...il.com>, takasi-y@....dti.ne.jp,
	SH-Linux <linux-sh@...r.kernel.org>,
	Ben Dooks <ben.dooks@...ethink.co.uk>,
	Shinya Kuribayashi <shinya.kuribayashi.px@...esas.com>,
	Laurent Pinchart <laurent.pinchart@...asonboard.com>,
	devicetree@...r.kernel.org,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 3/3] ARM: shmobile: kzm9d: Use common clock framework

On Mon, Dec 23, 2013 at 05:00:36PM +0900, Simon Horman wrote:
> On Sun, Dec 22, 2013 at 10:41:03PM -0800, Mike Turquette wrote:
> > Quoting Simon Horman (2013-12-01 18:25:51)
> > > On Tue, Oct 29, 2013 at 01:42:41PM +0900, Simon Horman wrote:
> > > > On Wed, Oct 09, 2013 at 07:39:18PM +0900, Simon Horman wrote:
> > > > > On Wed, Oct 09, 2013 at 04:29:56PM +0900, Magnus Damm wrote:
> > > > > > Hi Simon,
> > > > > > 
> > > > > > On Wed, Oct 9, 2013 at 3:54 PM, Simon Horman <horms@...ge.net.au> wrote:
> > > > > > > On Wed, Oct 09, 2013 at 01:59:46PM +0900, Magnus Damm wrote:
> > > > > > >> Hi Simon,
> > > > > > >>
> > > > > > >> On Wed, Oct 9, 2013 at 12:40 PM, Simon Horman <horms@...ge.net.au> wrote:
> > > > > > >> > On Tue, Oct 08, 2013 at 02:34:03PM +0900, takasi-y@....dti.ne.jp wrote:
> > > > > > >> >> Use common clock framework version of clock
> > > > > > >> >>  drivers/clk/shmobile/clk-emev2.c
> > > > > > >> >> instead of sh-clkfwk version
> > > > > > >> >>  arch/arm/mach-shmobile/clock-emev2.c
> > > > > > >> >> when it is configured as a part of multi-platform.
> > > > > > >> >>
> > > > > > >> >> Signed-off-by: Takashi Yoshii <takasi-y@....dti.ne.jp>
> > > > > > >> >
> > > > > > >> > Thanks.
> > > > > > >> >
> > > > > > >> > I plan to add this patch to a new topic branch,
> > > > > > >> > topic/emev2-common-clock, in the renesas tree and
> > > > > > >> > queue it up from there for inclusion in mainline
> > > > > > >> > if/when the first patch of this series is accepted
> > > > > > >> > by Mike Turquette.
> > > > > > >>
> > > > > > >> Thanks for picking up patches, Simon.
> > > > > > >>
> > > > > > >> I think you can simply merge this patch after the following series:
> > > > > > >>
> > > > > > >> [PATCH 00/05] ARM: shmobile: KZM9D Multiplatform update
> > > > > > >
> > > > > > > This is already queued up.
> > > > > > 
> > > > > > Yes, I know, thanks for that. I tried to explain the dependency in the
> > > > > > 00/00 cover letter but I will try to be more clear next time!
> > > > > > 
> > > > > > >> There are no build time dependencies on patch 1 and 2 so this patch
> > > > > > >> can be merged independently. Regarding run-time operation, the
> > > > > > >> multiplatform series above makes KZM9D DT reference only build for
> > > > > > >> multiplatform, and in such case CCF is required.
> > > > > > >>
> > > > > > >> So if you want to keep KZM9D DT reference working until Mike Turquette
> > > > > > >> accepts the CCF bits, then I recommend you to wait with "[PATCH 00/05]
> > > > > > >> ARM: shmobile: KZM9D Multiplatform update" until all EMEV2 CCF bits
> > > > > > >> have been merged.
> > > > > > >>
> > > > > > >> Another way is to merge "[PATCH 00/05] ARM: shmobile: KZM9D
> > > > > > >> Multiplatform update" before the EMEV2 CCF bits, but if so you may as
> > > > > > >> well merge this patch as well IMO. This
> > > > > > >> multiplatform-update-series-merge-before-CCF plan will result in
> > > > > > >> untestable KZM9D DT reference until EMEV2 CCF is merged. Either way is
> > > > > > >> fine with me.
> > > > > > >
> > > > > > > I am mainly concerned that the bindings may change before
> > > > > > > they are finally merged. And I thought it would be nice to avoid
> > > > > > > having to fix up the usage of the bindings if they change.
> > > > > > 
> > > > > > Sure, but the code in this patch looks like it simply starts DT CCF
> > > > > > probing regardless of what the bindings look like. So I don't think
> > > > > > you will have to worry about actual bindings detail.
> > > > > > 
> > > > > > > But I'm happy to just queue-up patches 2 and 3 of this series
> > > > > > > now if you prefer.
> > > > > > 
> > > > > > Well, I meant patch 3 only. Patch 1 and 2 are tied together by dependencies. =)
> > > > > 
> > > > > Thanks, I understand now.
> > > > > 
> > > > > I'll queue-up 3/3.
> > > > 
> > > > Which I did.
> > > > 
> > > > Then some time passed...
> > > > 
> > > > Mike has accepted my pull-request for patch 1 of this series
> > > > and accordingly I have queued-up this patch for v3.14.
> > > > It should appear in the devel branch the next time that I push it.
> > > 
> > > I was mistaken. It was not queued up  for v3.14.
> > > And still has not been queued up.
> > > 
> > > I have un-queued this patch accordingly.
> > 
> > I have pulled in the emev2 SMU clock support patch now.
> 
> Thanks

I have queued up kzm9d this patch again.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/