Re: [PATCH] ARM: DT: ifc6410: Add board memory range.

[Srinivas Kandagatla <srinivas.kandagatla@linaro.org>]

On 17/09/14 20:18, Josh Cartwright wrote:
> 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@smartplayin.com> 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@linaro.org>
>>>> ---
>>>> 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?
No, its not fixing any thing by itself.

However along with "ARM: zImage: Allow DTB to override broken ATAG_MEM" 
patch atleast I get to boot my IFC6410 with multi_v7_defconfig.

>