Commit 634a6037 authored May 26, 2015 by Lucas Stach Committed by Shawn Guo May 27, 2015

ARM: imx6: allow booting with old DT

The GPC rewrite to IRQ domains has been on the premise that it may break
suspend/resume for new kernels on old DT, but otherwise keep things working
from a user perspective. This was an accepted compromise to be able to move
the GIC cleanup forward.

What actually happened was that booting a new kernel on an old DT crashes
before even the console is up, so the user does not even see the warning
that the DT is too old. The warning message suggests that this has been
known before, which is clearly unacceptable.

Fix the early crash by mapping the GPC memory space if the IRQ controller
doesn't claim it. This keeps at least CPUidle and the needed CPU wakeup
workarounds working. With this fixed the system is able to boot up
properly minus the expected suspend/resume breakage.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>

parent e46b5a64

arch/arm/mach-imx/gpc.c

+9 −3

Original line number	Diff line number	Diff line
		@@ -280,9 +280,15 @@ void __init imx_gpc_check_dt(void)
		struct device_node *np;

		np = of_find_compatible_node(NULL, NULL, "fsl,imx6q-gpc");
		if (WARN_ON(!np \|\|
		!of_find_property(np, "interrupt-controller", NULL)))
		pr_warn("Outdated DT detected, system is about to crash!!!\n");
		if (WARN_ON(!np))
		return;

		if (WARN_ON(!of_find_property(np, "interrupt-controller", NULL))) {
		pr_warn("Outdated DT detected, suspend/resume will NOT work\n");

		/* map GPC, so that at least CPUidle and WARs keep working */
		gpc_base = of_iomap(np, 0);
		}
		}

		#ifdef CONFIG_PM_GENERIC_DOMAINS

Admin message