Re: [PATCH 00/14] cpufreq: cpu0: Extend support beyond CPU0, V2

From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: Viresh Kumar <viresh.kumar@linaro.org>
Cc: "Nishanth Menon" <nm@ti.com>, "Andrew Lunn" <andrew@lunn.ch>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	"Tomasz Figa" <t.figa@samsung.com>,
	"Arvind Chauhan" <arvind.chauhan@arm.com>,
	"Rob Herring" <rob.herring@linaro.org>,
	"Kukjin Kim" <kgene.kim@samsung.com>,
	"Michal Simek" <michal.simek@xilinx.com>,
	"Thomas P Abraham" <thomas.ab@samsung.com>,
	"Sebastian Hesselbarth" <sebastian.hesselbarth@gmail.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"Lists linaro-kernel" <linaro-kernel@lists.linaro.org>,
	"Jason Cooper" <jason@lakedaemon.net>,
	"linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>,
	"linux-arm-msm@vger.kernel.org" <linux-arm-msm@vger.kernel.org>,
	"Simon Horman" <horms@verge.net.au>,
	"Gregory Clément" <gregory.clement@free-electrons.com>,
	"Sachin Kamat" <spk.linux@gmail.com>,
	"Stephen Boyd" <sboyd@codeaurora.org>,
	"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 00/14] cpufreq: cpu0: Extend support beyond CPU0, V2
Date: Wed, 16 Jul 2014 19:28:15 +0200	[thread overview]
Message-ID: <20140716192815.786b1866@free-electrons.com> (raw)
In-Reply-To: <CAKohpo=bUMKa752fBXZTb0WPYMfJtuR5uk7z-YaXYV63fhJWtg@mail.gmail.com>

Dear Viresh Kumar,

On Wed, 16 Jul 2014 21:31:54 +0530, Viresh Kumar wrote:
> Cc'ing Thomas,

Thanks. Also adding Jason Cooper, Andrew Lunn, Grégory Clement and
Sebastian Hesselbart (the mvebu platform maintainers, which include
Marvell Armada XP).

> On 8 July 2014 10:20, Viresh Kumar <viresh.kumar@linaro.org> wrote:
> > On 4 July 2014 09:51, Viresh Kumar <viresh.kumar@linaro.org> wrote:
> >> Yeah, having something like what you suggested from DT is the perfect
> >> solution to get over this. The only reason why I am not touching that here
> >> is to not delay other patches just because of that.
> >>
> >> There are separate threads going on for that and probably somebody
> >> else was trying to push for that.
> >>
> >> That's it, nothing more. I would definitely like to use those bindings instead
> >>  of the crazy routines we are trying here, once that is finalized :)
> >
> > Do we have some kind of agreement for this temporary solution? Anyways
> > I will kick the other thread again to resolve the bindings soon.
> 
> Hi Mike/Rafael,
> 
> Thomas has a dependency on the patch adding "find_siblings()":
> http://www.spinics.net/lists/arm-kernel/msg348080.html
> 
> Would it be fine to get this temporary solution (Only within cpufreq-cpu0
> file, so that it doesn't become an API) for now and updating it later once
> bindings are closed?
> 
> I have tried pinging on the other thread as well, but no one replied.
> And it looks those bindings are going to take some time to settle.

Alternatively, I could respin my Armada XP specific cpufreq driver,
until a proper cpufreq-generic driver is sorted out, if that's needed.
Since I was told by Viresh that the cpufreq-generic driver supporting
independent per-CPU clocks would be merged for 3.17, I based my cpufreq
development on that. Please let us know quickly if that's not going to
be the case, so that we can enable cpufreq on Armada XP in 3.17, even
if it's done with a separate cpufreq driver, until cpufreq-generic
issues get sorted out.

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

_______________________________________________
linaro-kernel mailing list
linaro-kernel@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-kernel