[coreboot-gerrit] Change in coreboot[master]: lib/fmap: add ENV_SMM check to setup_preram_cache

[Aaron Durbin (Code Review)]

Aaron Durbin has submitted this change. ( https://review.coreboot.org/c/coreboot/+/45274 )

Change subject: lib/fmap: add ENV_SMM check to setup_preram_cache
......................................................................

lib/fmap: add ENV_SMM check to setup_preram_cache

Add check in setup_preram_cache to return if ENV_SMM is true.
This avoids false warning that post-RAM FMAP is accessed too early
caused by ENV_ROMSTAGE_OR_BEFORE evaluation in SMI handler.

BUG=b:167321319
BRANCH=None
TEST=None

Signed-off-by: Josie Nordrum 
Change-Id: I3a4c199c42ee556187d6c4277e8793a36e4d493b
Reviewed-on: https://review.coreboot.org/c/coreboot/+/45274
Tested-by: build bot (Jenkins) 
Reviewed-by: Arthur Heymans 
Reviewed-by: Aaron Durbin 
---
M src/lib/fmap.c
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Aaron Durbin: Looks good to me, approved
  Arthur Heymans: Looks good to me, but someone else must approve

diff --git a/src/lib/fmap.c b/src/lib/fmap.c
index e1e6a57..377123a 100644
--- a/src/lib/fmap.c
+++ b/src/lib/fmap.c
@@ -46,6 +46,10 @@
    if (CONFIG(NO_FMAP_CACHE))
    	return;

+	/* No need to use FMAP cache in SMM */
+	if (ENV_SMM)
+		return;
+
    if (!ENV_ROMSTAGE_OR_BEFORE) {
    	/* We get here if ramstage makes an FMAP access before calling
    	   cbmem_initialize(). We should avoid letting it come to that,