Hi Marek,

On Sat, 21 Jul 2012 17:02:23 +0200, Marek Vasut <[email protected]> wrote:

> Signed-off-by: Marek Vasut <[email protected]>
> Cc: Wolfgang Denk <[email protected]>
> Cc: Albert Aribaud <[email protected]>
> Cc: U-Boot DM <[email protected]>
> Cc: Tom Rini <[email protected]>
> ---
>  arch/arm/cpu/armv7/omap-common/Makefile
> |    3 ---
> drivers/gpio/Makefile
> |    1 + arch/arm/cpu/armv7/omap-common/gpio.c =>
> drivers/gpio/omap_gpio.c |    0
> include/configs/cm_t35.h
> |    1 +
> include/configs/devkit8000.h
> |    2 ++
> include/configs/dig297.h
> |    1 +
> include/configs/igep00x0.h
> |    1 +
> include/configs/mcx.h
> |    1 +
> include/configs/omap3_beagle.h
> |    1 +
> include/configs/omap3_evm_common.h
> |    1 +
> include/configs/omap3_logic.h
> |    1 +
> include/configs/omap3_mvblx.h
> |    1 +
> include/configs/omap3_overo.h
> |    1 +
> include/configs/omap3_pandora.h
> |    1 +
> include/configs/omap3_zoom2.h
> |    1 +
> include/configs/omap4_common.h
> |    1 +
> include/configs/omap5_evm.h
> |    1 +
> include/configs/tam3517-common.h
> |    1 + 18 files changed, 17 insertions(+), 3 deletions(-) rename
> arch/arm/cpu/armv7/omap-common/gpio.c => drivers/gpio/omap_gpio.c
> (100%)
> 
> diff --git a/arch/arm/cpu/armv7/omap-common/Makefile
> b/arch/arm/cpu/armv7/omap-common/Makefile index 2a6625f..d37b22d
> 100644 --- a/arch/arm/cpu/armv7/omap-common/Makefile
> +++ b/arch/arm/cpu/armv7/omap-common/Makefile
> @@ -29,9 +29,6 @@ SOBJS       := reset.o
>  
>  COBJS        := timer.o
>  COBJS        += utils.o
> -ifdef CONFIG_OMAP
> -COBJS        += gpio.o
> -endif
>  
>  ifneq ($(CONFIG_OMAP44XX)$(CONFIG_OMAP54XX),)
>  COBJS        += hwinit-common.o
> diff --git a/drivers/gpio/Makefile b/drivers/gpio/Makefile
> index 32a2474..8958bb4 100644
> --- a/drivers/gpio/Makefile
> +++ b/drivers/gpio/Makefile
> @@ -40,6 +40,7 @@ COBJS-$(CONFIG_TEGRA_GPIO)  += tegra_gpio.o
>  COBJS-$(CONFIG_DA8XX_GPIO)   += da8xx_gpio.o
>  COBJS-$(CONFIG_ALTERA_PIO)   += altera_pio.o
>  COBJS-$(CONFIG_MPC83XX_GPIO) += mpc83xx_gpio.o
> +COBJS-$(CONFIG_OMAP_GPIO)    += omap_gpio.o
>  
>  COBJS        := $(COBJS-y)
>  SRCS         := $(COBJS:.o=.c)
> diff --git a/arch/arm/cpu/armv7/omap-common/gpio.c
> b/drivers/gpio/omap_gpio.c similarity index 100%
> rename from arch/arm/cpu/armv7/omap-common/gpio.c
> rename to drivers/gpio/omap_gpio.c
> diff --git a/include/configs/cm_t35.h b/include/configs/cm_t35.h
> index ee4bce5..6eeb2d3 100644
> --- a/include/configs/cm_t35.h
> +++ b/include/configs/cm_t35.h
> @@ -37,6 +37,7 @@
>   */
>  #define CONFIG_OMAP  /* in a TI OMAP core */
>  #define CONFIG_OMAP34XX      /* which is a 34XX */
> +#define CONFIG_OMAP_GPIO
>  #define CONFIG_CM_T3X        /* working with CM-T35 and CM-T3730 */
>  
>  #define CONFIG_SYS_TEXT_BASE 0x80008000
> diff --git a/include/configs/devkit8000.h
> b/include/configs/devkit8000.h index 037a5bb..e529099 100644
> --- a/include/configs/devkit8000.h
> +++ b/include/configs/devkit8000.h
> @@ -36,6 +36,8 @@
>  #define CONFIG_OMAP34XX              1       /* which is a 34XX */
>  #define CONFIG_OMAP3_DEVKIT8000      1       /* working with
> DevKit8000 */ #define CONFIG_MACH_TYPE        MACH_TYPE_DEVKIT8000
> +#define CONFIG_OMAP_GPIO
> +
>  /*
>   * 1MB into the SDRAM to allow for SPL's bss at the beginning of
> SDRAM
>   * 64 bytes before this address should be set aside for u-boot.img's
> diff --git a/include/configs/dig297.h b/include/configs/dig297.h
> index 4845d51..f123675 100644
> --- a/include/configs/dig297.h
> +++ b/include/configs/dig297.h
> @@ -45,6 +45,7 @@
>   */
>  #define CONFIG_OMAP          /* in a TI OMAP core */
>  #define CONFIG_OMAP34XX              /* which is a 34XX */
> +#define CONFIG_OMAP_GPIO
>  
>  #define CONFIG_SYS_TEXT_BASE 0x80008000
>  
> diff --git a/include/configs/igep00x0.h b/include/configs/igep00x0.h
> index 88e2e3a..727b800 100644
> --- a/include/configs/igep00x0.h
> +++ b/include/configs/igep00x0.h
> @@ -30,6 +30,7 @@
>   */
>  #define CONFIG_OMAP          1       /* in a TI OMAP core */
>  #define CONFIG_OMAP34XX              1       /* which is a 34XX */
> +#define CONFIG_OMAP_GPIO
>  
>  #define CONFIG_SDRC  /* The chip has SDRC controller */
>  
> diff --git a/include/configs/mcx.h b/include/configs/mcx.h
> index 970c882..0b75460 100644
> --- a/include/configs/mcx.h
> +++ b/include/configs/mcx.h
> @@ -27,6 +27,7 @@
>  #define CONFIG_OMAP                  /* in a TI OMAP core */
>  #define CONFIG_OMAP34XX                      /* which is a 34XX */
>  #define CONFIG_OMAP3_MCX             /* working with mcx */
> +#define CONFIG_OMAP_GPIO
>  
>  #define MACH_TYPE_MCX                        3656
>  #define CONFIG_MACH_TYPE     MACH_TYPE_MCX
> diff --git a/include/configs/omap3_beagle.h
> b/include/configs/omap3_beagle.h index 657780e..382b791 100644
> --- a/include/configs/omap3_beagle.h
> +++ b/include/configs/omap3_beagle.h
> @@ -34,6 +34,7 @@
>  #define CONFIG_OMAP          1       /* in a TI OMAP core */
>  #define CONFIG_OMAP34XX              1       /* which is a 34XX */
>  #define CONFIG_OMAP3_BEAGLE  1       /* working with BEAGLE */
> +#define CONFIG_OMAP_GPIO
>  
>  #define CONFIG_SDRC  /* The chip has SDRC controller */
>  
> diff --git a/include/configs/omap3_evm_common.h
> b/include/configs/omap3_evm_common.h index 20192a9..b12ce2e 100644
> --- a/include/configs/omap3_evm_common.h
> +++ b/include/configs/omap3_evm_common.h
> @@ -22,6 +22,7 @@
>   */
>  #define CONFIG_OMAP                  /* This is TI OMAP core */
>  #define CONFIG_OMAP34XX                      /* belonging to 34XX
> family */ +#define CONFIG_OMAP_GPIO
>  
>  #define CONFIG_SDRC                  /* The chip has SDRC
> controller */ 
> diff --git a/include/configs/omap3_logic.h
> b/include/configs/omap3_logic.h index 2e1e6b9..6d4faed 100644
> --- a/include/configs/omap3_logic.h
> +++ b/include/configs/omap3_logic.h
> @@ -33,6 +33,7 @@
>  #define CONFIG_OMAP                  /* in a TI OMAP core */
>  #define CONFIG_OMAP34XX                      /* which is a 34XX */
>  #define CONFIG_OMAP3_LOGIC           /* working with Logic OMAP
> boards */ +#define CONFIG_OMAP_GPIO
>  
>  #undef CONFIG_USE_IRQ                        /* no support for IRQs
> */ 
> diff --git a/include/configs/omap3_mvblx.h
> b/include/configs/omap3_mvblx.h index 6a13046..30ca312 100644
> --- a/include/configs/omap3_mvblx.h
> +++ b/include/configs/omap3_mvblx.h
> @@ -39,6 +39,7 @@
>  #define CONFIG_OMAP34XX              1       /* which is a 34XX */
>  #define CONFIG_MVBLX         1       /* working with
> mvBlueLYNX-X */ #define CONFIG_MACH_TYPE      MACH_TYPE_MVBLX
> +#define CONFIG_OMAP_GPIO
>  
>  #define CONFIG_SDRC  /* The chip has SDRC controller */
>  
> diff --git a/include/configs/omap3_overo.h
> b/include/configs/omap3_overo.h index d29b326..f6afb7a 100644
> --- a/include/configs/omap3_overo.h
> +++ b/include/configs/omap3_overo.h
> @@ -25,6 +25,7 @@
>  #define CONFIG_OMAP                          /* in a TI OMAP
> core */ #define CONFIG_OMAP34XX                               /*
> which is a 34XX */ #define
> CONFIG_OMAP3_OVERO                    /* working with overo */
> +#define CONFIG_OMAP_GPIO 
>  #define CONFIG_SDRC                          /* The chip has
> SDRC controller */ 
> diff --git a/include/configs/omap3_pandora.h
> b/include/configs/omap3_pandora.h index 604b53d..28f8a3e 100644
> --- a/include/configs/omap3_pandora.h
> +++ b/include/configs/omap3_pandora.h
> @@ -29,6 +29,7 @@
>  #define CONFIG_OMAP          1       /* in a TI OMAP core */
>  #define CONFIG_OMAP34XX              1       /* which is a 34XX */
>  #define CONFIG_OMAP3_PANDORA 1       /* working with pandora
> */ +#define CONFIG_OMAP_GPIO
>  
>  #define CONFIG_SDRC  /* The chip has SDRC controller */
>  
> diff --git a/include/configs/omap3_zoom2.h
> b/include/configs/omap3_zoom2.h index b60ece3..98b93f8 100644
> --- a/include/configs/omap3_zoom2.h
> +++ b/include/configs/omap3_zoom2.h
> @@ -36,6 +36,7 @@
>  #define CONFIG_OMAP          1       /* in a TI OMAP core */
>  #define CONFIG_OMAP34XX              1       /* which is a 34XX */
>  #define CONFIG_OMAP3_ZOOM2   1       /* working with Zoom II */
> +#define CONFIG_OMAP_GPIO
>  
>  #define CONFIG_SDRC  /* The chip has SDRC controller */
>  
> diff --git a/include/configs/omap4_common.h
> b/include/configs/omap4_common.h index 2192c2b..bc73f21 100644
> --- a/include/configs/omap4_common.h
> +++ b/include/configs/omap4_common.h
> @@ -35,6 +35,7 @@
>  #define CONFIG_OMAP          1       /* in a TI OMAP core */
>  #define CONFIG_OMAP44XX              1       /* which is a 44XX */
>  #define CONFIG_OMAP4430              1       /* which is in a
> 4430 */ +#define CONFIG_OMAP_GPIO
>  
>  /* Get CPU defs */
>  #include <asm/arch/cpu.h>
> diff --git a/include/configs/omap5_evm.h b/include/configs/omap5_evm.h
> index c5874bb..0884280 100644
> --- a/include/configs/omap5_evm.h
> +++ b/include/configs/omap5_evm.h
> @@ -38,6 +38,7 @@
>  #define CONFIG_OMAP54XX      /* which is a 54XX */
>  #define CONFIG_OMAP5430      /* which is in a 5430 */
>  #define CONFIG_5430EVM       /* working with EVM */
> +#define CONFIG_OMAP_GPIO
>  
>  /* Get CPU defs */
>  #include <asm/arch/cpu.h>
> diff --git a/include/configs/tam3517-common.h
> b/include/configs/tam3517-common.h index 777f77c..94006e7 100644
> --- a/include/configs/tam3517-common.h
> +++ b/include/configs/tam3517-common.h
> @@ -27,6 +27,7 @@
>   */
>  #define CONFIG_OMAP          /* in a TI OMAP core */
>  #define CONFIG_OMAP34XX              /* which is a 34XX */
> +#define CONFIG_OMAP_GPIO
>  
>  #define CONFIG_SYS_TEXT_BASE 0x80008000
>  

