2022-09-21 01:06:50

by Konrad Dybcio

Subject: [PATCH] arm64: dts: qcom: msm8998-yoshino: Fix up SMD regulators formatting

Add a new line between each subnode and make the { } consistent.

Signed-off-by: Konrad Dybcio <[email protected]>
---
.../dts/qcom/msm8998-sony-xperia-yoshino.dtsi | 33 ++++++++++++++++++-
1 file changed, 32 insertions(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/msm8998-sony-xperia-yoshino.dtsi b/arch/arm64/boot/dts/qcom/msm8998-sony-xperia-yoshino.dtsi
index d08639082247..862d859af885 100644
--- a/arch/arm64/boot/dts/qcom/msm8998-sony-xperia-yoshino.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8998-sony-xperia-yoshino.dtsi
@@ -410,131 +410,162 @@ vreg_s3a_1p35: s3 {
regulator-min-microvolt = <1352000>;
regulator-max-microvolt = <1352000>;
};
+
vreg_s4a_1p8: s4 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <1800000>;
regulator-system-load = <100000>;
regulator-allow-set-load;
};
+
vreg_s5a_2p04: s5 {
regulator-min-microvolt = <1904000>;
regulator-max-microvolt = <2032000>;
};
+
vreg_s7a_1p025: s7 {
regulator-min-microvolt = <900000>;
regulator-max-microvolt = <1028000>;
};
+
vreg_l1a_0p875: l1 {
regulator-min-microvolt = <880000>;
regulator-max-microvolt = <880000>;
regulator-system-load = <73400>;
regulator-allow-set-load;
};
+
vreg_l2a_1p2: l2 {
regulator-min-microvolt = <1200000>;
regulator-max-microvolt = <1200000>;
regulator-system-load = <12560>;
regulator-allow-set-load;
};
+
vreg_l3a_1p0: l3 {
regulator-min-microvolt = <1000000>;
regulator-max-microvolt = <1000000>;
};
+
vreg_l5a_0p8: l5 {
regulator-min-microvolt = <800000>;
regulator-max-microvolt = <800000>;
};
+
vreg_l6a_1p8: l6 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <1800000>;
};
+
vreg_l7a_1p8: l7 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <1800000>;
};
+
vreg_l8a_1p2: l8 {
regulator-min-microvolt = <1200000>;
regulator-max-microvolt = <1200000>;
};
+
vreg_l9a_1p8: l9 {
regulator-min-microvolt = <1808000>;
regulator-max-microvolt = <2960000>;
};
+
vreg_l10a_1p8: l10 {
regulator-min-microvolt = <1808000>;
regulator-max-microvolt = <2960000>;
};
+
vreg_l11a_1p0: l11 {
regulator-min-microvolt = <1000000>;
regulator-max-microvolt = <1000000>;
};
+
vreg_l12a_1p8: l12 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <1800000>;
};
+
vreg_l13a_2p95: l13 {
regulator-min-microvolt = <1808000>;
regulator-max-microvolt = <2960000>;
regulator-allow-set-load;
};
+
vreg_l14a_1p85: l14 {
regulator-min-microvolt = <1848000>;
regulator-max-microvolt = <1856000>;
regulator-system-load = <32000>;
regulator-allow-set-load;
};
+
vreg_l15a_1p8: l15 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <1800000>;
};
+
vreg_l16a_2p7: l16 {
regulator-min-microvolt = <2704000>;
regulator-max-microvolt = <2704000>;
};
+
vreg_l17a_1p3: l17 {
regulator-min-microvolt = <1304000>;
regulator-max-microvolt = <1304000>;
};
- vreg_l18a_2p85: l18 {};
+
+ vreg_l18a_2p85: l18 { };
+
vreg_l19a_2p7: l19 {
regulator-min-microvolt = <2696000>;
regulator-max-microvolt = <2704000>;
};
+
vreg_l20a_2p95: l20 {
regulator-min-microvolt = <2960000>;
regulator-max-microvolt = <2960000>;
regulator-system-load = <10000>;
regulator-allow-set-load;
};
+
vreg_l21a_2p95: l21 {
regulator-min-microvolt = <2960000>;
regulator-max-microvolt = <2960000>;
regulator-system-load = <800000>;
regulator-allow-set-load;
};
+
vreg_l22a_2p85: l22 { };
+
vreg_l23a_3p3: l23 {
regulator-min-microvolt = <3312000>;
regulator-max-microvolt = <3312000>;
};
+
vreg_l24a_3p075: l24 {
regulator-min-microvolt = <3088000>;
regulator-max-microvolt = <3088000>;
};
+
vreg_l25a_3p3: l25 {
regulator-min-microvolt = <3104000>;
regulator-max-microvolt = <3312000>;
};
+
vreg_l26a_1p2: l26 {
regulator-min-microvolt = <1200000>;
regulator-max-microvolt = <1200000>;
regulator-allow-set-load;
};
+
vreg_l28_3p0: l28 {
regulator-min-microvolt = <3000000>;
regulator-max-microvolt = <3000000>;
};
+
vreg_lvs1a_1p8: lvs1 { };
+
vreg_lvs2a_1p8: lvs2 { };
};

