2022-09-25 21:28:29

by Luca Weiss

[permalink] [raw]
Subject: [PATCH 1/3] ARM: dts: qcom: pm8226: fix regulators node name

Adjust the node name to match bindings and fix the validation warning.

Signed-off-by: Luca Weiss <[email protected]>
---
arch/arm/boot/dts/qcom-pm8226.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom-pm8226.dtsi b/arch/arm/boot/dts/qcom-pm8226.dtsi
index 9b7d9d04ded6..be8c8032a3b1 100644
--- a/arch/arm/boot/dts/qcom-pm8226.dtsi
+++ b/arch/arm/boot/dts/qcom-pm8226.dtsi
@@ -98,7 +98,7 @@ pm8226_1: pm8226@1 {
#address-cells = <1>;
#size-cells = <0>;

- pm8226_spmi_regulators: pm8226-regulators {
+ pm8226_spmi_regulators: regulators {
compatible = "qcom,pm8226-regulators";
};

--
2.37.3


2022-09-26 09:16:37

by Neil Armstrong

[permalink] [raw]
Subject: Re: [PATCH 1/3] ARM: dts: qcom: pm8226: fix regulators node name

On 25/09/2022 23:02, Luca Weiss wrote:
> Adjust the node name to match bindings and fix the validation warning.
>
> Signed-off-by: Luca Weiss <[email protected]>
> ---
> arch/arm/boot/dts/qcom-pm8226.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-pm8226.dtsi b/arch/arm/boot/dts/qcom-pm8226.dtsi
> index 9b7d9d04ded6..be8c8032a3b1 100644
> --- a/arch/arm/boot/dts/qcom-pm8226.dtsi
> +++ b/arch/arm/boot/dts/qcom-pm8226.dtsi
> @@ -98,7 +98,7 @@ pm8226_1: pm8226@1 {
> #address-cells = <1>;
> #size-cells = <0>;
>
> - pm8226_spmi_regulators: pm8226-regulators {
> + pm8226_spmi_regulators: regulators {
> compatible = "qcom,pm8226-regulators";
> };
>

Reviewed-by: Neil Armstrong <[email protected]>

2022-09-26 09:18:46

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/3] ARM: dts: qcom: pm8226: fix regulators node name

On 25/09/2022 23:02, Luca Weiss wrote:
> Adjust the node name to match bindings and fix the validation warning.
>
> Signed-off-by: Luca Weiss <[email protected]>
> ---
> arch/arm/boot/dts/qcom-pm8226.dtsi | 2 +-

Thanks, I missed it in my last cleanups.


Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof

2022-09-26 09:26:10

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/3] ARM: dts: qcom: pm8226: fix regulators node name

On 26/09/2022 11:12, Krzysztof Kozlowski wrote:
> On 26/09/2022 11:05, Krzysztof Kozlowski wrote:
>> On 25/09/2022 23:02, Luca Weiss wrote:
>>> Adjust the node name to match bindings and fix the validation warning.
>>>
>>> Signed-off-by: Luca Weiss <[email protected]>
>>> ---
>>> arch/arm/boot/dts/qcom-pm8226.dtsi | 2 +-
>>
>> Thanks, I missed it in my last cleanups.
>
> No, this should be rather squashed with my cleanups:
> https://lore.kernel.org/all/[email protected]/

No, to a no, my patch was changing RPM regulators... Argh, damn that
naming...

Best regards,
Krzysztof

2022-09-26 09:41:58

by Neil Armstrong

[permalink] [raw]
Subject: Re: [PATCH 1/3] ARM: dts: qcom: pm8226: fix regulators node name

On 25/09/2022 23:02, Luca Weiss wrote:
> Adjust the node name to match bindings and fix the validation warning.
>
> Signed-off-by: Luca Weiss <[email protected]>
> ---
> arch/arm/boot/dts/qcom-pm8226.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-pm8226.dtsi b/arch/arm/boot/dts/qcom-pm8226.dtsi
> index 9b7d9d04ded6..be8c8032a3b1 100644
> --- a/arch/arm/boot/dts/qcom-pm8226.dtsi
> +++ b/arch/arm/boot/dts/qcom-pm8226.dtsi
> @@ -98,7 +98,7 @@ pm8226_1: pm8226@1 {
> #address-cells = <1>;
> #size-cells = <0>;
>
> - pm8226_spmi_regulators: pm8226-regulators {
> + pm8226_spmi_regulators: regulators {
> compatible = "qcom,pm8226-regulators";
> };
>

Reviewed-by: Neil Armstrong <[email protected]>

2022-09-26 10:04:02

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/3] ARM: dts: qcom: pm8226: fix regulators node name

On 26/09/2022 11:05, Krzysztof Kozlowski wrote:
> On 25/09/2022 23:02, Luca Weiss wrote:
>> Adjust the node name to match bindings and fix the validation warning.
>>
>> Signed-off-by: Luca Weiss <[email protected]>
>> ---
>> arch/arm/boot/dts/qcom-pm8226.dtsi | 2 +-
>
> Thanks, I missed it in my last cleanups.

No, this should be rather squashed with my cleanups:
https://lore.kernel.org/all/[email protected]/

Best regards,
Krzysztof

2022-09-27 14:17:52

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH 1/3] ARM: dts: qcom: pm8226: fix regulators node name



On 25.09.2022 23:02, Luca Weiss wrote:
> Adjust the node name to match bindings and fix the validation warning.
>
> Signed-off-by: Luca Weiss <[email protected]>
> ---
Reviewed-by: Konrad Dybcio <[email protected]>

Konrad
> arch/arm/boot/dts/qcom-pm8226.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-pm8226.dtsi b/arch/arm/boot/dts/qcom-pm8226.dtsi
> index 9b7d9d04ded6..be8c8032a3b1 100644
> --- a/arch/arm/boot/dts/qcom-pm8226.dtsi
> +++ b/arch/arm/boot/dts/qcom-pm8226.dtsi
> @@ -98,7 +98,7 @@ pm8226_1: pm8226@1 {
> #address-cells = <1>;
> #size-cells = <0>;
>
> - pm8226_spmi_regulators: pm8226-regulators {
> + pm8226_spmi_regulators: regulators {
> compatible = "qcom,pm8226-regulators";
> };
>