diff mbox series

arm64: dts: imx8mq: Add eLCDIF controller

Message ID 59793b1ae533636528942b2cec14ec68b9830fcf.1574510649.git.agx@sigxcpu.org (mailing list archive)
State New, archived
Headers show
Series arm64: dts: imx8mq: Add eLCDIF controller | expand

Commit Message

Guido Günther Nov. 23, 2019, 12:08 p.m. UTC
Add a node for the eLCDIF controller, "disabled" by default.

Signed-off-by: Guido Günther <agx@sigxcpu.org>
---
With some minimal support on imx8mq we might as well add it to the DT

 arch/arm64/boot/dts/freescale/imx8mq.dtsi | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

Comments

Fabio Estevam Nov. 23, 2019, 12:25 p.m. UTC | #1
Hi Guido,

On Sat, Nov 23, 2019 at 9:09 AM Guido Günther <agx@sigxcpu.org> wrote:

> diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> index 7f9319452b58..00aa63bfd816 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> @@ -448,6 +448,23 @@
>                                 fsl,sdma-ram-script-name = "imx/sdma/sdma-imx7d.bin";
>                         };
>
> +                       lcdif: lcdif@30320000 {
> +                               compatible = "fsl,imx8mq-lcdif", "fsl,imx28-lcdif";

fsl,imx8mq-lcdif should also be documented.

Reviewed-by: Fabio Estevam <festevam@gmail.com>

Thanks
Fabio Estevam Nov. 23, 2019, 5:09 p.m. UTC | #2
On Sat, Nov 23, 2019 at 9:09 AM Guido Günther <agx@sigxcpu.org> wrote:
>
> Add a node for the eLCDIF controller, "disabled" by default.
>
> Signed-off-by: Guido Günther <agx@sigxcpu.org>
> ---
> With some minimal support on imx8mq we might as well add it to the DT
>
>  arch/arm64/boot/dts/freescale/imx8mq.dtsi | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> index 7f9319452b58..00aa63bfd816 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> @@ -448,6 +448,23 @@
>                                 fsl,sdma-ram-script-name = "imx/sdma/sdma-imx7d.bin";
>                         };
>
> +                       lcdif: lcdif@30320000 {

Forgot to say that generic node names are preferred, so maybe:

lcdif: lcd-controller@30320000 {

instead?
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index 7f9319452b58..00aa63bfd816 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -448,6 +448,23 @@ 
 				fsl,sdma-ram-script-name = "imx/sdma/sdma-imx7d.bin";
 			};
 
+			lcdif: lcdif@30320000 {
+				compatible = "fsl,imx8mq-lcdif", "fsl,imx28-lcdif";
+				reg = <0x30320000 0x10000>;
+				interrupts = <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>;
+				clocks = <&clk IMX8MQ_CLK_LCDIF_PIXEL>;
+				clock-names = "pix";
+				assigned-clocks = <&clk IMX8MQ_VIDEO_PLL1_REF_SEL>,
+						  <&clk IMX8MQ_VIDEO_PLL1_BYPASS>,
+						  <&clk IMX8MQ_CLK_LCDIF_PIXEL>,
+						  <&clk IMX8MQ_VIDEO_PLL1>;
+				assigned-clock-parents = <&clk IMX8MQ_CLK_25M>,
+						  <&clk IMX8MQ_VIDEO_PLL1>,
+						  <&clk IMX8MQ_VIDEO_PLL1_OUT>;
+				assigned-clock-rates = <0>, <0>, <0>, <594000000>;
+				status = "disabled";
+			};
+
 			iomuxc: iomuxc@30330000 {
 				compatible = "fsl,imx8mq-iomuxc";
 				reg = <0x30330000 0x10000>;