Re: [PATCH v5 1/8] Documentation: arm: define DT idle states bindings

Nicolas Pitre
2014-06-26T19:30:25Z

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 <sebcape@gmail.com>
> > > Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
> >
> > Excellent.
> >
> > Reviewed-by: Nicolas Pitre <nico@linaro.org>
>
> 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