[PATCH 05/32] spl: mx6: powerpc: Drop the condition on timer_init()

[Simon Glass]

Hi Tom,

On Thu, 31 Aug 2023 at 11:51, Tom Rini <trini at konsulko.com> wrote:
>
> On Wed, Aug 30, 2023 at 12:04:36PM -0600, Simon Glass wrote:
>
> > It doesn't make sense to have some boards do this differently. Drop the
> > condition in the hope that the maintainers can figure out any run-time
> > problems.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> >
> >  common/spl/spl.c | 6 ------
> >  1 file changed, 6 deletions(-)
> >
> > diff --git a/common/spl/spl.c b/common/spl/spl.c
> > index 78db9ef5318..3f513b0563a 100644
> > --- a/common/spl/spl.c
> > +++ b/common/spl/spl.c
> > @@ -766,13 +766,7 @@ void board_init_r(gd_t *dummy1, ulong dummy2)
> >               if (spl_init())
> >                       hang();
> >       }
> > -#if !defined(CONFIG_PPC) && !defined(CONFIG_ARCH_MX6)
> > -     /*
> > -      * timer_init() does not exist on PPC systems. The timer is initialized
> > -      * and enabled (decrementer) in interrupt_init() here.
> > -      */
> >       timer_init();
> > -#endif
>
> PowerPC might be a little tricky, did qemu-ppce500 run?  And please
> reach out to some of the iMX folks instead of just dropping this and
> hoping it works.

Yes, CI passes.

+Christophe Leroy
+Marek Behún

PowerPC people, does this look OK?

Regards,
Simon