diff mbox series

powerpc/mpc85xx: Fix compiling error with DM_GPIO

Message ID 1588138413-18330-1-git-send-email-madalin.bucur@oss.nxp.com
State Accepted
Commit 9e998d5dc18ed75175cf72b5eeaaf432ae0da0a4
Headers show
Series powerpc/mpc85xx: Fix compiling error with DM_GPIO | expand

Commit Message

Madalin Bucur (OSS) April 29, 2020, 5:33 a.m. UTC
P4080DS compilation fails with DM_GPIO enabled:

drivers/gpio/gpio-uclass.c:355:5: error: redefinition of 'gpio_request'
 int gpio_request(unsigned gpio, const char *label)
     ^~~~~~~~~~~~
In file included from ./arch/powerpc/include/asm/arch/gpio.h:17:0,
                 from ./arch/powerpc/include/asm/gpio.h:1,
                 from drivers/gpio/gpio-uclass.c:16:
./arch/powerpc/include/asm/mpc85xx_gpio.h:68:19: note: previous definition of 'gpio_request' was here
 static inline int gpio_request(unsigned gpio, const char *label)
                   ^~~~~~~~~~~~

Signed-off-by: Madalin Bucur <madalin.bucur at oss.nxp.com>
---
 arch/powerpc/include/asm/mpc85xx_gpio.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Priyanka Jain (OSS) June 8, 2020, 7:53 a.m. UTC | #1
>-----Original Message-----
>From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Madalin Bucur
>Sent: Wednesday, April 29, 2020 11:04 AM
>To: wd at denx.de; u-boot at lists.denx.de
>Cc: Madalin Bucur (OSS) <madalin.bucur at oss.nxp.com>
>Subject: [PATCH] powerpc/mpc85xx: Fix compiling error with DM_GPIO
>
>P4080DS compilation fails with DM_GPIO enabled:
>
>drivers/gpio/gpio-uclass.c:355:5: error: redefinition of 'gpio_request'
> int gpio_request(unsigned gpio, const char *label)
>     ^~~~~~~~~~~~
>In file included from ./arch/powerpc/include/asm/arch/gpio.h:17:0,
>                 from ./arch/powerpc/include/asm/gpio.h:1,
>                 from drivers/gpio/gpio-uclass.c:16:
>./arch/powerpc/include/asm/mpc85xx_gpio.h:68:19: note: previous
>definition of 'gpio_request' was here  static inline int gpio_request(unsigned
>gpio, const char *label)
>                   ^~~~~~~~~~~~
>
>Signed-off-by: Madalin Bucur <madalin.bucur at oss.nxp.com>
>---
<snip>
Patch applied on mpc85xx. Awaiting upstream

Thanks
Priyanka
diff mbox series

Patch

diff --git a/arch/powerpc/include/asm/mpc85xx_gpio.h b/arch/powerpc/include/asm/mpc85xx_gpio.h
index 1b332f0..feebe15 100644
--- a/arch/powerpc/include/asm/mpc85xx_gpio.h
+++ b/arch/powerpc/include/asm/mpc85xx_gpio.h
@@ -6,6 +6,7 @@ 
 #ifndef POWERPC_ASM_MPC85XX_GPIO_H_
 #define POWERPC_ASM_MPC85XX_GPIO_H_
 
+#ifndef CONFIG_DM_GPIO
 # include <asm/immap_85xx.h>
 
 /*
@@ -110,5 +111,5 @@  static inline int gpio_is_valid(int gpio)
 {
 	return (gpio >= 0) && (gpio < 32);
 }
-
+#endif /* not CONFIG_DM_GPIO */
 #endif /* not POWERPC_ASM_MPC85XX_GPIO_H_ */