[U-Boot] [PATCH 1/2] arm: ls102xa: Add dummy gpio.h to enable CONFIG_OF_CONTROL

[Simon Glass]

Hi,

On 10 December 2014 at 00:50, Masahiro Yamada <yamada.m at jp.panasonic.com> wrote:
>
> On Wed, 10 Dec 2014 11:47:03 +0530
> Ruchika Gupta <ruchika.gupta at freescale.com> wrote:
>
>> If CONFIG_OF_CONTROL is enabled, lib/fdtdec.c is compiled.
>> It includes <asm/gpio.h> and then <asm/gpio.h> includes
>> <asm/arch/gpio.h>.  As a result, all the SoCs that enable
>> CONFIG_OF_CONTROL must have <asm/arch/gpio.h> even if they
>> do not support GPIO.
>>
>> The right fix would be to split the lib/fdtdec.c to remove
>> dependency on GPIO.This commit adds a dummy <asm/arch/gpio.h>
>> to support OF_CONTROL for LS102x platform. This dummy header
>> will be removed after FDT-GPIO stuff is fixed correctly.
>>
>> Signed-off-by: Ruchika Gupta <ruchika.gupta at freescale.com>
>> ---
>
> Oh, you too.
>
> (I assume you saw my commit 630bf80ebb34.)

This was discussed here:

http://patchwork.ozlabs.org/patch/415036/

These GPIO functions in fdtdec.c are currently only used on Tegra and
Exynos. They are not correctly implemented and we really need to
remove them in favour of proper ones in the uclass. Both of these
platforms fully support driver model so this should not be hard.

However this is the third case of hitting this problem. Let's say that
if we don't manage to remove this code by the the end of this year
then we can go ahead and split this out as Masahiro and you suggest.
It seems hard to remove old code sometimes.

[snip]

Regards,
Simon