[U-Boot] [PATCH 07/17] km/powerpc: remove unneeded defines

Heiko Schocher
2015-11-16T11:00:08Z

Hello Valentin,

Am 13.11.2015 um 15:59 schrieb Valentin Longchamp:
> From: Holger Brunck <holger.brunck@keymile.com>
>
> We use CONFIG_OF_LIBFDT and CONFIG_OF_BOARD_SETUP on all our powerpc
> targets, so there is no need to check these defines within our C code.
>
> Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
> Signed-off-by: Valentin Longchamp <valentin.longchamp@keymile.com>
> Conflicts:
> board/keymile/km82xx/km82xx.c
> board/keymile/km83xx/km83xx.c
>
> ---
>
> board/keymile/km82xx/km82xx.c | 8 ++------
> board/keymile/km83xx/km83xx.c | 4 +---
> 2 files changed, 3 insertions(+), 9 deletions(-)

Reviewed-by: Heiko Schocher <hs@denx.de>

bye,
Heiko
>
> diff --git a/board/keymile/km82xx/km82xx.c b/board/keymile/km82xx/km82xx.c
> index cc9a7c9..70f816b 100644
> --- a/board/keymile/km82xx/km82xx.c
> +++ b/board/keymile/km82xx/km82xx.c
> @@ -11,10 +11,7 @@
> #include <malloc.h>
> #include <asm/io.h>
>
> -#if defined(CONFIG_OF_BOARD_SETUP) && defined(CONFIG_OF_LIBFDT)
> #include <libfdt.h>
> -#endif
> -
> #include <i2c.h>
> #include "../common/common.h"
>
> @@ -465,14 +462,13 @@ static void setports(int gpio)
> }
> }
> #endif
> -#if defined(CONFIG_OF_BOARD_SETUP) && defined(CONFIG_OF_LIBFDT)
> -int ft_board_setup(void *blob, bd_t *bd)
> +
> +void ft_board_setup(void *blob, bd_t *bd)
> {
> ft_cpu_setup(blob, bd);
>
> return 0;
> }
> -#endif /* defined(CONFIG_OF_BOARD_SETUP) && defined(CONFIG_OF_LIBFDT) */
>
> #if defined(CONFIG_MGCOGE3NE)
> int get_testpin(void)
> diff --git a/board/keymile/km83xx/km83xx.c b/board/keymile/km83xx/km83xx.c
> index 4ebcdbd..a7f2bf7 100644
> --- a/board/keymile/km83xx/km83xx.c
> +++ b/board/keymile/km83xx/km83xx.c
> @@ -369,14 +369,12 @@ int checkboard(void)
> return 0;
> }
>
> -#if defined(CONFIG_OF_BOARD_SETUP)
> -int ft_board_setup(void *blob, bd_t *bd)
> +void ft_board_setup(void *blob, bd_t *bd)
> {
> ft_cpu_setup(blob, bd);
>
> return 0;
> }
> -#endif
>
> #if defined(CONFIG_HUSH_INIT_VAR)
> int hush_init_var(void)
>

--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany