Messages in this thread | | | Date | Wed, 25 Oct 2023 13:55:28 +0200 | From | Neil Armstrong <> | Subject | Re: [PATCH RFC 2/8] arm64: dts: qcom: add initial SM8650 dtsi |
| |
On 25/10/2023 11:01, Konrad Dybcio wrote: > > > On 10/25/23 09:47, Neil Armstrong wrote: >> Add initial DTSI for the Qualcomm SM8650 platform, >> only contains nodes which doesn't depend on interconnect. >> >> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> >> ---[...] > >> + CLUSTER_SLEEP_1: cluster-sleep-1 { >> + compatible = "domain-idle-state"; >> + arm,psci-suspend-param = <0x4100c344>; > I think this parameter signals the AOSS to attempt system > suspend and CLUSTER_SLEEP is a shallower, separate state.
I have a doubt, I'll double check on this,
> > [...] >> + qcom,tcs-config = <ACTIVE_TCS 3>, <SLEEP_TCS 2>, >> + <WAKE_TCS 2>, <CONTROL_TCS 0>; > Is <CONTROL_TCS 0> the correct value?
I assume it is, I'll double check.
Thanks, Neil
> > I think it looks good otherwise > > Konrad
| |