[PATCH] x86: Correct condition for init_cache_f_r()

[Simon Glass]

Hi Jerome,

On Wed, 4 Jun 2025 at 07:15, Jerome Forissier
<jerome.forissier at 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 <sjg at 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.forissier at 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?

Regards,
SImon