[U-Boot] [PATCH 16/16] board_f: powerpc: Drop unused headers

york sun york.sun at nxp.com
Mon Apr 3 15:56:30 UTC 2017


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 at 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 at nxp.com>


More information about the U-Boot mailing list