On Wed, Mar 27, 2013 at 06:30:45PM +0100, Philipp Zabel wrote:
> From: Steffen Trumtrar <s.trumt...@pengutronix.de>
> 
> Add ldb device tree node and clock lookups.
> 
> Signed-off-by: Steffen Trumtrar <s.trumt...@pengutronix.de>
> Signed-off-by: Philipp Zabel <p.za...@pengutronix.de>
> ---
>  arch/arm/boot/dts/imx6q.dtsi   | 17 +++++++++++++++++
>  arch/arm/boot/dts/imx6qdl.dtsi | 26 ++++++++++++++++++++++++++
>  2 files changed, 43 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx6q.dtsi b/arch/arm/boot/dts/imx6q.dtsi
> index cba021e..1a30227 100644
> --- a/arch/arm/boot/dts/imx6q.dtsi
> +++ b/arch/arm/boot/dts/imx6q.dtsi
> @@ -294,3 +294,20 @@
>               };
>       };
>  };
> +
> +&ldb {
> +     clocks = <&clks 33>, <&clks 34>,
> +              <&clks 39>, <&clks 40>, <&clks 41>, <&clks 42>,
> +              <&clks 135>, <&clks 136>;
> +     clock-names = "di0_pll", "di1_pll",
> +                   "di0_sel", "di1_sel", "di2_sel", "di3_sel",
> +                   "di0", "di1";

These are identical with the ones in imx6qdl.dtsi, so not needed at all?

> +
> +     lvds-channel@0 {
> +             crtcs = <&ipu1 0>, <&ipu1 1>, <&ipu2 0>, <&ipu2 1>;
> +     };
> +
> +     lvds-channel@1 {
> +             crtcs = <&ipu1 0>, <&ipu1 1>, <&ipu2 0>, <&ipu2 1>;
> +     };
> +};
> diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
> index 06ec460..dd5ef96 100644
> --- a/arch/arm/boot/dts/imx6qdl.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl.dtsi
> @@ -529,6 +529,32 @@
>                               reg = <0x020e0000 0x38>;
>                       };
>  
> +                     ldb: ldb@020e0008 {
> +                             #address-cells = <1>;
> +                             #size-cells = <0>;
> +                             compatible = "fsl,imx6q-ldb", "fsl,imx53-ldb";

Since both compatible strings are in the driver matching table, it's not
necessary to have "fsl,imx53-ldb" listed here.

Shawn

> +                             clocks = <&clks 33>, <&clks 34>,
> +                                      <&clks 39>, <&clks 40>,
> +                                      <&clks 135>, <&clks 136>;
> +                             clock-names = "di0_pll", "di1_pll",
> +                                           "di0_sel", "di1_sel",
> +                                           "di0", "di1";
> +                             gpr = <&gpr>;
> +                             status = "disabled";
> +
> +                             lvds-channel@0 {
> +                                     reg = <0>;
> +                                     crtcs = <&ipu1 0>;
> +                                     status = "disabled";
> +                             };
> +
> +                             lvds-channel@1 {
> +                                     reg = <1>;
> +                                     crtcs = <&ipu1 1>;
> +                                     status = "disabled";
> +                             };
> +                     };
> +
>                       dcic1: dcic@020e4000 {
>                               reg = <0x020e4000 0x4000>;
>                               interrupts = <0 124 0x04>;
> -- 
> 1.8.2.rc2
> 

_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss

Reply via email to