[coreboot-gerrit] [M] Change in coreboot[master]: cpu/x86: Support SMBASE relocation-only use-case

[Benjamin Doron (Code Review)]

Attention is currently required from: Angel Pons, Dinesh Gehlot, Felix Held, Fred Reitberger, Jason Glenesk, Kapil Porwal, Matt DeVillier, Maulik Vaghela, Raul Rangel, Ron Minnich, Subrata Banik, Tarun Tuli.

Benjamin Doron has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/70376?usp=email )

Change subject: cpu/x86: Support SMBASE relocation-only use-case
......................................................................

Patch Set 5:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/70376/comment/78435653_efbd62ef :
PS4, Line 9: NO_SMM
> I think so too. […]

Done

File src/cpu/x86/smm/smm_module_loader.c:

https://review.coreboot.org/c/coreboot/+/70376/comment/f408de47_6c31ccf6 :
PS4, Line 138: #if !CONFIG(X86_SMM_CALL_RELOCATION_HANDLER_ONLY)
             : u32 smm_get_cpu_smbase(unsigned int cpu_num)
             : {
             : 	if (cpu_num < CONFIG_MAX_CPUS) {
             : 		if (cpus[cpu_num].active)
             : 			return cpus[cpu_num].smbase;
             : 	}
             : 	return 0;
             : }
             : #endif
> It would be good to state in a comment that this function would otherwise be defined by platform cod […]

Done