Lina Iyer <lina.iyer@xxxxxxxxxx> writes: > Add allowable C-States for each cpu using the cpu-idle-states node. > ARM spec dictates WFI as the default idle state at 0. Support standalone > power collapse (power down that does not affect any SoC idle states) for > each cpu. > > Signed-off-by: Lina Iyer <lina.iyer@xxxxxxxxxx> [...] > @@ -55,6 +59,22 @@ > cache-level = <2>; > qcom,saw = <&saw_l2>; > }; > + > + idle-states { > + CPU_WFI: cpu-idle-state-0 { ^^^^^^^^^^^^^^^^ Minor nit: This name is what's being used to populate the idle state's ->name field in the CPUidle driver, which in turn is used by tools like powertop to display state names, etc. So first, that name field has to be short (it currently gets truncated, dropping the zero), and second that field should be more descriptive, like 'WFI', so viewing in the various tools shows something useful. Thanks, Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html