On Sep 16, 2014, at 11:34 PM, Pramod Gurav [off-list ref] wrote:

Hi Srini,

Thanks for the patch.
Tested-by: Pramod Gurav <redacted>

A minor nit.
On Wednesday 17 September 2014 11:06 AM, Srinivas Kandagatla wrote:

quoted

This patch adds memory details of IFC6410 as this is necessary to
fix the in-correct memory start comming from ATAGs.

Signed-off-by: Srinivas Kandagatla <redacted>
---
arch/arm/boot/dts/qcom-apq8064-ifc6410.dts | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
index 90db8af..3287f12 100644
--- a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
+++ b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts

@@ -4,6 +4,11 @@

	model = "Qualcomm APQ8064/IFC6410";
	compatible = "qcom,apq8064-ifc6410", "qcom,apq8064";

+	memory{

Space before {?

quoted

+		device_type = "memory";
+		reg = <0x80000000 0x80000000>;
+	};
+
	soc {
		gsbi at 16600000 {
			status = "ok?;

Hmm, this doesn?t seem to work for me.

- k

-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation