Hi,
On 10 December 2014 at 00:50, Masahiro Yamada yamada.m@jp.panasonic.com wrote:
On Wed, 10 Dec 2014 11:47:03 +0530 Ruchika Gupta ruchika.gupta@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@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