* Re: [PATCH 2/4] drm/mcde: Add device tree bindings
2019-02-07 8:36 ` [PATCH 2/4] drm/mcde: Add device tree bindings Linus Walleij
@ 2019-02-25 22:31 ` Rob Herring
0 siblings, 0 replies; 2+ messages in thread
From: Rob Herring @ 2019-02-25 22:31 UTC (permalink / raw)
To: Linus Walleij; +Cc: David Airlie, devicetree, dri-devel, linux-arm-kernel
On Thu, Feb 07, 2019 at 09:36:45AM +0100, Linus Walleij wrote:
> This adds the device tree bindings for the ST-Ericsson
> Multi Channel Display Engine MCDE as found in the U8500
> SoCs.
>
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> .../devicetree/bindings/display/ste,mcde.txt | 110 ++++++++++++++++++
> 1 file changed, 110 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/display/ste,mcde.txt
>
> diff --git a/Documentation/devicetree/bindings/display/ste,mcde.txt b/Documentation/devicetree/bindings/display/ste,mcde.txt
> new file mode 100644
> index 000000000000..fc58aa5effb5
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/display/ste,mcde.txt
> @@ -0,0 +1,110 @@
> +ST-Ericsson Multi Channel Display Engine MCDE
> +
> +The ST-Ericsson MCDE is a display controller with support for compositing
> +and displaying several channels memory resident graphics data on DSI or
> +LCD displays or bridges. It is used in the ST-Ericsson U8500 platform.
> +
> +Required properties:
> +
> +- compatible: must be:
> + "ste,mcde"
Only one version? This too is old enough, I'm not too worried about how
specific the compatibles are here.
> +- reg: register base for the main MCDE control registers, should be
> + 0x1000 in size
> +- interrupts: the interrupt line for the MCDE
> +- epod-supply: a phandle to the EPOD regulator
> +- vana-supply: a phandle to the analog voltage regulator
> +- clocks: an array of the MCDE clocks in this strict order:
> + MCDECLK (main MCDE clock), LCDCLK (LCD clock), PLLDSI
> + (HDMI clock), DSI0ESCLK (DSI0 energy save clock),
> + DSI1ESCLK (DSI1 energy save clock), DSI2ESCLK (DSI2 energy
> + save clock)
> +- clock-names: must be the following array:
> + "mcde", "lcd", "hdmi", "dsi0", "dsi1", "dsi0es", "dsi1es", "dsi2es"
dsi2 clock?
Should the dsi clocks be in the child nodes?
> + to match the required clock inputs above.
> +- #address-cells: should be <1> (for the DSI hosts that will be children)
> +- #size-cells: should be <1> (for the DSI hosts that will be children)
> +- ranges: this should always be stated
> +
> +Required subnodes:
> +
> +The devicetree must specify subnodes for the DSI host adapters.
> +These must have the following characteristics:
> +
> +- compatible: must be:
> + "ste,mcde-dsi"
> +- reg: must specify the register range for the DSI host
> +- vana-supply: phandle to the VANA voltage regulator
> +- #address-cells: should be <1>
> +- #size-cells: should be <0>
> +
> +Display panels and bridges will appear as children on the DSI hosts, and
> +the displays are connected to the DSI hosts using the common binding
> +for video transmitter interfaces; see
> +Documentation/devicetree/bindings/media/video-interfaces.txt
> +
> +If a DSI host is unused (not connected) it will have no children or ports
> +defined.
> +
> +Example:
> +
> +mcde@a0350000 {
> + compatible = "ste,mcde";
> + reg = <0xa0350000 0x1000>;
> + interrupts = <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>;
> + epod-supply = <&db8500_b2r2_mcde_reg>;
> + vana-supply = <&ab8500_ldo_ana_reg>;
> + clocks = <&prcmu_clk PRCMU_MCDECLK>, /* Main MCDE clock */
> + <&prcmu_clk PRCMU_LCDCLK>, /* LCD clock */
> + <&prcmu_clk PRCMU_PLLDSI>, /* HDMI clock */
> + <&prcmu_clk PRCMU_DSI0CLK>, /* DSI 0 */
> + <&prcmu_clk PRCMU_DSI1CLK>, /* DSI 1 */
> + <&prcmu_clk PRCMU_DSI0ESCCLK>, /* TVout clock 0 */
> + <&prcmu_clk PRCMU_DSI1ESCCLK>, /* TVout clock 1 */
> + <&prcmu_clk PRCMU_DSI2ESCCLK>; /* TVout clock 2 */
> + clock-names = "mcde", "lcd", "hdmi", "dsi0", "dsi1",
> + "dsi0es", "dsi1es", "dsi2es";
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges;
A non-empty ranges is preferred, then you can just do offsets below.
> +
> + dsi0: dsi@a0351000 {
> + compatible = "ste,mcde-dsi";
> + reg = <0xa0351000 0x1000>;
> + vana-supply = <&ab8500_ldo_ana_reg>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + port {
> + dsi0_out: endpoint {
> + remote-endpoint = <&panel_in>;
> + };
> + };
> +
> + panel: display {
'panel' would be the somewhat more standard node name.
> + compatible = "samsung,s6d16d0";
> + reg = <0>;
> + vdd1-supply = <&ab8500_ldo_aux1_reg>;
> + reset-gpios = <&gpio2 1 GPIO_ACTIVE_LOW>;
> +
> + port {
> + panel_in: endpoint {
> + remote-endpoint = <&dsi0_out>;
You don't really need the graph here as it is already a child. Generally
it is either or. The driver can support both ways though.
> + };
> + };
> + };
> +
> + };
> + dsi1: dsi@a0352000 {
> + compatible = "ste,mcde-dsi";
> + reg = <0xa0352000 0x1000>;
> + vana-supply = <&ab8500_ldo_ana_reg>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + };
> + dsi2: dsi@a0353000 {
> + compatible = "ste,mcde-dsi";
> + reg = <0xa0353000 0x1000>;
> + vana-supply = <&ab8500_ldo_ana_reg>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + };
> +};
> --
> 2.20.1
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 2+ messages in thread