| Message ID | 20220925172443.92900-1-luca@z3ntu.xyz |
|---|---|
| State | New |
| 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/arm/boot/dts/qcom-msm8974.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi > index 620900e5ffd5..da247d799492 100644 > --- a/arch/arm/boot/dts/qcom-msm8974.dtsi > +++ b/arch/arm/boot/dts/qcom-msm8974.dtsi > @@ -1571,7 +1571,7 @@ dsi0: dsi@fd922800 { > "core_mmss"; > > phys = <&dsi0_phy>; > - phy-names = "dsi-phy"; > + phy-names = "dsi"; > > status = "disabled"; > 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> > --- > 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> > --- > arch/arm/boot/dts/qcom-msm8974.dtsi | 2 +- Already sent: https://lore.kernel.org/all/20220924090108.166934-3-dmitry.baryshkov@linaro.org/ Best regards, Krzysztof
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
On 25/09/2022 20:24, Luca Weiss wrote: > Use dsi instead of dsi-phy as required by the binding. > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > --- > arch/arm/boot/dts/qcom-msm8974.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi > index 620900e5ffd5..da247d799492 100644 > --- a/arch/arm/boot/dts/qcom-msm8974.dtsi > +++ b/arch/arm/boot/dts/qcom-msm8974.dtsi > @@ -1571,7 +1571,7 @@ dsi0: dsi@fd922800 { > "core_mmss"; > > phys = <&dsi0_phy>; > - phy-names = "dsi-phy"; > + phy-names = "dsi"; I'd prefer to merge [1] instead [1] https://lore.kernel.org/linux-arm-msm/20220907000105.786265-1-bryan.odonoghue@linaro.org/ > > status = "disabled"; >
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/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi index 620900e5ffd5..da247d799492 100644 --- a/arch/arm/boot/dts/qcom-msm8974.dtsi +++ b/arch/arm/boot/dts/qcom-msm8974.dtsi @@ -1571,7 +1571,7 @@ dsi0: dsi@fd922800 { "core_mmss"; phys = <&dsi0_phy>; - phy-names = "dsi-phy"; + phy-names = "dsi"; status = "disabled";
Use dsi instead of dsi-phy as required by the binding. Signed-off-by: Luca Weiss <luca@z3ntu.xyz> --- arch/arm/boot/dts/qcom-msm8974.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)