From mboxrd@z Thu Jan  1 00:00:00 1970
From: york sun <york.sun@nxp.com>
Date: Mon, 3 Apr 2017 15:56:30 +0000
Subject: [U-Boot] [PATCH 16/16] board_f: powerpc: Drop unused headers
References: <20170331144039.14587-1-sjg@chromium.org>
 <20170331144039.14587-17-sjg@chromium.org>
Message-ID: <DB6PR0401MB26788D7DEE74B250B0CF4CB69A080@DB6PR0401MB2678.eurprd04.prod.outlook.com>
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

On 03/31/2017 07:41 AM, Simon Glass wrote:
> These includes don't seem to be needed now. Drop them. Reserve the
> mp.h header for PowerPC for now.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
>  common/board_f.c | 18 +++---------------
>  1 file changed, 3 insertions(+), 15 deletions(-)
>
> diff --git a/common/board_f.c b/common/board_f.c
> index bbcf6f4fe5..14f901571f 100644
> --- a/common/board_f.c
> +++ b/common/board_f.c
> @@ -22,21 +22,6 @@
>  #include <logbuff.h>
>  #include <malloc.h>
>  #include <mapmem.h>
> -
> -/* TODO: Can we move these into arch/ headers? */
> -#ifdef CONFIG_8xx
> -#include <mpc8xx.h>
> -#endif
> -#ifdef CONFIG_5xx
> -#include <mpc5xx.h>
> -#endif
> -#ifdef CONFIG_MPC5xxx
> -#include <mpc5xxx.h>
> -#endif
> -#if defined(CONFIG_MP) && (defined(CONFIG_MPC86xx) || defined(CONFIG_E500))
> -#include <asm/mp.h>
> -#endif
> -
>  #include <os.h>
>  #include <post.h>
>  #include <relocate.h>
> @@ -46,6 +31,9 @@
>  #include <trace.h>
>  #include <video.h>
>  #include <watchdog.h>
> +#if defined(CONFIG_MP) && defined(CONFIG_PPC)
> +#include <asm/mp.h>
> +#endif

I don't know if this is safe for other PPC SoCs. For FSL/NXP PPC (in 
this case, only MPC86xx and E500 have CONFIG_MP),

Reviewed-by: York Sun <york.sun@nxp.com>