On Wed, Sep 17, 2014 at 12:03:37PM -0700, Kumar Gala wrote: > On Sep 16, 2014, at 11:34 PM, Pramod Gurav <pramod.gurav@xxxxxxxxxxxxxxx> wrote: > > On Wednesday 17 September 2014 11:06 AM, Srinivas Kandagatla wrote: > >> 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 <srinivas.kandagatla@xxxxxxxxxx> > >> --- > >> 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 {? > >> + device_type = "memory"; > >> + reg = <0x80000000 0x80000000>; > >> + }; > >> + > >> soc { > >> gsbi@16600000 { > >> status = "ok?; > > Hmm, this doesn?t seem to work for me. I don't see how it could with qcom_defconfig, as we set CONFIG_ARM_ATAG_DTB_COMPAT. Is this patch fixing a real issue? -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html