Hi Benoit, Thank you for the patch. On Monday, 26 March 2018 19:21:24 EEST Benoit Parrot wrote: > Add common DISPC bindings into the top level bindings file. > Move common bindings here instead of having multiple copies of > the same information in all of the variant specific files. > > Signed-off-by: Benoit Parrot <bparrot@xxxxxx> > Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > --- > Documentation/devicetree/bindings/display/ti/ti,dra7-dss.txt | 5 ----- > Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt | 7 +++++++ > Documentation/devicetree/bindings/display/ti/ti,omap2-dss.txt | 4 ---- > Documentation/devicetree/bindings/display/ti/ti,omap3-dss.txt | 4 ---- > Documentation/devicetree/bindings/display/ti/ti,omap4-dss.txt | 4 ---- > Documentation/devicetree/bindings/display/ti/ti,omap5-dss.txt | 4 ---- > 6 files changed, 7 insertions(+), 21 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/ti/ti,dra7-dss.txt > b/Documentation/devicetree/bindings/display/ti/ti,dra7-dss.txt index > 91279f1060fe..c30f9ec189ed 100644 > --- a/Documentation/devicetree/bindings/display/ti/ti,dra7-dss.txt > +++ b/Documentation/devicetree/bindings/display/ti/ti,dra7-dss.txt > @@ -47,11 +47,6 @@ Required properties: > - clocks: handle to fclk > - clock-names: "fck" > > -Optional properties: > -- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth > limit - in bytes per second > - > - > HDMI > ---- > > diff --git a/Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt > b/Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt index > e1ef29569338..249e588d7865 100644 > --- a/Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt > +++ b/Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt > @@ -21,6 +21,13 @@ a RGB pixel stream to encoders. > The encoder modules encode the received RGB pixel stream to a video output > like HDMI, MIPI DPI, etc. > > +DISPC > +----- > + > +Optional properties: > +- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth > limit > + in bytes per second > + I like the idea, but I think you need to expand the description a bit. ti,omap-dss.txt should explain how each module should be represented by a child DT node in the DSS DT node, and this section should briefly explain what the DISPC is (you can simply move the 4th paragraph of the file here) and that it describes common properties. You could also move the description of other DISPC properties here. > Video Ports > ----------- > > diff --git a/Documentation/devicetree/bindings/display/ti/ti,omap2-dss.txt > b/Documentation/devicetree/bindings/display/ti/ti,omap2-dss.txt index > ee867c4d1152..afcd5a86c6a4 100644 > --- a/Documentation/devicetree/bindings/display/ti/ti,omap2-dss.txt > +++ b/Documentation/devicetree/bindings/display/ti/ti,omap2-dss.txt > @@ -28,10 +28,6 @@ Required properties: > - ti,hwmods: "dss_dispc" > - interrupts: the DISPC interrupt > > -Optional properties: > -- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth > limit - in bytes per second > - > > RFBI > ---- > diff --git a/Documentation/devicetree/bindings/display/ti/ti,omap3-dss.txt > b/Documentation/devicetree/bindings/display/ti/ti,omap3-dss.txt index > cd02516a40b6..dc66e1447c31 100644 > --- a/Documentation/devicetree/bindings/display/ti/ti,omap3-dss.txt > +++ b/Documentation/devicetree/bindings/display/ti/ti,omap3-dss.txt > @@ -37,10 +37,6 @@ Required properties: > - clocks: handle to fclk > - clock-names: "fck" > > -Optional properties: > -- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth > limit - in bytes per second > - > > RFBI > ---- > diff --git a/Documentation/devicetree/bindings/display/ti/ti,omap4-dss.txt > b/Documentation/devicetree/bindings/display/ti/ti,omap4-dss.txt index > 0f85f6b3a5a8..bc624db8888d 100644 > --- a/Documentation/devicetree/bindings/display/ti/ti,omap4-dss.txt > +++ b/Documentation/devicetree/bindings/display/ti/ti,omap4-dss.txt > @@ -36,10 +36,6 @@ Required properties: > - clocks: handle to fclk > - clock-names: "fck" > > -Optional properties: > -- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth > limit - in bytes per second > - > > RFBI > ---- > diff --git a/Documentation/devicetree/bindings/display/ti/ti,omap5-dss.txt > b/Documentation/devicetree/bindings/display/ti/ti,omap5-dss.txt index > 20861218649f..118a486c47bb 100644 > --- a/Documentation/devicetree/bindings/display/ti/ti,omap5-dss.txt > +++ b/Documentation/devicetree/bindings/display/ti/ti,omap5-dss.txt > @@ -36,10 +36,6 @@ Required properties: > - clocks: handle to fclk > - clock-names: "fck" > > -Optional properties: > -- max-memory-bandwidth: Input memory (from main memory to dispc) bandwidth > limit - in bytes per second > - > > RFBI > ---- -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
