2022-09-25 21:28:16

by Luca Weiss

[permalink] [raw]
Subject: [PATCH 3/3] arm64: dts: qcom: pm8998: adjust coincell node name to bindings

The pm8941-coincell node is supposed to be called 'charger'.

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

diff --git a/arch/arm64/boot/dts/qcom/pm8998.dtsi b/arch/arm64/boot/dts/qcom/pm8998.dtsi
index d09f2954b6f9..52c1b34e7d79 100644
--- a/arch/arm64/boot/dts/qcom/pm8998.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8998.dtsi
@@ -63,7 +63,7 @@ pm8998_temp: temp-alarm@2400 {
#thermal-sensor-cells = <0>;
};

- pm8998_coincell: coincell@2800 {
+ pm8998_coincell: charger@2800 {
compatible = "qcom,pm8941-coincell";
reg = <0x2800>;

--
2.37.3


2022-09-26 09:41:42

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 3/3] arm64: dts: qcom: pm8998: adjust coincell node name to bindings

On 25/09/2022 23:02, Luca Weiss wrote:
> The pm8941-coincell node is supposed to be called 'charger'.
>
> Signed-off-by: Luca Weiss <[email protected]>


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

Best regards,
Krzysztof

2022-09-27 13:49:20

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH 3/3] arm64: dts: qcom: pm8998: adjust coincell node name to bindings



On 25.09.2022 23:02, Luca Weiss wrote:
> The pm8941-coincell node is supposed to be called 'charger'.
>
> Signed-off-by: Luca Weiss <[email protected]>
> ---
Reviewed-by: Konrad Dybcio <[email protected]>

Konrad
> arch/arm64/boot/dts/qcom/pm8998.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/pm8998.dtsi b/arch/arm64/boot/dts/qcom/pm8998.dtsi
> index d09f2954b6f9..52c1b34e7d79 100644
> --- a/arch/arm64/boot/dts/qcom/pm8998.dtsi
> +++ b/arch/arm64/boot/dts/qcom/pm8998.dtsi
> @@ -63,7 +63,7 @@ pm8998_temp: temp-alarm@2400 {
> #thermal-sensor-cells = <0>;
> };
>
> - pm8998_coincell: coincell@2800 {
> + pm8998_coincell: charger@2800 {
> compatible = "qcom,pm8941-coincell";
> reg = <0x2800>;
>