From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Mon, 16 Nov 2015 12:00:08 +0100
Subject: [U-Boot] [PATCH 07/17] km/powerpc: remove unneeded defines
In-Reply-To: <1447426768-23226-8-git-send-email-valentin.longchamp@keymile.com>
References: <1447426768-23226-1-git-send-email-valentin.longchamp@keymile.com>
 <1447426768-23226-8-git-send-email-valentin.longchamp@keymile.com>
Message-ID: <5649B738.1000805@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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