[PATCH v2] ARM: dts: disable MDMA1 node for Exynos5420

[PATCH v2] ARM: dts: disable MDMA1 node for Exynos5420

[Seungwon Jeon]

This change places MDMA1 in disabled node for Exynos5420.
If MDMA1 region is configured with secure mode, it makes
the boot failure with the following on smdk5420 board.
("Unhandled fault: imprecise external abort (0x1406) at 0x00000000")
Thus, arndale-octa board don't need to do the same thing anymore.

Signed-off-by: Seungwon Jeon <tgih.jun@samsung.com>
---
Note: This patch comes from "[PATCH] ARM: dts: disable MDMA1 node for smdk5420 board"

 arch/arm/boot/dts/exynos5420-arndale-octa.dts |   12 ------------
 arch/arm/boot/dts/exynos5420.dtsi             |    7 +++++++
 2 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5420-arndale-octa.dts b/arch/arm/boot/dts/exynos5420-arndale-octa.dts
index 80a3bf4..896a2a6 100644
--- a/arch/arm/boot/dts/exynos5420-arndale-octa.dts
+++ b/arch/arm/boot/dts/exynos5420-arndale-octa.dts
@@ -364,16 +364,4 @@
 			gpio-key,wakeup;
 		};
 	};
-
-	amba {
-		mdma1: mdma@11C10000 {
-			/*
-			 * MDMA1 can support both secure and non-secure
-			 * AXI transactions. When this is enabled in the kernel
-			 * for boards that run in secure mode, we are getting
-			 * imprecise external aborts causing the kernel to oops.
-			 */
-			status = "disabled";
-		};
-	};
 };
diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
index c3a9a66..97cbcdd 100644
--- a/arch/arm/boot/dts/exynos5420.dtsi
+++ b/arch/arm/boot/dts/exynos5420.dtsi
@@ -336,6 +336,13 @@
 			#dma-cells = <1>;
 			#dma-channels = <8>;
 			#dma-requests = <1>;
+			/*
+			 * MDMA1 can support both secure and non-secure
+			 * AXI transactions. When this is enabled in the kernel
+			 * for boards that run in secure mode, we are getting
+			 * imprecise external aborts causing the kernel to oops.
+			 */
+			status = "disabled";
 		};
 	};
 
-- 
1.7.0.4

Re: [PATCH v2] ARM: dts: disable MDMA1 node for Exynos5420

[Javi Merino]

On Tue, Apr 22, 2014 at 02:05:26PM +0100, Seungwon Jeon wrote:
> This change places MDMA1 in disabled node for Exynos5420.
> If MDMA1 region is configured with secure mode, it makes
> the boot failure with the following on smdk5420 board.
> ("Unhandled fault: imprecise external abort (0x1406) at 0x00000000")
> Thus, arndale-octa board don't need to do the same thing anymore.
> 
> Signed-off-by: Seungwon Jeon <tgih.jun@samsung.com>

I've had some problem applying it (wrong EOL?).  Not sure if it's my
email client.  In any case, I've manually fixed the rejects and tested
it in my Arndale Octa.  It works for me so

Tested-by: Javi Merino <javi.merino@arm.com>

RE: [PATCH v2] ARM: dts: disable MDMA1 node for Exynos5420

[Kukjin Kim]

Javi Merino wrote:
> 
> On Tue, Apr 22, 2014 at 02:05:26PM +0100, Seungwon Jeon wrote:
> > This change places MDMA1 in disabled node for Exynos5420.
> > If MDMA1 region is configured with secure mode, it makes
> > the boot failure with the following on smdk5420 board.
> > ("Unhandled fault: imprecise external abort (0x1406) at 0x00000000")
> > Thus, arndale-octa board don't need to do the same thing anymore.
> >
> > Signed-off-by: Seungwon Jeon <tgih.jun@samsung.com>
> 
> I've had some problem applying it (wrong EOL?).  Not sure if it's my

Probably, it is due to Seungwon's e-mail client ;-)

> email client.  In any case, I've manually fixed the rejects and tested
> it in my Arndale Octa.  It works for me so
> 
> Tested-by: Javi Merino <javi.merino@arm.com>

Anyway, thanks for your test.

I will apply but I'm not sure this is critical bug or not for smdk5420...

Thanks,
Kukjin