2024-05-30 08:32:14

by Dmitry Baryshkov

[permalink] [raw]
Subject: [PATCH] arm64: dts: qcom: sm8650-hdk: remove redundant properties

The coomits 65931e59e039 ("arm64: dts: qcom: sm8650: move USB graph to
the SoC dtsi") and fbb22a182267 ("arm64: dts: qcom: sm8650: move PHY's
orientation-switch to SoC dtsi") have moved some of the properties from
the board DT files to the sm8650.dtsi. As the patch for sm8650 HDK
predates those commits, it still had those properties inside.

Drop these duplicate proerties from the sm8650-hdk.dts.

Signed-off-by: Dmitry Baryshkov <[email protected]>
---
arch/arm64/boot/dts/qcom/sm8650-hdk.dts | 15 ---------------
1 file changed, 15 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sm8650-hdk.dts b/arch/arm64/boot/dts/qcom/sm8650-hdk.dts
index 7f2dbada63b5..182864a3b6bd 100644
--- a/arch/arm64/boot/dts/qcom/sm8650-hdk.dts
+++ b/arch/arm64/boot/dts/qcom/sm8650-hdk.dts
@@ -881,7 +881,6 @@ &mdss_dp0 {

&mdss_dp0_out {
data-lanes = <0 1>;
- remote-endpoint = <&usb_dp_qmpphy_dp_in>;
};

&pcie0 {
@@ -1220,10 +1219,6 @@ &usb_1_dwc3_hs {
remote-endpoint = <&pmic_glink_hs_in>;
};

-&usb_1_dwc3_ss {
- remote-endpoint = <&usb_dp_qmpphy_usb_ss_in>;
-};
-
&usb_1_hsphy {
vdd-supply = <&vreg_l1i_0p88>;
vdda12-supply = <&vreg_l3i_1p2>;
@@ -1237,23 +1232,13 @@ &usb_dp_qmpphy {
vdda-phy-supply = <&vreg_l3i_1p2>;
vdda-pll-supply = <&vreg_l3g_0p91>;

- orientation-switch;
-
status = "okay";
};

-&usb_dp_qmpphy_dp_in {
- remote-endpoint = <&mdss_dp0_out>;
-};
-
&usb_dp_qmpphy_out {
remote-endpoint = <&pmic_glink_ss_in>;
};

-&usb_dp_qmpphy_usb_ss_in {
- remote-endpoint = <&usb_1_dwc3_ss>;
-};
-
&xo_board {
clock-frequency = <76800000>;
};

---
base-commit: efd5091e445509a76b0cdece7b189121ed6df673
change-id: 20240530-sm8650-hdk-redundant-00489e4e5fc7

Best regards,
--
Dmitry Baryshkov <[email protected]>



2024-05-31 13:47:41

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: sm8650-hdk: remove redundant properties

On 30.05.2024 10:31 AM, Dmitry Baryshkov wrote:
> The coomits 65931e59e039 ("arm64: dts: qcom: sm8650: move USB graph to
> the SoC dtsi") and fbb22a182267 ("arm64: dts: qcom: sm8650: move PHY's
> orientation-switch to SoC dtsi") have moved some of the properties from
> the board DT files to the sm8650.dtsi. As the patch for sm8650 HDK
> predates those commits, it still had those properties inside.
>
> Drop these duplicate proerties from the sm8650-hdk.dts.
>
> Signed-off-by: Dmitry Baryshkov <[email protected]>
> ---

Reviewed-by: Konrad Dybcio <[email protected]>

Konrad

2024-06-02 03:54:31

by Bjorn Andersson

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: sm8650-hdk: remove redundant properties


On Thu, 30 May 2024 11:31:56 +0300, Dmitry Baryshkov wrote:
> The coomits 65931e59e039 ("arm64: dts: qcom: sm8650: move USB graph to
> the SoC dtsi") and fbb22a182267 ("arm64: dts: qcom: sm8650: move PHY's
> orientation-switch to SoC dtsi") have moved some of the properties from
> the board DT files to the sm8650.dtsi. As the patch for sm8650 HDK
> predates those commits, it still had those properties inside.
>
> Drop these duplicate proerties from the sm8650-hdk.dts.
>
> [...]

Applied, thanks!

[1/1] arm64: dts: qcom: sm8650-hdk: remove redundant properties
commit: cf2a08e149b28b4c8d9c63f84348f8d7fff0b5a7

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