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 <s...@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....@nxp.com> _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
