Re: [PATCH v3 2/9] dt/bindings: drm/komeda: Add DT bindings for ARM display processor D71
On Mon, Dec 24, 2018 at 08:00:51PM +0800, Liviu Dudau wrote: > On Fri, Dec 21, 2018 at 09:59:12AM +, james qian wang (Arm Technology > China) wrote: > > Add DT bindings documentation for the ARM display processor D71 and later > > IPs. > > > > Signed-off-by: James (Qian) Wang > > > > Changes in v3: > > - Deleted unnecessary property: interrupt-names. > > - Dropped 'ports' and moving 'port' up a level. > > --- > > .../bindings/display/arm/arm,komeda.txt | 79 +++ > > 1 file changed, 79 insertions(+) > > create mode 100644 > > Documentation/devicetree/bindings/display/arm/arm,komeda.txt > > > > diff --git a/Documentation/devicetree/bindings/display/arm/arm,komeda.txt > > b/Documentation/devicetree/bindings/display/arm/arm,komeda.txt > > new file mode 100644 > > index ..b4e450243c7d > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/display/arm/arm,komeda.txt > > @@ -0,0 +1,79 @@ > > +Device Tree bindings for ARM Komeda display driver > > + > > +Required properties: > > +- compatible: Should be "arm,mali-d71" > > +- reg: Physical base address and length of the registers in the system > > +- interrupts: the interrupt line number of the device in the system > > +- clocks: A list of phandle + clock-specifier pairs, one for each entry > > +in 'clock-names' > > +- clock-names: A list of clock names. It should contain: > > + - "mclk": for the main processor clock > > + - "pclk": for the APB interface clock > > +- #address-cells: Must be 1 > > +- #size-cells: Must be 0 > > + > > +Required properties for sub-node: pipeline@nq > > +Each device contains one or two pipeline sub-nodes (at least one), each > > +pipeline node should provide properties: > > +- reg: Zero-indexed identifier for the pipeline > > +- clocks: A list of phandle + clock-specifier pairs, one for each entry > > +in 'clock-names' > > +- clock-names: should contain: > > + - "pxclk": pixel clock > > + - "aclk": AXI interface clock > > + > > +- port: each pipeline connect to an encoder input port. The connection is > > +modeled using the OF graph bindings specified in > > +Documentation/devicetree/bindings/graph.txt > > + > > +Optional properties: > > + - memory-region: phandle to a node describing memory (see > > +Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt) > > +to be used for the framebuffer; if not present, the framebuffer may > > +be located anywhere in memory. > > + > > +Example: > > +/ { > > + ... > > + > > + dp0: display@c0 { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + compatible = "arm,mali-d71"; > > + reg = <0xc0 0x2>; > > + interrupts = <0 168 4>; > > + clocks = <_mclk>, <_aclk>; > > + clock-names = "mclk", "pclk"; > > + > > + dp0_pipe0: pipeline@0 { > > + clocks = <>, <_aclk>; > > + clock-names = "pxclk", "aclk"; > > + reg = <0>; > > + > > + #address-cells = <1>; > > + #size-cells = <0>; > > These are undocumented and not necessary anyway, as the pipelines will > inherit display's attributes. thank you, will fix it in the next version. > > + > > + port@0 { > > + dp0_pipe0_out: endpoint { > > + remote-endpoint = <_dvi0_in>; > > + }; > > + }; > > + }; > > + > > + dp0_pipe1: pipeline@1 { > > + clocks = <>, <_aclk>; > > + clock-names = "pxclk", "aclk"; > > + reg = <1>; > > + > > + #address-cells = <1>; > > + #size-cells = <0>; > > same here. OK. > > > + > > + port@0 { > > + dp0_pipe1_out: endpoint { > > + remote-endpoint = <_dvi1_in>; > > + }; > > + }; > > + }; > > + }; > > + ... > > +}; > > -- > > 2.17.1 > > > > With these changes: > > Reviewed-by: Liviu Dudau > > Best regards, > Liviu > > > -- > > | I would like to | > | fix the world, | > | but they're not | > | giving me the | > \ source code! / > --- > ¯\_(ツ)_/¯ ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v3 2/9] dt/bindings: drm/komeda: Add DT bindings for ARM display processor D71
On Fri, Dec 21, 2018 at 09:59:12AM +, james qian wang (Arm Technology China) wrote: > Add DT bindings documentation for the ARM display processor D71 and later > IPs. > > Signed-off-by: James (Qian) Wang > > Changes in v3: > - Deleted unnecessary property: interrupt-names. > - Dropped 'ports' and moving 'port' up a level. > --- > .../bindings/display/arm/arm,komeda.txt | 79 +++ > 1 file changed, 79 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/display/arm/arm,komeda.txt > > diff --git a/Documentation/devicetree/bindings/display/arm/arm,komeda.txt > b/Documentation/devicetree/bindings/display/arm/arm,komeda.txt > new file mode 100644 > index ..b4e450243c7d > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/arm/arm,komeda.txt > @@ -0,0 +1,79 @@ > +Device Tree bindings for ARM Komeda display driver > + > +Required properties: > +- compatible: Should be "arm,mali-d71" > +- reg: Physical base address and length of the registers in the system > +- interrupts: the interrupt line number of the device in the system > +- clocks: A list of phandle + clock-specifier pairs, one for each entry > +in 'clock-names' > +- clock-names: A list of clock names. It should contain: > + - "mclk": for the main processor clock > + - "pclk": for the APB interface clock > +- #address-cells: Must be 1 > +- #size-cells: Must be 0 > + > +Required properties for sub-node: pipeline@nq > +Each device contains one or two pipeline sub-nodes (at least one), each > +pipeline node should provide properties: > +- reg: Zero-indexed identifier for the pipeline > +- clocks: A list of phandle + clock-specifier pairs, one for each entry > +in 'clock-names' > +- clock-names: should contain: > + - "pxclk": pixel clock > + - "aclk": AXI interface clock > + > +- port: each pipeline connect to an encoder input port. The connection is > +modeled using the OF graph bindings specified in > +Documentation/devicetree/bindings/graph.txt > + > +Optional properties: > + - memory-region: phandle to a node describing memory (see > +Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt) > +to be used for the framebuffer; if not present, the framebuffer may > +be located anywhere in memory. > + > +Example: > +/ { > + ... > + > + dp0: display@c0 { > + #address-cells = <1>; > + #size-cells = <0>; > + compatible = "arm,mali-d71"; > + reg = <0xc0 0x2>; > + interrupts = <0 168 4>; > + clocks = <_mclk>, <_aclk>; > + clock-names = "mclk", "pclk"; > + > + dp0_pipe0: pipeline@0 { > + clocks = <>, <_aclk>; > + clock-names = "pxclk", "aclk"; > + reg = <0>; > + > + #address-cells = <1>; > + #size-cells = <0>; These are undocumented and not necessary anyway, as the pipelines will inherit display's attributes. > + > + port@0 { > + dp0_pipe0_out: endpoint { > + remote-endpoint = <_dvi0_in>; > + }; > + }; > + }; > + > + dp0_pipe1: pipeline@1 { > + clocks = <>, <_aclk>; > + clock-names = "pxclk", "aclk"; > + reg = <1>; > + > + #address-cells = <1>; > + #size-cells = <0>; same here. > + > + port@0 { > + dp0_pipe1_out: endpoint { > + remote-endpoint = <_dvi1_in>; > + }; > + }; > + }; > + }; > + ... > +}; > -- > 2.17.1 > With these changes: Reviewed-by: Liviu Dudau Best regards, Liviu -- | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --- ¯\_(ツ)_/¯ ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
[PATCH v3 2/9] dt/bindings: drm/komeda: Add DT bindings for ARM display processor D71
Add DT bindings documentation for the ARM display processor D71 and later IPs. Signed-off-by: James (Qian) Wang Changes in v3: - Deleted unnecessary property: interrupt-names. - Dropped 'ports' and moving 'port' up a level. --- .../bindings/display/arm/arm,komeda.txt | 79 +++ 1 file changed, 79 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/arm/arm,komeda.txt diff --git a/Documentation/devicetree/bindings/display/arm/arm,komeda.txt b/Documentation/devicetree/bindings/display/arm/arm,komeda.txt new file mode 100644 index ..b4e450243c7d --- /dev/null +++ b/Documentation/devicetree/bindings/display/arm/arm,komeda.txt @@ -0,0 +1,79 @@ +Device Tree bindings for ARM Komeda display driver + +Required properties: +- compatible: Should be "arm,mali-d71" +- reg: Physical base address and length of the registers in the system +- interrupts: the interrupt line number of the device in the system +- clocks: A list of phandle + clock-specifier pairs, one for each entry +in 'clock-names' +- clock-names: A list of clock names. It should contain: + - "mclk": for the main processor clock + - "pclk": for the APB interface clock +- #address-cells: Must be 1 +- #size-cells: Must be 0 + +Required properties for sub-node: pipeline@nq +Each device contains one or two pipeline sub-nodes (at least one), each +pipeline node should provide properties: +- reg: Zero-indexed identifier for the pipeline +- clocks: A list of phandle + clock-specifier pairs, one for each entry +in 'clock-names' +- clock-names: should contain: + - "pxclk": pixel clock + - "aclk": AXI interface clock + +- port: each pipeline connect to an encoder input port. The connection is +modeled using the OF graph bindings specified in +Documentation/devicetree/bindings/graph.txt + +Optional properties: + - memory-region: phandle to a node describing memory (see +Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt) +to be used for the framebuffer; if not present, the framebuffer may +be located anywhere in memory. + +Example: +/ { + ... + + dp0: display@c0 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "arm,mali-d71"; + reg = <0xc0 0x2>; + interrupts = <0 168 4>; + clocks = <_mclk>, <_aclk>; + clock-names = "mclk", "pclk"; + + dp0_pipe0: pipeline@0 { + clocks = <>, <_aclk>; + clock-names = "pxclk", "aclk"; + reg = <0>; + + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + dp0_pipe0_out: endpoint { + remote-endpoint = <_dvi0_in>; + }; + }; + }; + + dp0_pipe1: pipeline@1 { + clocks = <>, <_aclk>; + clock-names = "pxclk", "aclk"; + reg = <1>; + + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + dp0_pipe1_out: endpoint { + remote-endpoint = <_dvi1_in>; + }; + }; + }; + }; + ... +}; -- 2.17.1 ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel