[PATCH v7 3/8] drivers: cpuidle: implement DT based idle states infrastructure
nicolas.pitre at linaro.org
Wed Aug 13 09:31:11 PDT 2014
On Wed, 13 Aug 2014, Lorenzo Pieralisi wrote:
> On most common ARM systems, the low-power states a CPU can be put into are
> not discoverable in HW and require device tree bindings to describe
> power down suspend operations and idle states parameters.
> In order to enable DT based idle states and configure idle drivers, this
> patch implements the bulk infrastructure required to parse the device tree
> idle states bindings and initialize the corresponding CPUidle driver states
> The parsing API accepts a start index that defines the first idle state
> that should be initialized by the parsing code in order to give new and
> legacy driver flexibility over which states should be parsed using the
> new DT mechanism.
> The idle states list is obtained from the first cpu in the driver
> cpumask, which implicitly means the parsing code expects idle states
> (and related list of phandles) to be the same for all CPUs in the
> CPUidle driver mask. The kernel does not check this assumption, it must
> be enforced by the bootloader to ensure correct system behaviour.
Can we make the kernel a little less reliant on bootloader to ensure
correct system behaviour please? If assumptions are assumed by the
kernel, it should at least print a warning and simply ignore the
information when such assumption are not respected.
> + /*
> + * We get the idle states for the first logical cpu in the
> + * driver mask. The kernel does not check idle states on all
> + * cpus in the driver mask, they are assumed to be the same
> + * by default.
> + */
What if they're not?
More information about the linux-arm-kernel