[PATCH v2 15/18] spl: Commit MTRRs only in board_init_f_r()
Bin Meng
bmeng.cn at gmail.com
Thu May 4 16:22:34 CEST 2023
Hi Simon,
On Tue, Apr 18, 2023 at 11:31 PM Simon Glass <sjg at chromium.org> wrote:
>
> We don't need to commit the SPI-flash MTRR change immediately, since it is
> now done in the board_init_f_r(). Also this causes chromebook_link64 to
> hang.
Could you add more details as to why this causes chromebook_link64 to hang?
As I see the mtrr_commit() has been there since TPL was introduced on
day 1, there must be some other places changes that caused the
regression on chromebook_link64??
>
> Drop the extra mtrr_commit() in the SPL implementation.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> (no changes since v1)
>
> arch/x86/lib/spl.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/x86/lib/spl.c b/arch/x86/lib/spl.c
> index 388b51086410..f0b74d04252a 100644
> --- a/arch/x86/lib/spl.c
> +++ b/arch/x86/lib/spl.c
> @@ -147,7 +147,6 @@ static int x86_spl_init(void)
> debug("%s: SPI cache setup failed (err=%d)\n", __func__, ret);
> return ret;
> }
> - mtrr_commit(true);
> # else
> ret = syscon_get_by_driver_data(X86_SYSCON_PUNIT, &punit);
> if (ret)
> --
Regards,
Bin
More information about the U-Boot
mailing list