* [PATCH] drm: rcar-du: Document the vsps property in the DT bindings @ 2017-03-27 9:56 Laurent Pinchart 2017-03-27 11:05 ` Geert Uytterhoeven 0 siblings, 1 reply; 7+ messages in thread From: Laurent Pinchart @ 2017-03-27 9:56 UTC (permalink / raw) To: devicetree; +Cc: linux-renesas-soc, dri-devel The property is used by the driver but is missing from the DT bindings. Document it. Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- Documentation/devicetree/bindings/display/renesas,du.txt | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt b/Documentation/devicetree/bindings/display/renesas,du.txt index 1a02f099a0ff..cf34893a1b53 100644 --- a/Documentation/devicetree/bindings/display/renesas,du.txt +++ b/Documentation/devicetree/bindings/display/renesas,du.txt @@ -36,6 +36,11 @@ Required Properties: When supplied they must be named "dclkin.x" with "x" being the input clock numerical index. +Optional Properties: + + - vsps: A list of phandles to the VSP nodes that handle the memory + interfaces for the DU channels (Gen3 only). + Required nodes: The connections to the DU output video ports are modeled using the OF graph -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: rcar-du: Document the vsps property in the DT bindings 2017-03-27 9:56 [PATCH] drm: rcar-du: Document the vsps property in the DT bindings Laurent Pinchart @ 2017-03-27 11:05 ` Geert Uytterhoeven 2017-03-31 9:19 ` Laurent Pinchart 0 siblings, 1 reply; 7+ messages in thread From: Geert Uytterhoeven @ 2017-03-27 11:05 UTC (permalink / raw) To: Laurent Pinchart Cc: Linux-Renesas, devicetree@vger.kernel.org, DRI Development, Sergei Shtylyov Hi Laurent, On Mon, Mar 27, 2017 at 11:56 AM, Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> wrote: > The property is used by the driver but is missing from the DT bindings. > Document it. > > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > Documentation/devicetree/bindings/display/renesas,du.txt | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt b/Documentation/devicetree/bindings/display/renesas,du.txt > index 1a02f099a0ff..cf34893a1b53 100644 > --- a/Documentation/devicetree/bindings/display/renesas,du.txt > +++ b/Documentation/devicetree/bindings/display/renesas,du.txt > @@ -36,6 +36,11 @@ Required Properties: > When supplied they must be named "dclkin.x" with "x" being the input > clock numerical index. > > +Optional Properties: > + > + - vsps: A list of phandles to the VSP nodes that handle the memory > + interfaces for the DU channels (Gen3 only). ", one per channel"? Required for Gen3, optional for Gen2? (cfr. Sergei's patches). 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 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: rcar-du: Document the vsps property in the DT bindings 2017-03-27 11:05 ` Geert Uytterhoeven @ 2017-03-31 9:19 ` Laurent Pinchart 2017-03-31 9:21 ` Geert Uytterhoeven 2017-04-18 17:36 ` Geert Uytterhoeven 0 siblings, 2 replies; 7+ messages in thread From: Laurent Pinchart @ 2017-03-31 9:19 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Laurent Pinchart, Sergei Shtylyov, devicetree@vger.kernel.org, DRI Development, Linux-Renesas Hi Geert, On Monday 27 Mar 2017 13:05:48 Geert Uytterhoeven wrote: > On Mon, Mar 27, 2017 at 11:56 AM, Laurent Pinchart wrote: > > The property is used by the driver but is missing from the DT bindings. > > Document it. > > > > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> > > Signed-off-by: Laurent Pinchart > > <laurent.pinchart+renesas@ideasonboard.com> > > --- > > > > Documentation/devicetree/bindings/display/renesas,du.txt | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt > > b/Documentation/devicetree/bindings/display/renesas,du.txt index > > 1a02f099a0ff..cf34893a1b53 100644 > > --- a/Documentation/devicetree/bindings/display/renesas,du.txt > > +++ b/Documentation/devicetree/bindings/display/renesas,du.txt > > > > @@ -36,6 +36,11 @@ Required Properties: > > When supplied they must be named "dclkin.x" with "x" being the > > input > > clock numerical index. > > > > +Optional Properties: > > + > > + - vsps: A list of phandles to the VSP nodes that handle the memory > > + interfaces for the DU channels (Gen3 only). > > ", one per channel"? > > Required for Gen3, optional for Gen2? (cfr. Sergei's patches). How about making it mandatory on Gen2 as well ? The VSPs are there, even if the driver doesn't use them, it makes sense to describe the connection. Of course the driver will treat the property as optional for backward compatibility. -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: rcar-du: Document the vsps property in the DT bindings 2017-03-31 9:19 ` Laurent Pinchart @ 2017-03-31 9:21 ` Geert Uytterhoeven 2017-04-05 7:54 ` Geert Uytterhoeven 2017-04-18 17:36 ` Geert Uytterhoeven 1 sibling, 1 reply; 7+ messages in thread From: Geert Uytterhoeven @ 2017-03-31 9:21 UTC (permalink / raw) To: Laurent Pinchart Cc: Laurent Pinchart, Sergei Shtylyov, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, DRI Development, Linux-Renesas Hi Laurent, On Fri, Mar 31, 2017 at 11:19 AM, Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org> wrote: > On Monday 27 Mar 2017 13:05:48 Geert Uytterhoeven wrote: >> On Mon, Mar 27, 2017 at 11:56 AM, Laurent Pinchart wrote: >> > The property is used by the driver but is missing from the DT bindings. >> > Document it. >> > >> > Reported-by: Geert Uytterhoeven <geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org> >> > Signed-off-by: Laurent Pinchart >> > <laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org> >> > --- >> > Documentation/devicetree/bindings/display/renesas,du.txt | 5 +++++ >> > 1 file changed, 5 insertions(+) >> > >> > diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt >> > b/Documentation/devicetree/bindings/display/renesas,du.txt index >> > 1a02f099a0ff..cf34893a1b53 100644 >> > --- a/Documentation/devicetree/bindings/display/renesas,du.txt >> > +++ b/Documentation/devicetree/bindings/display/renesas,du.txt >> > >> > @@ -36,6 +36,11 @@ Required Properties: >> > When supplied they must be named "dclkin.x" with "x" being the >> > input >> > clock numerical index. >> > >> > +Optional Properties: >> > + >> > + - vsps: A list of phandles to the VSP nodes that handle the memory >> > + interfaces for the DU channels (Gen3 only). >> >> ", one per channel"? >> >> Required for Gen3, optional for Gen2? (cfr. Sergei's patches). > > How about making it mandatory on Gen2 as well ? The VSPs are there, even if > the driver doesn't use them, it makes sense to describe the connection. Of Fine for me, as this is hardware description. > course the driver will treat the property as optional for backward > compatibility. OK. Disclaimer: I didn't follow the discussion about using or not using the vsps on Gen2 that closely, and don't remember the outcome. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: rcar-du: Document the vsps property in the DT bindings 2017-03-31 9:21 ` Geert Uytterhoeven @ 2017-04-05 7:54 ` Geert Uytterhoeven 2017-04-05 8:24 ` Laurent Pinchart 0 siblings, 1 reply; 7+ messages in thread From: Geert Uytterhoeven @ 2017-04-05 7:54 UTC (permalink / raw) To: Laurent Pinchart Cc: Laurent Pinchart, Sergei Shtylyov, devicetree@vger.kernel.org, DRI Development, Linux-Renesas Hi Laurent, On Fri, Mar 31, 2017 at 11:21 AM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > On Fri, Mar 31, 2017 at 11:19 AM, Laurent Pinchart > <laurent.pinchart@ideasonboard.com> wrote: >> On Monday 27 Mar 2017 13:05:48 Geert Uytterhoeven wrote: >>> On Mon, Mar 27, 2017 at 11:56 AM, Laurent Pinchart wrote: >>> > The property is used by the driver but is missing from the DT bindings. >>> > Document it. >>> > >>> > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> >>> > Signed-off-by: Laurent Pinchart >>> > <laurent.pinchart+renesas@ideasonboard.com> >>> > --- >>> > Documentation/devicetree/bindings/display/renesas,du.txt | 5 +++++ >>> > 1 file changed, 5 insertions(+) >>> > >>> > diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt >>> > b/Documentation/devicetree/bindings/display/renesas,du.txt index >>> > 1a02f099a0ff..cf34893a1b53 100644 >>> > --- a/Documentation/devicetree/bindings/display/renesas,du.txt >>> > +++ b/Documentation/devicetree/bindings/display/renesas,du.txt >>> > >>> > @@ -36,6 +36,11 @@ Required Properties: >>> > When supplied they must be named "dclkin.x" with "x" being the >>> > input >>> > clock numerical index. >>> > >>> > +Optional Properties: >>> > + >>> > + - vsps: A list of phandles to the VSP nodes that handle the memory >>> > + interfaces for the DU channels (Gen3 only). >>> >>> ", one per channel"? >>> >>> Required for Gen3, optional for Gen2? (cfr. Sergei's patches). >> >> How about making it mandatory on Gen2 as well ? The VSPs are there, even if >> the driver doesn't use them, it makes sense to describe the connection. Of > > Fine for me, as this is hardware description. > >> course the driver will treat the property as optional for backward >> compatibility. > > OK. Will you do this as an incremental update? I noticed the initial version is now in drm-next. Thanks! 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: rcar-du: Document the vsps property in the DT bindings 2017-04-05 7:54 ` Geert Uytterhoeven @ 2017-04-05 8:24 ` Laurent Pinchart 0 siblings, 0 replies; 7+ messages in thread From: Laurent Pinchart @ 2017-04-05 8:24 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Linux-Renesas, devicetree@vger.kernel.org, Laurent Pinchart, DRI Development, Sergei Shtylyov Hi Geert, On Wednesday 05 Apr 2017 09:54:38 Geert Uytterhoeven wrote: > On Fri, Mar 31, 2017 at 11:21 AM, Geert Uytterhoeven wrote: > > On Fri, Mar 31, 2017 at 11:19 AM, Laurent Pinchart wrote: > >> On Monday 27 Mar 2017 13:05:48 Geert Uytterhoeven wrote: > >>> On Mon, Mar 27, 2017 at 11:56 AM, Laurent Pinchart wrote: > >>>> The property is used by the driver but is missing from the DT > >>>> bindings. Document it. > >>>> > >>>> Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> > >>>> Signed-off-by: Laurent Pinchart > >>>> <laurent.pinchart+renesas@ideasonboard.com> > >>>> --- > >>>> > >>>> Documentation/devicetree/bindings/display/renesas,du.txt | 5 +++++ > >>>> 1 file changed, 5 insertions(+) > >>>> > >>>> diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt > >>>> b/Documentation/devicetree/bindings/display/renesas,du.txt index > >>>> 1a02f099a0ff..cf34893a1b53 100644 > >>>> --- a/Documentation/devicetree/bindings/display/renesas,du.txt > >>>> +++ b/Documentation/devicetree/bindings/display/renesas,du.txt > >>>> > >>>> @@ -36,6 +36,11 @@ Required Properties: > >>>> When supplied they must be named "dclkin.x" with "x" being the > >>>> input clock numerical index. > >>> > > >>> > +Optional Properties: > >>> > + > >>> > + - vsps: A list of phandles to the VSP nodes that handle the memory > >>> > + interfaces for the DU channels (Gen3 only). > >>> > >>> ", one per channel"? > >>> > >>> Required for Gen3, optional for Gen2? (cfr. Sergei's patches). > >> > >> How about making it mandatory on Gen2 as well ? The VSPs are there, even > >> if the driver doesn't use them, it makes sense to describe the > >> connection. Of > > > > Fine for me, as this is hardware description. > > > >> course the driver will treat the property as optional for backward > >> compatibility. > > > > OK. > > Will you do this as an incremental update? > I noticed the initial version is now in drm-next. I've included it in the pull request by mistake and realized that too late, sorry :-/ I will send an incremental update. -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm: rcar-du: Document the vsps property in the DT bindings 2017-03-31 9:19 ` Laurent Pinchart 2017-03-31 9:21 ` Geert Uytterhoeven @ 2017-04-18 17:36 ` Geert Uytterhoeven 1 sibling, 0 replies; 7+ messages in thread From: Geert Uytterhoeven @ 2017-04-18 17:36 UTC (permalink / raw) To: Laurent Pinchart Cc: Laurent Pinchart, Sergei Shtylyov, devicetree@vger.kernel.org, DRI Development, Linux-Renesas Hi Laurent, On Fri, Mar 31, 2017 at 11:19 AM, Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > On Monday 27 Mar 2017 13:05:48 Geert Uytterhoeven wrote: >> On Mon, Mar 27, 2017 at 11:56 AM, Laurent Pinchart wrote: >> > The property is used by the driver but is missing from the DT bindings. >> > Document it. >> > >> > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> >> > Signed-off-by: Laurent Pinchart >> > <laurent.pinchart+renesas@ideasonboard.com> >> > --- >> > >> > Documentation/devicetree/bindings/display/renesas,du.txt | 5 +++++ >> > 1 file changed, 5 insertions(+) >> > >> > diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt >> > b/Documentation/devicetree/bindings/display/renesas,du.txt index >> > 1a02f099a0ff..cf34893a1b53 100644 >> > --- a/Documentation/devicetree/bindings/display/renesas,du.txt >> > +++ b/Documentation/devicetree/bindings/display/renesas,du.txt >> > >> > @@ -36,6 +36,11 @@ Required Properties: >> > When supplied they must be named "dclkin.x" with "x" being the >> > input >> > clock numerical index. >> > >> > +Optional Properties: >> > + >> > + - vsps: A list of phandles to the VSP nodes that handle the memory >> > + interfaces for the DU channels (Gen3 only). >> >> ", one per channel"? >> >> Required for Gen3, optional for Gen2? (cfr. Sergei's patches). > > How about making it mandatory on Gen2 as well ? The VSPs are there, even if > the driver doesn't use them, it makes sense to describe the connection. Of > course the driver will treat the property as optional for backward > compatibility. Now it's mandatory, the vsps property should be present in the example, too. 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-04-18 17:36 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-03-27 9:56 [PATCH] drm: rcar-du: Document the vsps property in the DT bindings Laurent Pinchart 2017-03-27 11:05 ` Geert Uytterhoeven 2017-03-31 9:19 ` Laurent Pinchart 2017-03-31 9:21 ` Geert Uytterhoeven 2017-04-05 7:54 ` Geert Uytterhoeven 2017-04-05 8:24 ` Laurent Pinchart 2017-04-18 17:36 ` Geert Uytterhoeven
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).