On 13 December 2022 23:44:04 EET, Kuogee Hsieh <quic_khsieh@xxxxxxxxxxx> wrote: >Move data-lanes property from mdss_dp node to dp_out endpoint. Also >add link-frequencies property into dp_out endpoint as well. The last >frequency specified at link-frequencies will be the max link rate >supported by DP. > >Changes in v5: >-- revert changes at sc7180.dtsi and sc7280.dtsi >-- add &dp_out to sc7180-trogdor.dtsi and sc7280-herobrine.dtsi > >Changes in v6: >-- add data-lanes and link-frequencies to yaml > >Changes in v7: >-- change 160000000 to 1620000000 >-- separate yaml to different patch > >Changes in v8: >-- correct Bjorn mail address to kernel.org > >Changes in v9: >-- use symbol rate (hz) for link-frequencies at dp_out at sc7180_trogdor.dtsi > >Signed-off-by: Kuogee Hsieh <quic_khsieh@xxxxxxxxxxx> >--- > arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 6 +++++- > arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 6 +++++- > 2 files changed, 10 insertions(+), 2 deletions(-) > >diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi >index eae22e6..93b0cde 100644 >--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi >+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi >@@ -814,7 +814,11 @@ hp_i2c: &i2c9 { > status = "okay"; > pinctrl-names = "default"; > pinctrl-0 = <&dp_hot_plug_det>; >- data-lanes = <0 1>; >+}; >+ >+&dp_out { >+ data-lanes = <0 1>; Quoting Krzysztof from v12: Why adding two spaces? Just cut previous line and paste it, don't change it. >+ link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000>; > }; > > &pm6150_adc { >diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi >index c11e371..3c7a9d8 100644 >--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi >+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi >@@ -442,7 +442,11 @@ ap_i2c_tpm: &i2c14 { > status = "okay"; > pinctrl-names = "default"; > pinctrl-0 = <&dp_hot_plug_det>; >- data-lanes = <0 1>; >+}; >+ >+&dp_out { >+ data-lanes = <0 1>; >+ link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000 8100000000>; > }; > > &mdss_mdp { -- With best wishes Dmitry