From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolas Pitre Subject: Re: [PATCH v5 1/8] Documentation: arm: define DT idle states bindings Date: Thu, 26 Jun 2014 15:30:22 -0400 (EDT) Message-ID: References: <1403705421-17597-1-git-send-email-lorenzo.pieralisi@arm.com> <1403705421-17597-2-git-send-email-lorenzo.pieralisi@arm.com> <20140626101707.GB25130@e102568-lin.cambridge.arm.com> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Return-path: In-Reply-To: <20140626101707.GB25130@e102568-lin.cambridge.arm.com> Sender: linux-pm-owner@vger.kernel.org To: Lorenzo Pieralisi 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 Thu, 26 Jun 2014, Lorenzo Pieralisi wrote: > 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. As you wish. The parts I care about are now well covered. I don't think I know enough about timers to comment further. Nicolas From mboxrd@z Thu Jan 1 00:00:00 1970 From: nicolas.pitre@linaro.org (Nicolas Pitre) Date: Thu, 26 Jun 2014 15:30:22 -0400 (EDT) Subject: [PATCH v5 1/8] Documentation: arm: define DT idle states bindings In-Reply-To: <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> <20140626101707.GB25130@e102568-lin.cambridge.arm.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, 26 Jun 2014, Lorenzo Pieralisi wrote: > 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. As you wish. The parts I care about are now well covered. I don't think I know enough about timers to comment further. Nicolas