| Message ID | 20211229191838.27922-1-laurent.pinchart+renesas@ideasonboard.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | [1/2] arm64: dts: renesas: Remove empty lvds endpoints | expand |
Hi Laurent, On Wed, Dec 29, 2021 at 8:18 PM Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> wrote: > Endpoints node must have a remote-endpoint property, as endpoints only > exist to model a link between ports. Drop the empty lvds endpoints from > SoC dtsi files, then should be instead declared in the board dts or in > overlays. > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Thanks for your patch! I'm interested in hearing Rob's opinion about the general idea... > --- > arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a774b1.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a774c0.dtsi | 4 ---- > arch/arm64/boot/dts/renesas/r8a774e1.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77951.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77960.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77965.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77970.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77980.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77990.dtsi | 4 ---- > arch/arm64/boot/dts/renesas/r8a77995.dtsi | 4 ---- > 11 files changed, 28 deletions(-) > > diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > index 6f4fffacfca2..9a8c1bd923af 100644 > --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > @@ -2771,8 +2771,6 @@ lvds0_in: endpoint { > }; > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; As there are several DTS files that refer to this label, this change will break them? > base-commit: 2b534e90a1e31c7fc9536b512b72274cc3575f4c Why does this depend on a commit in drm-next? DTS changes are supposed to go in through renesas-devel and soc. > prerequisite-patch-id: 98434eb22c020924bc338316e54a7b42ea339e9f > prerequisite-patch-id: ce261c9865255051f2c2107a930b0f15ce862703 > prerequisite-patch-id: 500ab746415c5421e41128c2b0db35987d8d7a49 > prerequisite-patch-id: ad2d5ad025f9cfa0dc453d7e0b2aa96b4514afd7 > prerequisite-patch-id: ac76bcc0f7b278863e362db083a7d64bbd0bd030 > prerequisite-patch-id: c7a7c3a6b9dbbae18a768160db70d7010ea71ca5 While Google does not index by commit ID or patch ID, at least the base-commit can be found easily in linux-next. But even with some scripting, the prerequisites cannot. What are these prerequisites? Do you have lore links? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Hi Geert, On Mon, Jan 10, 2022 at 11:10:59AM +0100, Geert Uytterhoeven wrote: > On Wed, Dec 29, 2021 at 8:18 PM Laurent Pinchart wrote: > > Endpoints node must have a remote-endpoint property, as endpoints only > > exist to model a link between ports. Drop the empty lvds endpoints from > > SoC dtsi files, then should be instead declared in the board dts or in > > overlays. > > > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > > Thanks for your patch! > I'm interested in hearing Rob's opinion about the general idea... Me too :-) > > --- > > arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a774b1.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a774c0.dtsi | 4 ---- > > arch/arm64/boot/dts/renesas/r8a774e1.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77951.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77960.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77965.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77970.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77980.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77990.dtsi | 4 ---- > > arch/arm64/boot/dts/renesas/r8a77995.dtsi | 4 ---- > > 11 files changed, 28 deletions(-) > > > > diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > index 6f4fffacfca2..9a8c1bd923af 100644 > > --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > @@ -2771,8 +2771,6 @@ lvds0_in: endpoint { > > }; > > port@1 { > > reg = <1>; > > - lvds0_out: endpoint { > > - }; > > As there are several DTS files that refer to this label, this change > will break them? While files are you referring to ? > > base-commit: 2b534e90a1e31c7fc9536b512b72274cc3575f4c > > Why does this depend on a commit in drm-next? > DTS changes are supposed to go in through renesas-devel and soc. Because it happens to have been developed there, that's all. There's no specific reason. > > prerequisite-patch-id: 98434eb22c020924bc338316e54a7b42ea339e9f > > prerequisite-patch-id: ce261c9865255051f2c2107a930b0f15ce862703 > > prerequisite-patch-id: 500ab746415c5421e41128c2b0db35987d8d7a49 > > prerequisite-patch-id: ad2d5ad025f9cfa0dc453d7e0b2aa96b4514afd7 > > prerequisite-patch-id: ac76bcc0f7b278863e362db083a7d64bbd0bd030 > > prerequisite-patch-id: c7a7c3a6b9dbbae18a768160db70d7010ea71ca5 > > While Google does not index by commit ID or patch ID, at least the > base-commit can be found easily in linux-next. But even with some > scripting, the prerequisites cannot. What are these prerequisites? > Do you have lore links? Probably something completely unrelated that I had in my tree when generating the series. You can likely ignore that, sorry for the noise.
Hi Laurent, On Mon, Jan 10, 2022 at 12:24 PM Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > On Mon, Jan 10, 2022 at 11:10:59AM +0100, Geert Uytterhoeven wrote: > > On Wed, Dec 29, 2021 at 8:18 PM Laurent Pinchart wrote: > > > Endpoints node must have a remote-endpoint property, as endpoints only > > > exist to model a link between ports. Drop the empty lvds endpoints from > > > SoC dtsi files, then should be instead declared in the board dts or in > > > overlays. > > > > > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > > > > Thanks for your patch! > > I'm interested in hearing Rob's opinion about the general idea... > > Me too :-) > > > > --- > > > arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 2 -- > > > arch/arm64/boot/dts/renesas/r8a774b1.dtsi | 2 -- > > > arch/arm64/boot/dts/renesas/r8a774c0.dtsi | 4 ---- > > > arch/arm64/boot/dts/renesas/r8a774e1.dtsi | 2 -- > > > arch/arm64/boot/dts/renesas/r8a77951.dtsi | 2 -- > > > arch/arm64/boot/dts/renesas/r8a77960.dtsi | 2 -- > > > arch/arm64/boot/dts/renesas/r8a77965.dtsi | 2 -- > > > arch/arm64/boot/dts/renesas/r8a77970.dtsi | 2 -- > > > arch/arm64/boot/dts/renesas/r8a77980.dtsi | 2 -- > > > arch/arm64/boot/dts/renesas/r8a77990.dtsi | 4 ---- > > > arch/arm64/boot/dts/renesas/r8a77995.dtsi | 4 ---- > > > 11 files changed, 28 deletions(-) > > > > > > diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > > index 6f4fffacfca2..9a8c1bd923af 100644 > > > --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > > +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > > @@ -2771,8 +2771,6 @@ lvds0_in: endpoint { > > > }; > > > port@1 { > > > reg = <1>; > > > - lvds0_out: endpoint { > > > - }; > > > > As there are several DTS files that refer to this label, this change > > will break them? > > While files are you referring to ? The various board files. Seems like I was slightly misled by your fake prerequisites. Upon closer inspection, the board files do not rely on the labels in the SoC .dtsi files, but (re)define their own, matching the former. So everything is fine. Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > base-commit: 2b534e90a1e31c7fc9536b512b72274cc3575f4c > > > > Why does this depend on a commit in drm-next? > > DTS changes are supposed to go in through renesas-devel and soc. > > Because it happens to have been developed there, that's all. There's no > specific reason. > > > > prerequisite-patch-id: 98434eb22c020924bc338316e54a7b42ea339e9f > > > prerequisite-patch-id: ce261c9865255051f2c2107a930b0f15ce862703 > > > prerequisite-patch-id: 500ab746415c5421e41128c2b0db35987d8d7a49 > > > prerequisite-patch-id: ad2d5ad025f9cfa0dc453d7e0b2aa96b4514afd7 > > > prerequisite-patch-id: ac76bcc0f7b278863e362db083a7d64bbd0bd030 > > > prerequisite-patch-id: c7a7c3a6b9dbbae18a768160db70d7010ea71ca5 > > > > While Google does not index by commit ID or patch ID, at least the > > base-commit can be found easily in linux-next. But even with some > > scripting, the prerequisites cannot. What are these prerequisites? > > Do you have lore links? > > Probably something completely unrelated that I had in my tree when > generating the series. You can likely ignore that, sorry for the noise. OK. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
On Mon, Jan 10, 2022 at 11:10:59AM +0100, Geert Uytterhoeven wrote: > On Wed, Dec 29, 2021 at 8:18 PM Laurent Pinchart wrote: > > Endpoints node must have a remote-endpoint property, as endpoints only > > exist to model a link between ports. Drop the empty lvds endpoints from > > SoC dtsi files, then should be instead declared in the board dts or in > > overlays. > > > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > > Thanks for your patch! > I'm interested in hearing Rob's opinion about the general idea... Let's see if Rob could find a few cycles to spare then :-) > > --- > > arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a774b1.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a774c0.dtsi | 4 ---- > > arch/arm64/boot/dts/renesas/r8a774e1.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77951.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77960.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77965.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77970.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77980.dtsi | 2 -- > > arch/arm64/boot/dts/renesas/r8a77990.dtsi | 4 ---- > > arch/arm64/boot/dts/renesas/r8a77995.dtsi | 4 ---- > > 11 files changed, 28 deletions(-) > > > > diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > index 6f4fffacfca2..9a8c1bd923af 100644 > > --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > > @@ -2771,8 +2771,6 @@ lvds0_in: endpoint { > > }; > > port@1 { > > reg = <1>; > > - lvds0_out: endpoint { > > - }; > > As there are several DTS files that refer to this label, this change > will break them? > > > base-commit: 2b534e90a1e31c7fc9536b512b72274cc3575f4c > > Why does this depend on a commit in drm-next? > DTS changes are supposed to go in through renesas-devel and soc. > > > prerequisite-patch-id: 98434eb22c020924bc338316e54a7b42ea339e9f > > prerequisite-patch-id: ce261c9865255051f2c2107a930b0f15ce862703 > > prerequisite-patch-id: 500ab746415c5421e41128c2b0db35987d8d7a49 > > prerequisite-patch-id: ad2d5ad025f9cfa0dc453d7e0b2aa96b4514afd7 > > prerequisite-patch-id: ac76bcc0f7b278863e362db083a7d64bbd0bd030 > > prerequisite-patch-id: c7a7c3a6b9dbbae18a768160db70d7010ea71ca5 > > While Google does not index by commit ID or patch ID, at least the > base-commit can be found easily in linux-next. But even with some > scripting, the prerequisites cannot. What are these prerequisites? > Do you have lore links?
Hi Laurent, Quoting Laurent Pinchart (2021-12-29 19:18:37) > Endpoints node must have a remote-endpoint property, as endpoints only > exist to model a link between ports. Drop the empty lvds endpoints from > SoC dtsi files, then should be instead declared in the board dts or in > overlays. Aha, I was hit by this recently(ish) too, as trying to declare things that are not yet connected as placeholders was ending up generating warnings for me. It's a pain, as having the label placeholder makes it easier to connect to the endpoint later. But this was failing validation otherwise for me, so I presume it's the correct thing to do. Seems we're still hoping for some feedback from Rob on this, but it's fine with me. Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a774b1.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a774c0.dtsi | 4 ---- > arch/arm64/boot/dts/renesas/r8a774e1.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77951.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77960.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77965.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77970.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77980.dtsi | 2 -- > arch/arm64/boot/dts/renesas/r8a77990.dtsi | 4 ---- > arch/arm64/boot/dts/renesas/r8a77995.dtsi | 4 ---- > 11 files changed, 28 deletions(-) > > diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > index 6f4fffacfca2..9a8c1bd923af 100644 > --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi > @@ -2771,8 +2771,6 @@ lvds0_in: endpoint { > }; > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/renesas/r8a774b1.dtsi b/arch/arm64/boot/dts/renesas/r8a774b1.dtsi > index 0f7bdfc90a0d..8912c1665ed9 100644 > --- a/arch/arm64/boot/dts/renesas/r8a774b1.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a774b1.dtsi > @@ -2616,8 +2616,6 @@ lvds0_in: endpoint { > }; > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/renesas/r8a774c0.dtsi b/arch/arm64/boot/dts/renesas/r8a774c0.dtsi > index d597772c4c37..263f39fb8ad9 100644 > --- a/arch/arm64/boot/dts/renesas/r8a774c0.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a774c0.dtsi > @@ -1910,8 +1910,6 @@ lvds0_in: endpoint { > > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > @@ -1937,8 +1935,6 @@ lvds1_in: endpoint { > > port@1 { > reg = <1>; > - lvds1_out: endpoint { > - }; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/renesas/r8a774e1.dtsi b/arch/arm64/boot/dts/renesas/r8a774e1.dtsi > index 379a1300272b..02b35e68a3fa 100644 > --- a/arch/arm64/boot/dts/renesas/r8a774e1.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a774e1.dtsi > @@ -2891,8 +2891,6 @@ lvds0_in: endpoint { > }; > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/renesas/r8a77951.dtsi b/arch/arm64/boot/dts/renesas/r8a77951.dtsi > index 1768a3e6bb8d..0919d8f540c4 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77951.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a77951.dtsi > @@ -3362,8 +3362,6 @@ lvds0_in: endpoint { > }; > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/renesas/r8a77960.dtsi b/arch/arm64/boot/dts/renesas/r8a77960.dtsi > index 2bd8169735d3..221ee1ec415d 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77960.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a77960.dtsi > @@ -2959,8 +2959,6 @@ lvds0_in: endpoint { > }; > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi > index 08df75606430..5f3046eee365 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi > @@ -2771,8 +2771,6 @@ lvds0_in: endpoint { > }; > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/renesas/r8a77970.dtsi b/arch/arm64/boot/dts/renesas/r8a77970.dtsi > index 517892cf6294..ad021b2703ab 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77970.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a77970.dtsi > @@ -1190,8 +1190,6 @@ lvds0_in: endpoint { > }; > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/renesas/r8a77980.dtsi b/arch/arm64/boot/dts/renesas/r8a77980.dtsi > index 6347d15e66b6..29f1161da957 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77980.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a77980.dtsi > @@ -1567,8 +1567,6 @@ lvds0_in: endpoint { > > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/renesas/r8a77990.dtsi b/arch/arm64/boot/dts/renesas/r8a77990.dtsi > index 0ea300a8147d..735c46217835 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77990.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi > @@ -2059,8 +2059,6 @@ lvds0_in: endpoint { > > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > @@ -2086,8 +2084,6 @@ lvds1_in: endpoint { > > port@1 { > reg = <1>; > - lvds1_out: endpoint { > - }; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/renesas/r8a77995.dtsi b/arch/arm64/boot/dts/renesas/r8a77995.dtsi > index 16ad5fc23a67..759505bd1a44 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77995.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a77995.dtsi > @@ -1380,8 +1380,6 @@ lvds0_in: endpoint { > > port@1 { > reg = <1>; > - lvds0_out: endpoint { > - }; > }; > }; > }; > @@ -1407,8 +1405,6 @@ lvds1_in: endpoint { > > port@1 { > reg = <1>; > - lvds1_out: endpoint { > - }; > }; > }; > }; > > base-commit: 2b534e90a1e31c7fc9536b512b72274cc3575f4c > prerequisite-patch-id: 98434eb22c020924bc338316e54a7b42ea339e9f > prerequisite-patch-id: ce261c9865255051f2c2107a930b0f15ce862703 > prerequisite-patch-id: 500ab746415c5421e41128c2b0db35987d8d7a49 > prerequisite-patch-id: ad2d5ad025f9cfa0dc453d7e0b2aa96b4514afd7 > prerequisite-patch-id: ac76bcc0f7b278863e362db083a7d64bbd0bd030 > prerequisite-patch-id: c7a7c3a6b9dbbae18a768160db70d7010ea71ca5 > -- > Regards, > > Laurent Pinchart >
diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi index 6f4fffacfca2..9a8c1bd923af 100644 --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi @@ -2771,8 +2771,6 @@ lvds0_in: endpoint { }; port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a774b1.dtsi b/arch/arm64/boot/dts/renesas/r8a774b1.dtsi index 0f7bdfc90a0d..8912c1665ed9 100644 --- a/arch/arm64/boot/dts/renesas/r8a774b1.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a774b1.dtsi @@ -2616,8 +2616,6 @@ lvds0_in: endpoint { }; port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a774c0.dtsi b/arch/arm64/boot/dts/renesas/r8a774c0.dtsi index d597772c4c37..263f39fb8ad9 100644 --- a/arch/arm64/boot/dts/renesas/r8a774c0.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a774c0.dtsi @@ -1910,8 +1910,6 @@ lvds0_in: endpoint { port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; @@ -1937,8 +1935,6 @@ lvds1_in: endpoint { port@1 { reg = <1>; - lvds1_out: endpoint { - }; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a774e1.dtsi b/arch/arm64/boot/dts/renesas/r8a774e1.dtsi index 379a1300272b..02b35e68a3fa 100644 --- a/arch/arm64/boot/dts/renesas/r8a774e1.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a774e1.dtsi @@ -2891,8 +2891,6 @@ lvds0_in: endpoint { }; port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a77951.dtsi b/arch/arm64/boot/dts/renesas/r8a77951.dtsi index 1768a3e6bb8d..0919d8f540c4 100644 --- a/arch/arm64/boot/dts/renesas/r8a77951.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a77951.dtsi @@ -3362,8 +3362,6 @@ lvds0_in: endpoint { }; port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a77960.dtsi b/arch/arm64/boot/dts/renesas/r8a77960.dtsi index 2bd8169735d3..221ee1ec415d 100644 --- a/arch/arm64/boot/dts/renesas/r8a77960.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a77960.dtsi @@ -2959,8 +2959,6 @@ lvds0_in: endpoint { }; port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi index 08df75606430..5f3046eee365 100644 --- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi @@ -2771,8 +2771,6 @@ lvds0_in: endpoint { }; port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a77970.dtsi b/arch/arm64/boot/dts/renesas/r8a77970.dtsi index 517892cf6294..ad021b2703ab 100644 --- a/arch/arm64/boot/dts/renesas/r8a77970.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a77970.dtsi @@ -1190,8 +1190,6 @@ lvds0_in: endpoint { }; port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a77980.dtsi b/arch/arm64/boot/dts/renesas/r8a77980.dtsi index 6347d15e66b6..29f1161da957 100644 --- a/arch/arm64/boot/dts/renesas/r8a77980.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a77980.dtsi @@ -1567,8 +1567,6 @@ lvds0_in: endpoint { port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a77990.dtsi b/arch/arm64/boot/dts/renesas/r8a77990.dtsi index 0ea300a8147d..735c46217835 100644 --- a/arch/arm64/boot/dts/renesas/r8a77990.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a77990.dtsi @@ -2059,8 +2059,6 @@ lvds0_in: endpoint { port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; @@ -2086,8 +2084,6 @@ lvds1_in: endpoint { port@1 { reg = <1>; - lvds1_out: endpoint { - }; }; }; }; diff --git a/arch/arm64/boot/dts/renesas/r8a77995.dtsi b/arch/arm64/boot/dts/renesas/r8a77995.dtsi index 16ad5fc23a67..759505bd1a44 100644 --- a/arch/arm64/boot/dts/renesas/r8a77995.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a77995.dtsi @@ -1380,8 +1380,6 @@ lvds0_in: endpoint { port@1 { reg = <1>; - lvds0_out: endpoint { - }; }; }; }; @@ -1407,8 +1405,6 @@ lvds1_in: endpoint { port@1 { reg = <1>; - lvds1_out: endpoint { - }; }; }; };
Endpoints node must have a remote-endpoint property, as endpoints only exist to model a link between ports. Drop the empty lvds endpoints from SoC dtsi files, then should be instead declared in the board dts or in overlays. Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 2 -- arch/arm64/boot/dts/renesas/r8a774b1.dtsi | 2 -- arch/arm64/boot/dts/renesas/r8a774c0.dtsi | 4 ---- arch/arm64/boot/dts/renesas/r8a774e1.dtsi | 2 -- arch/arm64/boot/dts/renesas/r8a77951.dtsi | 2 -- arch/arm64/boot/dts/renesas/r8a77960.dtsi | 2 -- arch/arm64/boot/dts/renesas/r8a77965.dtsi | 2 -- arch/arm64/boot/dts/renesas/r8a77970.dtsi | 2 -- arch/arm64/boot/dts/renesas/r8a77980.dtsi | 2 -- arch/arm64/boot/dts/renesas/r8a77990.dtsi | 4 ---- arch/arm64/boot/dts/renesas/r8a77995.dtsi | 4 ---- 11 files changed, 28 deletions(-) base-commit: 2b534e90a1e31c7fc9536b512b72274cc3575f4c prerequisite-patch-id: 98434eb22c020924bc338316e54a7b42ea339e9f prerequisite-patch-id: ce261c9865255051f2c2107a930b0f15ce862703 prerequisite-patch-id: 500ab746415c5421e41128c2b0db35987d8d7a49 prerequisite-patch-id: ad2d5ad025f9cfa0dc453d7e0b2aa96b4514afd7 prerequisite-patch-id: ac76bcc0f7b278863e362db083a7d64bbd0bd030 prerequisite-patch-id: c7a7c3a6b9dbbae18a768160db70d7010ea71ca5