On 11/5/25 10:21 AM, Imran Shaik wrote: > > > On 11/5/2025 1:01 AM, Konrad Dybcio wrote: >> From: Konrad Dybcio <[email protected]> >> >> To make sure that power rail is voted for, wire it up to its consumers. >> >> Fixes: 9bd07f2c558f ("arm64: dts: qcom: sc8280xp: Add in CAMCC for sc8280xp") >> Fixes: 152d1faf1e2f ("arm64: dts: qcom: add SC8280XP platform") >> Signed-off-by: Konrad Dybcio <[email protected]> >> --- >> arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 22 ++++++++++++++++------ >> 1 file changed, 16 insertions(+), 6 deletions(-) >> >> diff --git a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi >> b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi >> index 5334adebf278..643a61cc91b4 100644 >> --- a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi >> +++ b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi >> @@ -4586,8 +4586,10 @@ camcc: clock-controller@ad00000 { >> <&rpmhcc RPMH_CXO_CLK>, >> <&rpmhcc RPMH_CXO_CLK_A>, >> <&sleep_clk>; >> - power-domains = <&rpmhpd SC8280XP_MMCX>; >> - required-opps = <&rpmhpd_opp_low_svs>; >> + power-domains = <&rpmhpd SC8280XP_MMCX>, >> + <&rpmhpd SC8280XP_MXC>; > > I see that none of the CAMCC PLLs/clocks are on MXC rail on SC8280XP target. > So, MXC support is not required for CAMCC.
Hm, to the extent that camss is supported upstream, I could indeed boot without pd_ignore_unused and the camera actually kept working.. I see in docs that there are links between MXC and various IPs within camera so I'm a little confused. Would you know more? I wasn't able to test venus because it locks up either way.. Konrad
