[U-Boot] [PATCH 089/126] x86: Set up the MTRR for SDRAM
Bin Meng
bmeng.cn at gmail.com
Thu Oct 10 09:18:53 UTC 2019
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass <sjg at chromium.org> wrote:
>
> Set up MTRRs for the FST SDRAM regions to improve performance.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> arch/x86/lib/fsp/fsp_dram.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/arch/x86/lib/fsp/fsp_dram.c b/arch/x86/lib/fsp/fsp_dram.c
> index 9161af65b98..4a87516f422 100644
> --- a/arch/x86/lib/fsp/fsp_dram.c
> +++ b/arch/x86/lib/fsp/fsp_dram.c
> @@ -8,6 +8,7 @@
> #include <asm/fsp/fsp_support.h>
> #include <asm/e820.h>
> #include <asm/mrccache.h>
> +#include <asm/mtrr.h>
> #include <asm/post.h>
>
> DECLARE_GLOBAL_DATA_PTR;
> @@ -70,6 +71,8 @@ int dram_init_banksize(void)
> gd->bd->bi_dram[0].start = 0;
> gd->bd->bi_dram[0].size = low_end;
>
> + mtrr_add_request(MTRR_TYPE_WRBACK, 0, low_end);
I suspect we don't need explicit MTRR configuration. I see on the FSP
1.0 boards, MTRR is already programmed by FSP.
But I am fine adding this. So if we are adding this MTRR, please also
move MTRR programming from previous patch ([088/126] x86: Set the DRAM
banks to reflect real location) to this patch.
> +
> return 0;
Regards,
Bin
More information about the U-Boot
mailing list