Messages in this thread | | | Date | Fri, 27 Jan 2023 13:54:54 +0000 | Subject | Re: [PATCH v2 2/4] arm64: dts: qcom: sm6350: Add camera clock controller | From | Bryan O'Donoghue <> |
| |
On 27/01/2023 13:11, Luca Weiss wrote: >> Doh I meant to say a power-domain to an mmcx a la >> >> power-domains = <&rpmhpd SM8250_MMCX>; >> required-opps = <&rpmhpd_opp_low_svs>; >> >> TITAN_TOP should be in your cci and camss dt nodes. > Okay, that makes more sense. > > What I don't quite understand is why sm8250 only has MMCX listed there > since downstream has both vdd_mx-supply = <&VDD_MX_LEVEL> and > vdd_mm-supply = <&VDD_MMCX_LEVEL> and both "supplies" are used for > different clocks using .vdd_class
power-domains = <&rpmhpd SM8250_MMCX>; == MMCX_LEVEL required for camcc power-domains = <&camcc TITAN_TOP_GDSC>; required for cci/camss
now that you ask the question about MX_LEVEL you're making me doubt we have a 100% complete representation upstream TB perfectly honest, warrants a deep dive..
I just remember that on 8250 we tripped over MMCX not being switched on when - display I think was switched off.
--- bod
| |