[M] Change in coreboot[master]: vendorcode/amd/fsp/common: Refactor dmi_info.h

[Eric Lai (Code Review)]

Attention is currently required from: Felix Held, Fred Reitberger, Jason Glenesk, Konrad Adamczyk, Martin L Roth, Matt DeVillier, Paul Menzel, Raul Rangel.

Eric Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76107?usp=email )

Change subject: vendorcode/amd/fsp/common: Refactor dmi_info.h
......................................................................

Patch Set 2:

(1 comment)

File src/vendorcode/amd/fsp/common/dmi_info.h:

https://review.coreboot.org/c/coreboot/+/76107/comment/4d1db78f_b0e34cb2 :
PS2, Line 144: } DMI_T17_MEMORY_TYPE;
> That's valid point (in fact, more enums/structs defined in this file might be reused here). […]

Yes, I agree. we can clean up if we have better idea. What I am trying to say is if something is defined in smbios, we can rely on smbios to decode. FSP can report the enum and just pass to cbmem dmi table, let coreboot to decode it. Assume the value is followed the smbios spec.

SoC level todo is bypass the FSP mem_hob and fill the dmi_info. SoC don't need the detail here since FSP already do the job, and smbios will do it too.