--
2.37.3

2022-09-21 07:41:15

by Neil Armstrong

Subject: Re: [PATCH] arm64: dts: qcom: msm8998-yoshino: Fix up SMD regulators formatting

On 21/09/2022 02:47, Konrad Dybcio wrote:
> Add a new line between each subnode and make the { } consistent.
>
> Signed-off-by: Konrad Dybcio <[email protected]>
> ---
> .../dts/qcom/msm8998-sony-xperia-yoshino.dtsi | 33 ++++++++++++++++++-
> 1 file changed, 32 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/msm8998-sony-xperia-yoshino.dtsi b/arch/arm64/boot/dts/qcom/msm8998-sony-xperia-yoshino.dtsi
> index d08639082247..862d859af885 100644
> --- a/arch/arm64/boot/dts/qcom/msm8998-sony-xperia-yoshino.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8998-sony-xperia-yoshino.dtsi
> @@ -410,131 +410,162 @@ vreg_s3a_1p35: s3 {
> regulator-min-microvolt = <1352000>;
> regulator-max-microvolt = <1352000>;
> };
> +
> vreg_s4a_1p8: s4 {
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> regulator-system-load = <100000>;
> regulator-allow-set-load;
> };
> +
> vreg_s5a_2p04: s5 {
> regulator-min-microvolt = <1904000>;
> regulator-max-microvolt = <2032000>;
> };
> +
> vreg_s7a_1p025: s7 {
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <1028000>;
> };
> +
> vreg_l1a_0p875: l1 {
> regulator-min-microvolt = <880000>;
> regulator-max-microvolt = <880000>;
> regulator-system-load = <73400>;
> regulator-allow-set-load;
> };
> +
> vreg_l2a_1p2: l2 {
> regulator-min-microvolt = <1200000>;
> regulator-max-microvolt = <1200000>;
> regulator-system-load = <12560>;
> regulator-allow-set-load;
> };
> +
> vreg_l3a_1p0: l3 {
> regulator-min-microvolt = <1000000>;
> regulator-max-microvolt = <1000000>;
> };
> +
> vreg_l5a_0p8: l5 {
> regulator-min-microvolt = <800000>;
> regulator-max-microvolt = <800000>;
> };
> +
> vreg_l6a_1p8: l6 {
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> };
> +
> vreg_l7a_1p8: l7 {
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> };
> +
> vreg_l8a_1p2: l8 {
> regulator-min-microvolt = <1200000>;
> regulator-max-microvolt = <1200000>;
> };
> +
> vreg_l9a_1p8: l9 {
> regulator-min-microvolt = <1808000>;
> regulator-max-microvolt = <2960000>;
> };
> +
> vreg_l10a_1p8: l10 {
> regulator-min-microvolt = <1808000>;
> regulator-max-microvolt = <2960000>;
> };
> +
> vreg_l11a_1p0: l11 {
> regulator-min-microvolt = <1000000>;
> regulator-max-microvolt = <1000000>;
> };
> +
> vreg_l12a_1p8: l12 {
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> };
> +
> vreg_l13a_2p95: l13 {
> regulator-min-microvolt = <1808000>;
> regulator-max-microvolt = <2960000>;
> regulator-allow-set-load;
> };
> +
> vreg_l14a_1p85: l14 {
> regulator-min-microvolt = <1848000>;
> regulator-max-microvolt = <1856000>;
> regulator-system-load = <32000>;
> regulator-allow-set-load;
> };
> +
> vreg_l15a_1p8: l15 {
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> };
> +
> vreg_l16a_2p7: l16 {
> regulator-min-microvolt = <2704000>;
> regulator-max-microvolt = <2704000>;
> };
> +
> vreg_l17a_1p3: l17 {
> regulator-min-microvolt = <1304000>;
> regulator-max-microvolt = <1304000>;
> };
> - vreg_l18a_2p85: l18 {};
> +
> + vreg_l18a_2p85: l18 { };
> +
> vreg_l19a_2p7: l19 {
> regulator-min-microvolt = <2696000>;
> regulator-max-microvolt = <2704000>;
> };
> +
> vreg_l20a_2p95: l20 {
> regulator-min-microvolt = <2960000>;
> regulator-max-microvolt = <2960000>;
> regulator-system-load = <10000>;
> regulator-allow-set-load;
> };
> +
> vreg_l21a_2p95: l21 {
> regulator-min-microvolt = <2960000>;
> regulator-max-microvolt = <2960000>;
> regulator-system-load = <800000>;
> regulator-allow-set-load;
> };
> +
> vreg_l22a_2p85: l22 { };
> +
> vreg_l23a_3p3: l23 {
> regulator-min-microvolt = <3312000>;
> regulator-max-microvolt = <3312000>;
> };
> +
> vreg_l24a_3p075: l24 {
> regulator-min-microvolt = <3088000>;
> regulator-max-microvolt = <3088000>;
> };
> +
> vreg_l25a_3p3: l25 {
> regulator-min-microvolt = <3104000>;
> regulator-max-microvolt = <3312000>;
> };
> +
> vreg_l26a_1p2: l26 {
> regulator-min-microvolt = <1200000>;
> regulator-max-microvolt = <1200000>;
> regulator-allow-set-load;
> };
> +
> vreg_l28_3p0: l28 {
> regulator-min-microvolt = <3000000>;
> regulator-max-microvolt = <3000000>;
> };
> +
> vreg_lvs1a_1p8: lvs1 { };
> +
> vreg_lvs2a_1p8: lvs2 { };
> };
>

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

2022-09-21 09:55:03

by Krzysztof Kozlowski

Subject: Re: [PATCH] arm64: dts: qcom: msm8998-yoshino: Fix up SMD regulators formatting

On 21/09/2022 02:47, Konrad Dybcio wrote:
> Add a new line between each subnode and make the { } consistent.
>
> Signed-off-by: Konrad Dybcio <[email protected]>
> ---
> .../dts/qcom/msm8998-sony-xperia-yoshino.dtsi | 33 ++++++++++++++++++-


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

Best regards,
Krzysztof

2022-10-18 03:10:18

by Bjorn Andersson

Subject: Re: [PATCH] arm64: dts: qcom: msm8998-yoshino: Fix up SMD regulators formatting

On Wed, 21 Sep 2022 02:47:41 +0200, Konrad Dybcio wrote:
> Add a new line between each subnode and make the { } consistent.
>
>

Applied, thanks!

[1/1] arm64: dts: qcom: msm8998-yoshino: Fix up SMD regulators formatting
commit: 732479bda06e1dfe5f46bfc682d94f40dff0af1f

Best regards,
--
Bjorn Andersson <[email protected]>