| Message ID | 20180618132242.8673-4-tomi.valkeinen@ti.com |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | None | expand |
On Mon, Jun 18, 2018 at 04:22:36PM +0300, Tomi Valkeinen wrote: > Add DT bindings for Texas Instruments AM65x SoC Display Subsystem. The > DSS7 on AM65x SoC has two video ports (DPI and OLDI) and two video > pipelines. > > Signed-off-by: Jyri Sarha <jsarha@ti.com> > Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com> > Cc: devicetree@vger.kernel.org > --- > .../bindings/display/ti/ti,am6-dss.txt | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt > > diff --git a/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt b/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt > new file mode 100644 > index 000000000000..3466f095fb84 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt > @@ -0,0 +1,16 @@ > +Texas Instruments AM65x Display Subsystem > +========================================== > + > +Required properties: > +- compatible: "ti,am65x-dss", "ti,am6-dss" Why do you need 2 compatibles? What's the difference for the 'x' in 'am65x'? > +- reg: address and length of the register spaces for DSS submodules > +- reg-names: "common", "vidl1", "vid", "ovr1", "ovr2", "vp1", "vp2" > +- clocks: phandle to fclk, vp1, and vp2 clocks > +- clock-names: "fck", "vp1", "vp2" fck or fclk? > +- interrupts: phandle to the DISPC interrupt > +- syscon: phandle to syscon device handling OLDI_PWRDN_TX (partition 1 of > + AM654 CTRL MMR0) > + > +The DSS outputs are described using the device graphs as documented in > +Documentation/devicetree/bindings/graph.txt. AM6 DSS has a DPI output as port 0 > +and an OLDI output as port 1. What's OLDI? > -- > Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. > Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Rob, On 25/06/18 21:59, Rob Herring wrote: > On Mon, Jun 18, 2018 at 04:22:36PM +0300, Tomi Valkeinen wrote: >> Add DT bindings for Texas Instruments AM65x SoC Display Subsystem. The >> DSS7 on AM65x SoC has two video ports (DPI and OLDI) and two video >> pipelines. >> >> Signed-off-by: Jyri Sarha <jsarha@ti.com> >> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com> >> Cc: devicetree@vger.kernel.org >> --- >> .../bindings/display/ti/ti,am6-dss.txt | 16 ++++++++++++++++ >> 1 file changed, 16 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt >> >> diff --git a/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt b/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt >> new file mode 100644 >> index 000000000000..3466f095fb84 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt >> @@ -0,0 +1,16 @@ >> +Texas Instruments AM65x Display Subsystem >> +========================================== >> + >> +Required properties: >> +- compatible: "ti,am65x-dss", "ti,am6-dss" > > Why do you need 2 compatibles? What's the difference for the 'x' in > 'am65x'? For now we have only one AM6xx SoC, but probably there will be more variants, possibly with slightly different DSS setups (say, a single output version). But then, maybe all the variants will have the exact same DSS. I'm not quite sure how to handle it here, so I added am6-dss as a base, so that in the DT file we could have the compatible as "exact-model-name","am6-dss", and only handle am6-dss in the driver unless we get models with different DSS features. But then I probably should not use am65x, and instead use the exact model name here (am654), even if the last number in the model should not affect DSS. >> +- reg: address and length of the register spaces for DSS submodules >> +- reg-names: "common", "vidl1", "vid", "ovr1", "ovr2", "vp1", "vp2" >> +- clocks: phandle to fclk, vp1, and vp2 clocks >> +- clock-names: "fck", "vp1", "vp2" > > fck or fclk? fck. I'll fix that. >> +- interrupts: phandle to the DISPC interrupt >> +- syscon: phandle to syscon device handling OLDI_PWRDN_TX (partition 1 of >> + AM654 CTRL MMR0) >> + >> +The DSS outputs are described using the device graphs as documented in >> +Documentation/devicetree/bindings/graph.txt. AM6 DSS has a DPI output as port 0 >> +and an OLDI output as port 1. > > What's OLDI? OpenLDI (https://en.wikipedia.org/wiki/OpenLDI). I should probably not abbreviate it here. Tomi
Hi Tomi, Thank you for the patch. On Monday, 18 June 2018 16:22:36 EEST Tomi Valkeinen wrote: > Add DT bindings for Texas Instruments AM65x SoC Display Subsystem. The > DSS7 on AM65x SoC has two video ports (DPI and OLDI) and two video > pipelines. > > Signed-off-by: Jyri Sarha <jsarha@ti.com> > Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com> > Cc: devicetree@vger.kernel.org > --- > .../bindings/display/ti/ti,am6-dss.txt | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt > > diff --git a/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt > b/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt new file mode > 100644 > index 000000000000..3466f095fb84 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt > @@ -0,0 +1,16 @@ > +Texas Instruments AM65x Display Subsystem > +========================================== > + > +Required properties: > +- compatible: "ti,am65x-dss", "ti,am6-dss" > +- reg: address and length of the register spaces for DSS submodules > +- reg-names: "common", "vidl1", "vid", "ovr1", "ovr2", "vp1", "vp2" > +- clocks: phandle to fclk, vp1, and vp2 clocks > +- clock-names: "fck", "vp1", "vp2" > +- interrupts: phandle to the DISPC interrupt > +- syscon: phandle to syscon device handling OLDI_PWRDN_TX (partition 1 of > + AM654 CTRL MMR0) > + > +The DSS outputs are described using the device graphs as documented in > +Documentation/devicetree/bindings/graph.txt. AM6 DSS has a DPI output as > port 0 > +and an OLDI output as port 1. Without documentation for the AM65x it's hard to review these DT bindings. However, my comment regarding the registers of the k2g-dss bindings applies here too.
On 30/07/18 02:04, Laurent Pinchart wrote: > Hi Tomi, > > Thank you for the patch. > > On Monday, 18 June 2018 16:22:36 EEST Tomi Valkeinen wrote: >> Add DT bindings for Texas Instruments AM65x SoC Display Subsystem. The >> DSS7 on AM65x SoC has two video ports (DPI and OLDI) and two video >> pipelines. >> >> Signed-off-by: Jyri Sarha <jsarha@ti.com> >> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com> >> Cc: devicetree@vger.kernel.org >> --- >> .../bindings/display/ti/ti,am6-dss.txt | 16 ++++++++++++++++ >> 1 file changed, 16 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt >> >> diff --git a/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt >> b/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt new file mode >> 100644 >> index 000000000000..3466f095fb84 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt >> @@ -0,0 +1,16 @@ >> +Texas Instruments AM65x Display Subsystem >> +========================================== >> + >> +Required properties: >> +- compatible: "ti,am65x-dss", "ti,am6-dss" >> +- reg: address and length of the register spaces for DSS submodules >> +- reg-names: "common", "vidl1", "vid", "ovr1", "ovr2", "vp1", "vp2" >> +- clocks: phandle to fclk, vp1, and vp2 clocks >> +- clock-names: "fck", "vp1", "vp2" >> +- interrupts: phandle to the DISPC interrupt >> +- syscon: phandle to syscon device handling OLDI_PWRDN_TX (partition 1 of >> + AM654 CTRL MMR0) >> + >> +The DSS outputs are described using the device graphs as documented in >> +Documentation/devicetree/bindings/graph.txt. AM6 DSS has a DPI output as >> port 0 >> +and an OLDI output as port 1. > > Without documentation for the AM65x it's hard to review these DT bindings. > However, my comment regarding the registers of the k2g-dss bindings applies > here too. Sorry that I didn't provide the link. I guess it's not easily found from ti.com at the moment: http://www.ti.com/lit/ug/spruid7/spruid7.pdf
diff --git a/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt b/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt new file mode 100644 index 000000000000..3466f095fb84 --- /dev/null +++ b/Documentation/devicetree/bindings/display/ti/ti,am6-dss.txt @@ -0,0 +1,16 @@ +Texas Instruments AM65x Display Subsystem +========================================== + +Required properties: +- compatible: "ti,am65x-dss", "ti,am6-dss" +- reg: address and length of the register spaces for DSS submodules +- reg-names: "common", "vidl1", "vid", "ovr1", "ovr2", "vp1", "vp2" +- clocks: phandle to fclk, vp1, and vp2 clocks +- clock-names: "fck", "vp1", "vp2" +- interrupts: phandle to the DISPC interrupt +- syscon: phandle to syscon device handling OLDI_PWRDN_TX (partition 1 of + AM654 CTRL MMR0) + +The DSS outputs are described using the device graphs as documented in +Documentation/devicetree/bindings/graph.txt. AM6 DSS has a DPI output as port 0 +and an OLDI output as port 1.