When applied on top of u-boot-arm/master, this patch causes a raise
in build failures. With master:

--------------------- SUMMARY ----------------------------
Boards compiled: 315
Boards with warnings but no errors: 2 ( VCMA9 smdk2410 )
----------------------------------------------------------

With master + this patch:

--------------------- SUMMARY ----------------------------
Boards compiled: 315
Boards with errors: 23 ( mcx igep0030_nand omap3_evm_quick_nand
omap3_evm omap3_pandora omap3_overo omap5_evm omap4_panda mt_ventoux
igep0020_nand omap4_sdp4430 twister omap3_zoom2 igep0030 igep0020
omap3_evm_quick_mmc cm_t35 omap3_beagle am335x_evm dig297 omap3_logic
omap3_mvblx devkit8000 )
Boards with warnings but no errors: 2 ( VCMA9 smdk2410 )
----------------------------------------------------------

Typical diagnostics:

/home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:
In function
`gpio_set_value': /home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:146:
multiple definition of
`gpio_set_value' 
/home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:/home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:146:
first defined
here /home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:
In function
`gpio_get_value': /home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:161:
multiple definition of
`gpio_get_value' 
/home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:/home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:161:
first defined
here /home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:
In function
`gpio_direction_input': 
/home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:195:
multiple definition of
`gpio_direction_input' 
/home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:/home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:195:
first defined
here /home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:
In function
`gpio_direction_output': 
/home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:211:
multiple definition of
`gpio_direction_output' 
/home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:/home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:211:
first defined
here /home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:
In function
`gpio_request': /home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:230:
multiple definition of
`gpio_request' 
/home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:/home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:230:
first defined
here /home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:
In function
`gpio_free': /home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:243:
multiple definition of
`gpio_free' 
/home/albert/src/u-boot-arm/build/omap4_panda/drivers/gpio/omap_gpio.o:/home/albert/src/u-boot-arm/drivers/gpio/omap_gpio.c:243:
first defined here

Am I missing some dependency of this patch?

Amicalement,
-- 
Albert.
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to