arm: ls102xa: Add dummy gpio.h to enable CONFIG_OF_CONTROL

PATCH New Ruchika Gupta 4 participants 4 messages Dec 10, 2014 – Jan 24, 2015 Arm (32-bit)

Share & tools

Ruchika Gupta

author

2014-12-10 06:17 UTC #0 lore raw

PATCH 1/2 arm: ls102xa: Add dummy gpio.h to enable CONFIG_OF_CONTROL

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 <[email protected]>

---
 arch/arm/include/asm/arch-ls102xa/gpio.h | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
 create mode 100644 arch/arm/include/asm/arch-ls102xa/gpio.h

arch/arm/include/asm/arch-ls102xa/gpio.h new file +15 -0

new file mode 100644
index 0000000..b704436
@@ -0,0 +1,15 @@
+/*
+ * SPDX-License-Identifier:	GPL-2.0+
+ */
+
+/*
+ * Dummy header file to enable CONFIG_OF_CONTROL.
+ * If CONFIG_OF_CONTROL is enabled, lib/fdtdec.c is compiled.
+ * It includes <asm/arch/gpio.h> via <asm/gpio.h>, so those SoCs that enable
+ * OF_CONTROL must have arch/gpio.h.
+ */
+
+#ifndef __ASM_ARCH_LS102XA_GPIO_H_
+#define __ASM_ARCH_LS102XA_GPIO_H_
+
+#endif

Signature · 1 line

1.8.1.4

Masahiro Yamada

2014-12-10 07:50 UTC #1 lore raw

PATCH 1/2 arm: ls102xa: Add dummy gpio.h to enable CONFIG_OF_CONTROL

Quoted 6 lines · On Wed, 10 Dec 2014 11:47:03 +0530 Ruchika Gupta <[email protected]>…

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.

Quoted 7 lines

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 <[email protected]>
---

Oh, you too.

(I assume you saw my commit 630bf80ebb34.)

When I try to do something on my board,
I pretty often encounter awful code.

To fix that crap, I have to fix other crap first, which requires me to fix other crap in advance... My plate gets filled with a bunch of issues soon.

I am getting sick of this project.

Best Regards
Masahiro Yamada

Simon Glass

2014-12-15 14:40 UTC #2 lore raw

PATCH 1/2 arm: ls102xa: Add dummy gpio.h to enable CONFIG_OF_CONTROL

Hi,

Quoted 15 lines · Masahiro Yamada

Quoted 11 lines · On Wed, 10 Dec 2014 11:47:03 +0530 Ruchika Gupta <[email protected]>…
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.

Quoted 6 lines

Quoted 2 lines
Signed-off-by Ruchika Gupta <[email protected]>
---
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

York Sun

2015-01-24 15:01 UTC #3 lore raw

PATCH 1/2 arm: ls102xa: Add dummy gpio.h to enable CONFIG_OF_CONTROL

In reply to Ruchika Gupta

Quoted 6 lines · Ruchika Gupta

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.

Quoted 7 lines

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 <[email protected]>
---

Applied to u-boot-fsl-qoriq master branch, awaiting upstream.

York