]> git.dujemihanovic.xyz Git - u-boot.git/commit
video: ivybridge: Use mtrr_set_next_var() for graphics memory
authorBin Meng <bmeng.cn@gmail.com>
Mon, 31 Jul 2023 06:01:06 +0000 (14:01 +0800)
committerBin Meng <bmeng.cn@gmail.com>
Tue, 1 Aug 2023 02:06:46 +0000 (10:06 +0800)
commit0f497b2b8cb2c448cef70460d466c4ad1459f5ec
tree245861c3a72a18caefe810827053b35fe77a34a2
parentb5126f26d0c8d8560081b1f36ed7aee7f083104b
video: ivybridge: Use mtrr_set_next_var() for graphics memory

At present this uses mtrr_add_request() & mtrr_commit() combination
to program the MTRR for graphics memory. This usage has two major
issues as below:

- mtrr_commit() will re-initialize all MTRR registers from index 0,
  using the settings previously added by mtrr_add_request() and saved
  in gd->arch.mtrr_req[], which won't cause any issue but is unnecessary
- The way such combination works is based on the assumption that U-Boot
  has full control with MTRR programming (e.g.: U-Boot without any blob
  that does all low-level initialization on its own, or using FSP2 which
  does not touch MTRR), but this is not the case with FSP. FSP programs
  some MTRRs during its execution but U-Boot does not have the settings
  saved in gd->arch.mtrr_req[] and when doing mtrr_commit() it will
  corrupt what was already programmed previously.

Correct this to use mtrr_set_next_var() instead.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
drivers/video/ivybridge_igd.c