2023-01-27 13:15:59

by Abel Vesa

Subject: [PATCH] arm64: dts: qcom: sm8550: Fix the aoss_qmp node name

The proper name for it is power-management. Currently, with the node
name being power-controller, the bindings check fails due to the
property #power-domain-cells missing.

Fixes: ffc50b2d3828 ("arm64: dts: qcom: Add base SM8550 dtsi")
Signed-off-by: Abel Vesa <[email protected]>
---
arch/arm64/boot/dts/qcom/sm8550.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sm8550.dtsi b/arch/arm64/boot/dts/qcom/sm8550.dtsi
index 6ff135191ee0..57878ea64ee0 100644
--- a/arch/arm64/boot/dts/qcom/sm8550.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8550.dtsi
@@ -2503,7 +2503,7 @@ tsens2: thermal-sensor@c273000 {
#thermal-sensor-cells = <1>;
};

- aoss_qmp: power-controller@c300000 {
+ aoss_qmp: power-management@c300000 {
compatible = "qcom,sm8550-aoss-qmp", "qcom,aoss-qmp";
reg = <0 0x0c300000 0 0x400>;
interrupt-parent = <&ipcc>;
--
2.34.1

2023-01-27 13:20:49

by Konrad Dybcio

Subject: Re: [PATCH] arm64: dts: qcom: sm8550: Fix the aoss_qmp node name

On 27.01.2023 14:14, Abel Vesa wrote:
> The proper name for it is power-management. Currently, with the node
> name being power-controller, the bindings check fails due to the
> property #power-domain-cells missing.
>
> Fixes: ffc50b2d3828 ("arm64: dts: qcom: Add base SM8550 dtsi")
> Signed-off-by: Abel Vesa <[email protected]>
> ---
Reviewed-by: Konrad Dybcio <[email protected]>

Konrad

> arch/arm64/boot/dts/qcom/sm8550.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm8550.dtsi b/arch/arm64/boot/dts/qcom/sm8550.dtsi
> index 6ff135191ee0..57878ea64ee0 100644
> --- a/arch/arm64/boot/dts/qcom/sm8550.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8550.dtsi
> @@ -2503,7 +2503,7 @@ tsens2: thermal-sensor@c273000 {
> #thermal-sensor-cells = <1>;
> };
>
> - aoss_qmp: power-controller@c300000 {
> + aoss_qmp: power-management@c300000 {
> compatible = "qcom,sm8550-aoss-qmp", "qcom,aoss-qmp";
> reg = <0 0x0c300000 0 0x400>;
> interrupt-parent = <&ipcc>;

2023-01-27 16:15:08

by Sai Prakash Ranjan

Subject: Re: [PATCH] arm64: dts: qcom: sm8550: Fix the aoss_qmp node name

On 1/27/2023 6:44 PM, Abel Vesa wrote:
> The proper name for it is power-management. Currently, with the node
> name being power-controller, the bindings check fails due to the
> property #power-domain-cells missing.
>
> Fixes: ffc50b2d3828 ("arm64: dts: qcom: Add base SM8550 dtsi")
> Signed-off-by: Abel Vesa <[email protected]>
> ---

Reviewed-by: Sai Prakash Ranjan <[email protected]>


> arch/arm64/boot/dts/qcom/sm8550.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm8550.dtsi b/arch/arm64/boot/dts/qcom/sm8550.dtsi
> index 6ff135191ee0..57878ea64ee0 100644
> --- a/arch/arm64/boot/dts/qcom/sm8550.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8550.dtsi
> @@ -2503,7 +2503,7 @@ tsens2: thermal-sensor@c273000 {
> #thermal-sensor-cells = <1>;
> };
>
> - aoss_qmp: power-controller@c300000 {
> + aoss_qmp: power-management@c300000 {
> compatible = "qcom,sm8550-aoss-qmp", "qcom,aoss-qmp";
> reg = <0 0x0c300000 0 0x400>;
> interrupt-parent = <&ipcc>;

2023-02-09 04:27:52

by Bjorn Andersson

Subject: Re: [PATCH] arm64: dts: qcom: sm8550: Fix the aoss_qmp node name

On Fri, 27 Jan 2023 15:14:41 +0200, Abel Vesa wrote:
> The proper name for it is power-management. Currently, with the node
> name being power-controller, the bindings check fails due to the
> property #power-domain-cells missing.
>
>

Applied, thanks!

[1/1] arm64: dts: qcom: sm8550: Fix the aoss_qmp node name
commit: 3a63e478b3c01b7c0eddba1121200b089bf1df12

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