On Tue 23 Jun 15:48 PDT 2020, Konrad Dybcio wrote:
> Add support for SMD RPM, including pm8994 and pmi8994
> regulators.
>
> Signed-off-by: Konrad Dybcio
> ---
> arch/arm64/boot/dts/qcom/msm8994.dtsi | 111 ++
> 1 file changed, 111 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/msm8994.dtsi
> b/arch/arm64/boot/dts/qcom/msm8994.dtsi
> index e7f4b06eb12c..a4edc3be7024 100644
> --- a/arch/arm64/boot/dts/qcom/msm8994.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8994.dtsi
> @@ -159,9 +159,100 @@ smem_mem: smem_region@6a0 {
> };
> };
>
> + smd {
> + compatible = "qcom,smd";
> + rpm {
> + interrupts = ;
> + qcom,ipc = < 8 0>;
Can you please extend qcom-apcs-ipc-mailbox and its binding to add
MSM8994 and use mboxes = < 0>; here instead?
I do see that while I added this support a few years ago I migrated some
of the remoteproc edges, but never any of the RPM ones. But it should
work...
> + qcom,smd-edge = <15>;
> + qcom,local-pid = <0>;
> + qcom,remote-pid = <6>;
> +
> + rpm-requests {
> + compatible = "qcom,rpm-msm8994";
> + qcom,smd-channels = "rpm_requests";
> +
> + rpmcc: rpmcc {
> + compatible = "qcom,rpmcc-msm8994";
> + #clock-cells = <1>;
> + };
> +
> + pm8994_regulators: pm8994-regulators {
We've learned from experience that PMICs, naming of regulators and their
routing are board specific (or at least OEM specific).
So please give rpm-requests a label and add pm8994-regulators and
pmi8994-regulators to it in the board (your case platform) dts(i).
Regards,
Bjorn
> + compatible =
> "qcom,rpm-pm8994-regulators";
> + vdd_l1-supply = <_s1>;
> + vdd_l2_26_28-supply = <_s3>;
> + vdd_l3_11-supply = <_s3>;
> + vdd_l4_27_31-supply = <_s3>;
> + vdd_l5_7-supply = <_s3>;
> + vdd_l6_12_32-supply = <_s5>;
> + vdd_l8_16_30-supply = <_vph_pwr>;
> + vdd_l9_10_18_22-supply =
> <_vph_pwr>;
> + vdd_l13_19_23_24-supply =
> <_vph_pwr>;
> + vdd_l14_15-supply = <_s5>;
> + vdd_l17_29-supply = <_vph_pwr>;
> + vdd_l20_21-supply = <_vph_pwr>;
> + vdd_l25-supply = <_s5>;
> + vdd_lvs1_2 = <_s4>;
> +
> + pm8994_s1: s1 {};
> + pm8994_s2: s2 {};
> + pm8994_s3: s3 {};
> + pm8994_s4: s4 {};
> + pm8994_s5: s5 {};
> + pm8994_s6: s6 {};
> + pm8994_s7: s7 {};
> +
> + pm8994_l1: l1 {};
> + pm8994_l2: l2 {};
> + pm8994_l3: l3 {};
> + pm8994_l4: l4 {};
> + pm8994_l6: l6 {};
> + pm8994_l8: l8 {};
> + pm8994_l9: l9 {};
> + pm8994_l10: l10 {};
> + pm8994_l11: l11 {};
> + pm8994_l12: l12 {};
> + pm8994_l13: l13 {};
> + pm8994_l14: l14 {};
> + pm8994_l15: l15 {};
> + pm8994_l16: l16 {};
> + pm8994_l17: l17 {};
> + pm8994_l18: l18 {};
> + pm8994_l19: l19 {};
> + pm8994_l20: l20 {};
> + pm8994_l21: l21 {};
> + pm8994_l22: l22 {};
> + pm8994_l23: l23 {};
> + pm8994_l24: l24 {};
> + pm8994_l25: l25 {};
> + pm8994_l26: l26 {};
> + pm8994_l27: l27 {};
> + pm8994_l28: l28 {};
> + pm8994_l29: l29 {};
> +
