diff mbox

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

Message ID 000a01cf5e2b$87601760$96204620$%jun@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Seungwon Jeon April 22, 2014, 1:05 p.m. UTC
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(-)

Comments

Javi Merino April 22, 2014, 1:45 p.m. UTC | #1
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>

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Kim Kukjin April 22, 2014, 10:41 p.m. UTC | #2
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

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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";
 		};
 	};