[coreboot-gerrit] Change in ...coreboot[master]: soc/intel: Fix for missing mem_rank info in SMBIOS

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31235 )

Change subject: soc/intel: Fix for missing mem_rank info in SMBIOS
......................................................................

Patch Set 9:

(3 comments)

https://review.coreboot.org/#/c/31235/9/3rdparty/blobs 
File 3rdparty/blobs:

PS9: 
This should be removed.

https://review.coreboot.org/#/c/31235/9/src/soc/intel/common/smbios.h 
File src/soc/intel/common/smbios.h:

https://review.coreboot.org/#/c/31235/9/src/soc/intel/common/smbios.h@24 
PS9, Line 24: dim
dimm?

https://review.coreboot.org/#/c/31235/9/src/soc/intel/common/smbios.c 
File src/soc/intel/common/smbios.c:

https://review.coreboot.org/#/c/31235/9/src/soc/intel/common/smbios.c@23 
PS9, Line 23: rankInDimm
keep this the same as the param name in declaration in smbios.h?