From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans-Christian Egtvedt Subject: Re: [PATCH 04/37] cpufreq: at32ap: don't initialize part of policy that is set by core too Date: Wed, 14 Aug 2013 16:44:49 +0200 Message-ID: <20130814144449.GA9485@samfundet.no> References: <6fe9b31a6e199052e0a91b38c8f6d110e9f57b28.1376486713.git.viresh.kumar@linaro.org> <20130814143535.GB3263@samfundet.no> Mime-Version: 1.0 Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-pm-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Viresh Kumar Cc: "Rafael J. Wysocki" , Lists linaro-kernel , Patch Tracking , "cpufreq@vger.kernel.org" , "linux-pm@vger.kernel.org" , Linux Kernel Mailing List Around Wed 14 Aug 2013 20:07:44 +0530 or thereabout, Viresh Kumar wrote: > On 14 August 2013 20:05, Hans-Christian Egtvedt wrote: >> Around Wed 14 Aug 2013 19:01:53 +0530 or thereabout, Viresh Kumar wrote: >>> Many common initializations of struct policy are moved to core now and hence >>> this driver doesn't need to do it. This patch removes such code. >>> >>> Most recent of those changes is to call ->get() in the core after calling >>> ->init(). >> >> It would be easier to review if you add me to CC on the new APIs you >> implement as well. Now I had to look up what the ->get() call did. > > I know my track record hasn't been impressive on this subject but I am > sure I am following this since sometime now :) > > https://lkml.org/lkml/2013/8/14/285 Sorry, my bad, I actually had gotten it, it was just so too short to notice. -- mvh Hans-Christian Egtvedt