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