On 02/12, Lina Iyer wrote: > @@ -101,6 +105,27 @@ > }; > }; > > + CPU_PD: cpu-pd@0 { > + #power-domain-cells = <0>; > + power-states = <&CLUSTER_RET>, <&CLUSTER_PWR_DWN>; Why isn't this part of the psci node? PSCI is the node that's providing the code/logic for the power domain. > + }; > + > + pd-power-states { > + CLUSTER_RET: power-state@1 { > + state-param = <0x1000010>; > + entry-latency-us = <500>; > + exit-latency-us = <500>; > + residency-us = <2000>; > + }; > + > + CLUSTER_PWR_DWN: power-state@2 { > + state-param = <0x1000030>; > + entry-latency-us = <2000>; > + exit-latency-us = <2000>; > + residency-us = <6000>; > + }; > + }; > + And I would expect these to be put somewhere inside the power domain provider as well? Is this documented somewhere? > psci { > compatible = "arm,psci-1.0"; > method = "smc"; -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html