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

Heiko Schocher hs at denx.de
Mon Nov 16 12:00:08 CET 2015


Hello Valentin,

Am 13.11.2015 um 15:59 schrieb Valentin Longchamp:
> From: Holger Brunck <holger.brunck at 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 at keymile.com>
> Signed-off-by: Valentin Longchamp <valentin.longchamp at 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 at 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


More information about the U-Boot mailing list