From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lorenzo Pieralisi Subject: Re: [PATCH v5 1/8] Documentation: arm: define DT idle states bindings Date: Thu, 26 Jun 2014 11:17:07 +0100 Message-ID: <20140626101707.GB25130@e102568-lin.cambridge.arm.com> References: <1403705421-17597-1-git-send-email-lorenzo.pieralisi@arm.com> <1403705421-17597-2-git-send-email-lorenzo.pieralisi@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: Content-Disposition: inline Sender: linux-pm-owner@vger.kernel.org To: Nicolas Pitre Cc: "linux-arm-kernel@lists.infradead.org" , "linux-pm@vger.kernel.org" , "devicetree@vger.kernel.org" , Mark Rutland , Sudeep Holla , Catalin Marinas , Charles Garcia-Tobin , Rob Herring , "grant.likely@linaro.org" , Peter De Schrijver , Santosh Shilimkar , Daniel Lezcano , Amit Kucheria , Vincent Guittot , Antti Miettinen , Stephen Boyd , Kevin Hilman , Sebastian Capella , Tomasz Figa , Mark Brown List-Id: devicetree@vger.kernel.org On Wed, Jun 25, 2014 at 04:56:02PM +0100, Nicolas Pitre wrote: > On Wed, 25 Jun 2014, Lorenzo Pieralisi wrote: > > > ARM based platforms implement a variety of power management schemes that > > allow processors to enter idle states at run-time. > > The parameters defining these idle states vary on a per-platform basis forcing > > the OS to hardcode the state parameters in platform specific static tables > > whose size grows as the number of platforms supported in the kernel increases > > and hampers device drivers standardization. > > > > Therefore, this patch aims at standardizing idle state device tree bindings for > > ARM platforms. Bindings define idle state parameters inclusive of entry methods > > and state latencies, to allow operating systems to retrieve the configuration > > entries from the device tree and initialize the related power management > > drivers, paving the way for common code in the kernel to deal with idle > > states and removing the need for static data in current and previous kernel > > versions. > > > > Reviewed-by: Sebastian Capella > > Signed-off-by: Lorenzo Pieralisi > > Excellent. > > Reviewed-by: Nicolas Pitre Thanks Nico, there are still a couple of niggles to sort out (ie local timer state), but the bulk of the document should be complete I hope. I will postpone adding your (and Seb's) Reviewed-by until we have a final agreement if it is ok with you. Thanks ! Lorenzo