[v3] arm64: dts: qcom: add DP Controller to SM8550 DTS

[PATCH v3 1/2] arm64: dts: qcom: sm8550: fix low_svs RPMhPD labels

Posted by Neil Armstrong 2 years, 8 months ago

"low" was written "lov", fix this.

Fixes: 99d33ee61cb0 ("arm64: dts: qcom: sm8550: Add missing RPMhPD OPP levels")
Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
 arch/arm64/boot/dts/qcom/sm8550.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sm8550.dtsi b/arch/arm64/boot/dts/qcom/sm8550.dtsi
index 75cd374943eb..972df1ef86ee 100644
--- a/arch/arm64/boot/dts/qcom/sm8550.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8550.dtsi
@@ -3649,15 +3649,15 @@ rpmhpd_opp_min_svs: opp-48 {
 						opp-level = <RPMH_REGULATOR_LEVEL_MIN_SVS>;
 					};
 
-					rpmhpd_opp_lov_svs_d2: opp-52 {
+					rpmhpd_opp_low_svs_d2: opp-52 {
 						opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS_D2>;
 					};
 
-					rpmhpd_opp_lov_svs_d1: opp-56 {
+					rpmhpd_opp_low_svs_d1: opp-56 {
 						opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS_D1>;
 					};
 
-					rpmhpd_opp_lov_svs_d0: opp-60 {
+					rpmhpd_opp_low_svs_d0: opp-60 {
 						opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS_D0>;
 					};
 

-- 
2.34.1

Re: [PATCH v3 1/2] arm64: dts: qcom: sm8550: fix low_svs RPMhPD labels

Posted by Konrad Dybcio 2 years, 8 months ago


On 12.06.2023 17:26, Neil Armstrong wrote:
> "low" was written "lov", fix this.
> 
> Fixes: 99d33ee61cb0 ("arm64: dts: qcom: sm8550: Add missing RPMhPD OPP levels")
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
https://lore.kernel.org/linux-arm-msm/1d3c05f5-c1bd-6844-1788-8df0b863a02e@linaro.org/

somebody forgot to run `b4 trailers -u` :P

Konrad
>  arch/arm64/boot/dts/qcom/sm8550.dtsi | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sm8550.dtsi b/arch/arm64/boot/dts/qcom/sm8550.dtsi
> index 75cd374943eb..972df1ef86ee 100644
> --- a/arch/arm64/boot/dts/qcom/sm8550.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8550.dtsi
> @@ -3649,15 +3649,15 @@ rpmhpd_opp_min_svs: opp-48 {
>  						opp-level = <RPMH_REGULATOR_LEVEL_MIN_SVS>;
>  					};
>  
> -					rpmhpd_opp_lov_svs_d2: opp-52 {
> +					rpmhpd_opp_low_svs_d2: opp-52 {
>  						opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS_D2>;
>  					};
>  
> -					rpmhpd_opp_lov_svs_d1: opp-56 {
> +					rpmhpd_opp_low_svs_d1: opp-56 {
>  						opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS_D1>;
>  					};
>  
> -					rpmhpd_opp_lov_svs_d0: opp-60 {
> +					rpmhpd_opp_low_svs_d0: opp-60 {
>  						opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS_D0>;
>  					};
>  
>

Re: [PATCH v3 1/2] arm64: dts: qcom: sm8550: fix low_svs RPMhPD labels

Posted by Neil Armstrong 2 years, 8 months ago

On 12/06/2023 18:50, Konrad Dybcio wrote:
> 
> 
> On 12.06.2023 17:26, Neil Armstrong wrote:
>> "low" was written "lov", fix this.
>>
>> Fixes: 99d33ee61cb0 ("arm64: dts: qcom: sm8550: Add missing RPMhPD OPP levels")
>> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
>> ---
> https://lore.kernel.org/linux-arm-msm/1d3c05f5-c1bd-6844-1788-8df0b863a02e@linaro.org/
> 
> somebody forgot to run `b4 trailers -u` :P

Damn, resending a v4 now :-)

Thanks,
Neil

> 
> Konrad
>>   arch/arm64/boot/dts/qcom/sm8550.dtsi | 6 +++---
>>   1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sm8550.dtsi b/arch/arm64/boot/dts/qcom/sm8550.dtsi
>> index 75cd374943eb..972df1ef86ee 100644
>> --- a/arch/arm64/boot/dts/qcom/sm8550.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sm8550.dtsi
>> @@ -3649,15 +3649,15 @@ rpmhpd_opp_min_svs: opp-48 {
>>   						opp-level = <RPMH_REGULATOR_LEVEL_MIN_SVS>;
>>   					};
>>   
>> -					rpmhpd_opp_lov_svs_d2: opp-52 {
>> +					rpmhpd_opp_low_svs_d2: opp-52 {
>>   						opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS_D2>;
>>   					};
>>   
>> -					rpmhpd_opp_lov_svs_d1: opp-56 {
>> +					rpmhpd_opp_low_svs_d1: opp-56 {
>>   						opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS_D1>;
>>   					};
>>   
>> -					rpmhpd_opp_lov_svs_d0: opp-60 {
>> +					rpmhpd_opp_low_svs_d0: opp-60 {
>>   						opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS_D0>;
>>   					};
>>   
>>