On Wed, Jun 04, 2025 at 10:17:32AM -0600, Simon Glass wrote:
> Hi Jerome,
>
> On Wed, 4 Jun 2025 at 07:15, Jerome Forissier
> <jerome.foriss...@linaro.org> wrote:
> >
> > On 6/4/25 15:09, Simon Glass wrote:
> > > The condition here is reversed, which makes link and coral very slow,
> > > leading to lab failures.
> > >
> > > Signed-off-by: Simon Glass <s...@chromium.org>
> > > Fixes 6c171f7a184 ("common: board: make initcalls static")
> > > ---
> > >
> > > common/board_f.c | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/common/board_f.c b/common/board_f.c
> > > index bff465d9cb2..c8a612d6070 100644
> > > --- a/common/board_f.c
> > > +++ b/common/board_f.c
> > > @@ -1079,7 +1079,7 @@ void board_init_f(ulong boot_flags)
> > > */
> > > static void initcall_run_f_r(void)
> > > {
> > > -#if CONFIG_IS_ENABLED(X86_64)
> > > +#if !CONFIG_IS_ENABLED(X86_64)
> > > INITCALL(init_cache_f_r);
> > > #endif
> > > }
> >
> > Sorry for that mistake.
> >
> > Reviewed-by: Jerome Forissier <jerome.foriss...@linaro.org>
>
> It's fine. I only noticed it because the lab tests started failing (I
> have a short boot timeout on one of the boards).
>
> Do you think some of the conditions could use if() instead of #if now?I'm not sure the readability nor code coverage will be improved by that in this file. And please find the time to reply to my thread about your future in the project before making further suggestions about the overall code base, thanks. -- Tom
signature.asc
Description: PGP signature
