diff mbox series

[5/7] arm64: dts: qcom: sm8450-hdk: remove pmr735b PMIC inclusion

Message ID 20230630061315.4027453-6-dmitry.baryshkov@linaro.org (mailing list archive)
State Superseded
Headers show
Series arm64: dts: qcom: sm8450-hdk: improve thermal monitoring | expand

Commit Message

Dmitry Baryshkov June 30, 2023, 6:13 a.m. UTC
The 8450-HDK doesn't use PMR735B PMIC. Drop its inclusion to remove the
warning during the HDK bootup.

Fixes: 30464456a1ea ("arm64: dts: qcom: sm8450-hdk: add pmic files")
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm64/boot/dts/qcom/sm8450-hdk.dts | 1 -
 1 file changed, 1 deletion(-)

Comments

Konrad Dybcio June 30, 2023, 8:13 a.m. UTC | #1
On 30.06.2023 08:13, Dmitry Baryshkov wrote:
> The 8450-HDK doesn't use PMR735B PMIC. Drop its inclusion to remove the
> warning during the HDK bootup.
> 
> Fixes: 30464456a1ea ("arm64: dts: qcom: sm8450-hdk: add pmic files")
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
That's.. strange.. Did you confirm this with schematics, or even better,
socinfo?

Konrad
>  arch/arm64/boot/dts/qcom/sm8450-hdk.dts | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
> index bc4c125d1832..dabb7e872f38 100644
> --- a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
> @@ -14,7 +14,6 @@
>  #include "pm8450.dtsi"
>  #include "pmk8350.dtsi"
>  #include "pmr735a.dtsi"
> -#include "pmr735b.dtsi"
>  
>  / {
>  	model = "Qualcomm Technologies, Inc. SM8450 HDK";
Neil Armstrong June 30, 2023, 8:25 a.m. UTC | #2
On 30/06/2023 10:13, Konrad Dybcio wrote:
> On 30.06.2023 08:13, Dmitry Baryshkov wrote:
>> The 8450-HDK doesn't use PMR735B PMIC. Drop its inclusion to remove the
>> warning during the HDK bootup.
>>
>> Fixes: 30464456a1ea ("arm64: dts: qcom: sm8450-hdk: add pmic files")
>> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>> ---
> That's.. strange.. Did you confirm this with schematics, or even better,
> socinfo?

I confirm it doesn't have a 735b from the Schematics

Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>

> 
> Konrad
>>   arch/arm64/boot/dts/qcom/sm8450-hdk.dts | 1 -
>>   1 file changed, 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
>> index bc4c125d1832..dabb7e872f38 100644
>> --- a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
>> +++ b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
>> @@ -14,7 +14,6 @@
>>   #include "pm8450.dtsi"
>>   #include "pmk8350.dtsi"
>>   #include "pmr735a.dtsi"
>> -#include "pmr735b.dtsi"
>>   
>>   / {
>>   	model = "Qualcomm Technologies, Inc. SM8450 HDK";
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
index bc4c125d1832..dabb7e872f38 100644
--- a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
+++ b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
@@ -14,7 +14,6 @@ 
 #include "pm8450.dtsi"
 #include "pmk8350.dtsi"
 #include "pmr735a.dtsi"
-#include "pmr735b.dtsi"
 
 / {
 	model = "Qualcomm Technologies, Inc. SM8450 HDK";