linux-kernel - Re: [PATCH 08/11] arm64: dts: qcom: sm8150: align RPMh regulator nodes with bindings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ed921920-44b5-21ef-8ed5-82820482ada6@linaro.org>
Date:   Fri, 27 Jan 2023 12:56:52 +0100
From:   Konrad Dybcio <konrad.dybcio@...aro.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <andersson@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 08/11] arm64: dts: qcom: sm8150: align RPMh regulator
 nodes with bindings



On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@...aro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sa8155p-adp.dts                  | 4 ++--
>  arch/arm64/boot/dts/qcom/sm8150-hdk.dts                   | 6 +++---
>  arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts | 6 +++---
>  arch/arm64/boot/dts/qcom/sm8150-mtp.dts                   | 6 +++---
>  arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi   | 4 ++--
>  5 files changed, 13 insertions(+), 13 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sa8155p-adp.dts b/arch/arm64/boot/dts/qcom/sa8155p-adp.dts
> index eafdfbbf40b9..459384ec8f23 100644
> --- a/arch/arm64/boot/dts/qcom/sa8155p-adp.dts
> +++ b/arch/arm64/boot/dts/qcom/sa8155p-adp.dts
> @@ -72,7 +72,7 @@ queue0 {
>  };
>  
>  &apps_rsc {
> -	pmm8155au-1-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pmm8155au-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -201,7 +201,7 @@ vreg_l17a_2p96: ldo17 {
>  		};
>  	};
>  
> -	pmm8155au-2-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmm8155au-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8150-hdk.dts b/arch/arm64/boot/dts/qcom/sm8150-hdk.dts
> index 3331ee957d64..8f014a232526 100644
> --- a/arch/arm64/boot/dts/qcom/sm8150-hdk.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8150-hdk.dts
> @@ -56,7 +56,7 @@ key-vol-up {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -211,7 +211,7 @@ vreg_l17a_3p0: ldo17 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -324,7 +324,7 @@ vreg_l11c_3p3: ldo11 {
>  		};
>  	};
>  
> -	pm8009-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8009-rpmh-regulators";
>  		qcom,pmic-id = "f";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts b/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
> index 5397fba9417b..69024f7c7f10 100644
> --- a/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
> @@ -61,7 +61,7 @@ key-vol-up {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -216,7 +216,7 @@ vreg_l17a_3p0: ldo17 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -329,7 +329,7 @@ vreg_l11c_3p3: ldo11 {
>  		};
>  	};
>  
> -	pm8009-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8009-rpmh-regulators";
>  		qcom,pmic-id = "f";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8150-mtp.dts b/arch/arm64/boot/dts/qcom/sm8150-mtp.dts
> index 46b5cf9a1192..eff995a07ab7 100644
> --- a/arch/arm64/boot/dts/qcom/sm8150-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8150-mtp.dts
> @@ -51,7 +51,7 @@ vreg_s4a_1p8: pm8150-s4 {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -206,7 +206,7 @@ vreg_l17a_3p0: ldo17 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -319,7 +319,7 @@ vreg_l11c_3p3: ldo11 {
>  		};
>  	};
>  
> -	pm8009-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8009-rpmh-regulators";
>  		qcom,pmic-id = "f";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi b/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi
> index 64602748c657..ff77cc3c879a 100644
> --- a/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi
> @@ -173,7 +173,7 @@ &adsp_mem {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -306,7 +306,7 @@ vreg_l18a_0p8: ldo18 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>