On Sun 24 Mar 09:32 PDT 2019, Vinod Koul wrote:
> From: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@xxxxxxxxxx>
>
> The PMS405 sports 5 SMPS and 13 LDO regulators.
>
> Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@xxxxxxxxxx>
> Signed-off-by: Khasim Syed Mohammed <khasim.mohammed@xxxxxxxxxx>
> Signed-off-by: Vinod Koul <vkoul@xxxxxxxxxx>
> ---
> arch/arm64/boot/dts/qcom/pms405.dtsi | 20 ++++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/pms405.dtsi b/arch/arm64/boot/dts/qcom/pms405.dtsi
> index 1bb836d1e8aa..8da4dee1db9b 100644
> --- a/arch/arm64/boot/dts/qcom/pms405.dtsi
> +++ b/arch/arm64/boot/dts/qcom/pms405.dtsi
> @@ -131,4 +131,24 @@
> interrupts = <0x0 0x61 0x1 IRQ_TYPE_NONE>;
> };
> };
> +
> + pms405_1: pms405@1 {
> + compatible = "qcom,spmi-pmic";
> + reg = <0x1 SPMI_USID>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + regulators {
Label the regulators node and keep the compatible here. s3-supply and
pms405_s3 are board specific, so define these in the qcs404-evb.dtsi
instead.
Also double check the EVB schematics for the name of the line coming out
of S3; I would assume vdd_apc or something like that.
Regards,
Bjorn
> + compatible = "qcom,pms405-regulators";
> + vdd_s3-supply = <&pms405_s3>;
> +
> + pms405_s3: s3 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-name = "vdd_cpu";
> + regulator-min-microvolt = <1048000>;
> + regulator-max-microvolt = <1352000>;
> + };
> + };
> + };
> };
> --
> 2.20.1
>
