[U-Boot] [PATCH V3 4/4] trats: fdt: disable unused DW MMC

[Przemyslaw Marczak]

Hello Jaehoon,

On 10/01/2015 05:37 AM, Jaehoon Chung wrote:
> Hi, Przemyslaw.
>
> On 09/30/2015 08:14 PM, Przemyslaw Marczak wrote:
>> This device uses SDHCI driver, for eMMC and SD cards.
>> Trying bind the DW MMC driver with fdt node without all
>> required properties, causes printing an error.
>>
>> This commit disables the DW MMC node.
>
> Why does it need?
> Trats board doesn't support the Designware IP, so i think right that it shouldn't build.
>
> If needs to modify, exynos-common.h should be modified.
>

I think, that some day, we will have a single config, for at least 
exynos5 and exynos4 (if doesn't exceed the size limit), so using the 
generic configuration is reasonable here.

Trats is based on Exynos4210, which supports this IP, and I checked the 
documentation, the address 0x12550000 is proper - Mobile Storage Host.

For a long time it wasn't enable on this device, and only printed an 
error, that 'bus-width' not found. I tried to enable this, but it 
doesn't work for the same settings as for Trats2. Now I don't have time 
to debug why, so it can be disabled.

>>
>> Tested-on: Trats
>>
>> Signed-off-by: Przemyslaw Marczak <p.marczak at samsung.com>
>> Cc: Łukasz Majewski <l.majewski at samsung.com>
>> Cc: Minkyu Kang <mk7.kang at samsung.com>
>> --
>> Changes V3:
>> - new commit
>> ---
>>   arch/arm/dts/exynos4210-trats.dts | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm/dts/exynos4210-trats.dts b/arch/arm/dts/exynos4210-trats.dts
>> index 36d02df..f3fac80 100644
>> --- a/arch/arm/dts/exynos4210-trats.dts
>> +++ b/arch/arm/dts/exynos4210-trats.dts
>> @@ -117,4 +117,8 @@
>>   	sdhci at 12540000 {
>>   		status = "disabled";
>>   	};
>> +
>> +	dwmmc at 12550000 {
>> +		status = "disabled";
>> +	};
>
> It seems to support dwmmc controller. 12550000 addr is for sdhci controller.

Please check manual for E4210, I'm sure it's right.

>
> Best Regards,
> Jaehoon Chung
>
>>   };
>>
>
>

Best regards,
-- 
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com