| Message ID | 20250417-b4-linux-next-25-03-13-dtsi-x1e80100-camss-v7-5-3fd4124cf35a@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | Add dt-bindings and dtsi changes for CAMSS on x1e80100 silicon | expand |
On 4/17/25 1:27 PM, Bryan O'Donoghue wrote: > Add pmic,id = m rpmh to regulator definitions. This regulator set provides > vreg_l3m_1p8 the regulator for the ov08x40 RGB sensor on the CRD. > > Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> > --- > arch/arm64/boot/dts/qcom/x1-crd.dtsi | 30 ++++++++++++++++++++++++++++++ > 1 file changed, 30 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/x1-crd.dtsi b/arch/arm64/boot/dts/qcom/x1-crd.dtsi > index e2f6e342db7e2f7cfbda17cbe28199546de5449d..74bf2f48d93522d3f5b7ca990c06519ca664d905 100644 > --- a/arch/arm64/boot/dts/qcom/x1-crd.dtsi > +++ b/arch/arm64/boot/dts/qcom/x1-crd.dtsi > @@ -858,6 +858,36 @@ vreg_l3j_0p8: ldo3 { > regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; > }; > }; > + > + regulators-8 { > + compatible = "qcom,pm8010-rpmh-regulators"; > + qcom,pmic-id = "m"; > + > + vdd-l1-l2-supply = <&vreg_s5j_1p2>; > + vdd-l3-l4-supply = <&vreg_s4c_1p8>; > + vdd-l7-supply = <&vreg_bob1>; > + > + vreg_l3m_1p8: ldo3 { > + regulator-name = "vreg_l3m_1p8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1808000>; Is this and below +8000mV a fluke? > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; > + }; > + > + vrer_l4m_1p8: ldo4 { > + regulator-name = "vrer_l4m_1p8"; typo: vrer-vreg > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1808000>; > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; > + }; > + > + vreg_l7m_2p9: ldo7 { > + regulator-name = "vreg_l7m_2p9"; > + regulator-min-microvolt = <2912000>; > + regulator-max-microvolt = <2912000>; PM8010 coupled with SM8450 has: regulator-min-microvolt = <2912000>; regulator-max-microvolt = <3304000>; although the x1e crd schem calls this pin ...2p9 (though the names are doubtful at times too) - please check if the consumer is happy with 2.9v
diff --git a/arch/arm64/boot/dts/qcom/x1-crd.dtsi b/arch/arm64/boot/dts/qcom/x1-crd.dtsi index e2f6e342db7e2f7cfbda17cbe28199546de5449d..74bf2f48d93522d3f5b7ca990c06519ca664d905 100644 --- a/arch/arm64/boot/dts/qcom/x1-crd.dtsi +++ b/arch/arm64/boot/dts/qcom/x1-crd.dtsi @@ -858,6 +858,36 @@ vreg_l3j_0p8: ldo3 { regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; }; }; + + regulators-8 { + compatible = "qcom,pm8010-rpmh-regulators"; + qcom,pmic-id = "m"; + + vdd-l1-l2-supply = <&vreg_s5j_1p2>; + vdd-l3-l4-supply = <&vreg_s4c_1p8>; + vdd-l7-supply = <&vreg_bob1>; + + vreg_l3m_1p8: ldo3 { + regulator-name = "vreg_l3m_1p8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1808000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vrer_l4m_1p8: ldo4 { + regulator-name = "vrer_l4m_1p8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1808000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l7m_2p9: ldo7 { + regulator-name = "vreg_l7m_2p9"; + regulator-min-microvolt = <2912000>; + regulator-max-microvolt = <2912000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + }; }; &gpu {
Add pmic,id = m rpmh to regulator definitions. This regulator set provides vreg_l3m_1p8 the regulator for the ov08x40 RGB sensor on the CRD. Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> --- arch/arm64/boot/dts/qcom/x1-crd.dtsi | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+)