Message ID | 20220925172443.92900-2-luca@z3ntu.xyz (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | [1/2] ARM: dts: qcom: msm8974: Align dsi phy-names with schema | expand |
On 25/09/2022 19:24, Luca Weiss wrote: > Use dsi instead of dsi-phy as required by the binding. > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > --- > arch/arm64/boot/dts/qcom/msm8916.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi > index a831064700ee..48f4b4deb8b1 100644 > --- a/arch/arm64/boot/dts/qcom/msm8916.dtsi > +++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi > @@ -1046,7 +1046,7 @@ dsi0: dsi@1a98000 { > "pixel", > "core"; > phys = <&dsi_phy0>; > - phy-names = "dsi-phy"; > + phy-names = "dsi"; > > #address-cells = <1>; > #size-cells = <0>; Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
On 25/09/2022 19:24, Luca Weiss wrote: > Use dsi instead of dsi-phy as required by the binding. > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > --- Already sent: https://lore.kernel.org/all/20220924090108.166934-4-dmitry.baryshkov@linaro.org/ Best regards, Krzysztof
Hi Krzysztof, On Montag, 26. September 2022 10:53:03 CEST Krzysztof Kozlowski wrote: > On 25/09/2022 19:24, Luca Weiss wrote: > > Use dsi instead of dsi-phy as required by the binding. > > > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > > --- > > Already sent: > > https://lore.kernel.org/all/20220924090108.166934-4-dmitry.baryshkov@linaro. > org/ This patch is phy-names property while the other patch is for node name, unless I'm missing something. But as Dmitry pointed out, there's a patchset to completely remove the property which is also totally fine by me. Regards Luca > > Best regards, > Krzysztof
On 26/09/2022 17:02, Luca Weiss wrote: > Hi Krzysztof, > > On Montag, 26. September 2022 10:53:03 CEST Krzysztof Kozlowski wrote: >> On 25/09/2022 19:24, Luca Weiss wrote: >>> Use dsi instead of dsi-phy as required by the binding. >>> >>> Signed-off-by: Luca Weiss <luca@z3ntu.xyz> >>> --- >> >> Already sent: >> >> https://lore.kernel.org/all/20220924090108.166934-4-dmitry.baryshkov@linaro. >> org/ > > This patch is phy-names property while the other patch is for node name, > unless I'm missing something. Ah, right. -ETOOMANYPHYs... > > But as Dmitry pointed out, there's a patchset to completely remove the > property which is also totally fine by me. Best regards, Krzysztof
diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi index a831064700ee..48f4b4deb8b1 100644 --- a/arch/arm64/boot/dts/qcom/msm8916.dtsi +++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi @@ -1046,7 +1046,7 @@ dsi0: dsi@1a98000 { "pixel", "core"; phys = <&dsi_phy0>; - phy-names = "dsi-phy"; + phy-names = "dsi"; #address-cells = <1>; #size-cells = <0>;
Use dsi instead of dsi-phy as required by the binding. Signed-off-by: Luca Weiss <luca@z3ntu.xyz> --- arch/arm64/boot/dts/qcom/msm8916.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)