Hi Tom, On Thu, 31 Aug 2023 at 11:51, Tom Rini <tr...@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 <s...@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
