mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] ARM: imx6: fix GK802 DT
@ 2015-07-07  7:33 Philipp Zabel
  2015-07-07  8:31 ` Uwe Kleine-König
  0 siblings, 1 reply; 2+ messages in thread
From: Philipp Zabel @ 2015-07-07  7:33 UTC (permalink / raw)
  To: barebox

From: Philipp Zabel <philipp.zabel@gmail.com>

Including "imx6q.dtsi" after <arm/imx6q-gk802.dts> also includes
<arm/imx6ql.dtsi> again, disabling all peripherals, including the
serial console uart. We can't include it first either, because
arm/imx6q-gk802.dts contains the /dts-v1/ header.
This patch drops the offending include statement and instead adds
the missing IPU aliases manually.

Signed-off-by: Philipp Zabel <philipp.zabel@gmail.com>
---
 arch/arm/dts/imx6q-gk802.dts | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/arch/arm/dts/imx6q-gk802.dts b/arch/arm/dts/imx6q-gk802.dts
index 310d6f0..d712e00 100644
--- a/arch/arm/dts/imx6q-gk802.dts
+++ b/arch/arm/dts/imx6q-gk802.dts
@@ -7,7 +7,13 @@
  */
 
 #include <arm/imx6q-gk802.dts>
-#include "imx6q.dtsi"
+
+/ {
+	aliases {
+		ipu0 = &ipu1;
+		ipu1 = &ipu2;
+	};
+};
 
 /* External USB-A port (USBOTG) */
 &usbotg {
-- 
2.2.1.209.g41e5f3a


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] ARM: imx6: fix GK802 DT
  2015-07-07  7:33 [PATCH] ARM: imx6: fix GK802 DT Philipp Zabel
@ 2015-07-07  8:31 ` Uwe Kleine-König
  0 siblings, 0 replies; 2+ messages in thread
From: Uwe Kleine-König @ 2015-07-07  8:31 UTC (permalink / raw)
  To: Philipp Zabel; +Cc: barebox

On Tue, Jul 07, 2015 at 09:33:20AM +0200, Philipp Zabel wrote:
> From: Philipp Zabel <philipp.zabel@gmail.com>
> 
> Including "imx6q.dtsi" after <arm/imx6q-gk802.dts> also includes
> <arm/imx6ql.dtsi> again, disabling all peripherals, including the
> serial console uart. We can't include it first either, because
> arm/imx6q-gk802.dts contains the /dts-v1/ header.
> This patch drops the offending include statement and instead adds
> the missing IPU aliases manually.
> 
> Signed-off-by: Philipp Zabel <philipp.zabel@gmail.com>
> ---
>  arch/arm/dts/imx6q-gk802.dts | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/dts/imx6q-gk802.dts b/arch/arm/dts/imx6q-gk802.dts
> index 310d6f0..d712e00 100644
> --- a/arch/arm/dts/imx6q-gk802.dts
> +++ b/arch/arm/dts/imx6q-gk802.dts
> @@ -7,7 +7,13 @@
>   */
>  
>  #include <arm/imx6q-gk802.dts>
> -#include "imx6q.dtsi"
> +
> +/ {
> +	aliases {
> +		ipu0 = &ipu1;
> +		ipu1 = &ipu2;
> +	};
> +};
>  
>  /* External USB-A port (USBOTG) */
>  &usbotg {
Alternatively you could remove the line

	#include <arm/imx6q.dtsi>

from arch/arm/dts/imx6q.dtsi, right? This is already included via

arch/arm/dts/imx6q-gk802.dts -> #include <arm/imx6q-gk802.dts>
dts/src/arm/imx6q-gk802.dts -> #include "imx6q.dtsi"

anyhow. This would need a fixup for all imx6q boards that don't have a
dts in dts/src though (i.e. all but imx6q-gk802 if my grep magic was
correct).

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-07-07  8:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-07  7:33 [PATCH] ARM: imx6: fix GK802 DT Philipp Zabel
2015-07-07  8:31 ` Uwe Kleine-König

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox