RE: [PATCH 1/3] PM / OPP: Add support for descending order for cpufreq table

From: Jonghwan Choi <jhbird.choi@samsung.com>
To: "'Viresh Kumar'" <viresh.kumar@linaro.org>,
	"'Nishanth Menon'" <nm@ti.com>
Cc: "'Linux PM list'" <linux-pm@vger.kernel.org>,
	"'open list'" <linux-kernel@vger.kernel.org>,
	"'Rafael J. Wysocki'" <rjw@rjwysocki.net>,
	"'Len Brown'" <len.brown@intel.com>,
	"'Amit Daniel Kachhap'" <amit.daniel@samsung.com>
Subject: RE: [PATCH 1/3] PM / OPP: Add support for descending order for cpufreq table
Date: Wed, 07 May 2014 08:43:29 +0900	[thread overview]
Message-ID: <003e01cf6984$fb950280$f2bf0780$@samsung.com> (raw)
In-Reply-To: <CAKohpokfcptseU6ST+e_GfU6poVJxXCXMeoYCvT65060v5GJYQ@mail.gmail.com>

Hi  

My holiday is finished.

I implemented another cpufreq driver. And that driver also have to use exynos_sort_descend_freq_table().
Then exynos5440 and new cpufreq have a duplicate function.(exynos_sort_descend_freq_table().
So I want to solve it.

Thanks.~

> -----Original Message-----
> From: Viresh Kumar [mailto:viresh.kumar@linaro.org]
> Sent: Monday, May 05, 2014 11:39 PM
> To: Nishanth Menon
> Cc: Jonghwan Choi; Linux PM list; open list; Rafael J. Wysocki; Len Brown;
> Amit Daniel Kachhap
> Subject: Re: [PATCH 1/3] PM / OPP: Add support for descending order for
> cpufreq table
> 
> On 5 May 2014 19:53, Nishanth Menon <nm@ti.com> wrote:
> > ceil and floor allows us to walk down the opp entries the direction we
> > want it to.
> > one can convert that data any way one wants it - especially when
> > custom mapping such as this is desired.
> 
> Yeah, but doing that for every frequency transition is not right.
> Otherwise they already have a solution, where they reverse order of
> frequencies for their driver. exynos_sort_descend_freq_table().
> 
> Probably that's a better solution then :)
> 
> But, Jonghwan was probably trying to get this solved in the framework only,
> in case anybody else needs it.
> 
> --
> viresh