arm64: dts: qcom: sm8550: Fix the aoss_qmp node name

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

Posted by Abel Vesa 2 years, 7 months ago

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 <abel.vesa@linaro.org>
---
 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

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

Posted by Bjorn Andersson 2 years, 7 months ago

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 <andersson@kernel.org>

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

Posted by Sai Prakash Ranjan 2 years, 7 months ago

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 <abel.vesa@linaro.org>
> ---

Reviewed-by: Sai Prakash Ranjan <quic_saipraka@quicinc.com>


>   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>;

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

Posted by Konrad Dybcio 2 years, 7 months ago


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 <abel.vesa@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

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>;