Francois Toguo Fotso has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32090 )
Change subject: smbios: Add and adapt fields for SMBIOS update to version 3.2 ......................................................................
Patch Set 12:
(1 comment)
Patch Set 12:
(1 comment)
This question was addressed in a previous comments as follows:
The changes already present in here will take care of the type 17 new clock units. Another alternative would be to remove the change here and push it down in the FSP codes, the ultimate provider of these information. Which will need to be done for all SOCs since the FSP varies per SOC. .. Past SOCs and GLK (current) for example won't be making the change. Hence the decision to implement it here.
In short, with the x 2 multiplier, no change needed in src/include/memory_info.h
https://review.coreboot.org/#/c/32090/12/src/arch/x86/smbios.c File src/arch/x86/smbios.c:
https://review.coreboot.org/#/c/32090/12/src/arch/x86/smbios.c@279 PS12, Line 279: clock_speed
is it Configured Memory Speed ?
Done