On Mon, Feb 03, 2020 at 07:05:39PM +0530, Maulik Shah wrote: > Add device bindings for cpuidle states for cpu devices. > > Cc: devicetree@xxxxxxxxxxxxxxx > Signed-off-by: Maulik Shah <mkshah@xxxxxxxxxxxxxx> > --- > arch/arm64/boot/dts/qcom/sc7180.dtsi | 78 ++++++++++++++++++++++++++++++++++++ > 1 file changed, 78 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi > index 8011c5f..0aa0ced 100644 > --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi > +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi > @@ -86,6 +86,9 @@ > compatible = "arm,armv8"; > reg = <0x0 0x0>; > enable-method = "psci"; > + cpu-idle-states = <&LITTLE_CPU_SLEEP_0 > + &LITTLE_CPU_SLEEP_1 > + &CLUSTER_SLEEP_0>; These entries are deleted again by the next patch in this series ('arm64: dts: qcom: sc7180: Convert to the hierarchical CPU topology layout'). What is the point in adding them in the first place?