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

[Patrick Rudolph (Code Review)]

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

Change subject: Fix for missing  mem_rank info in SMBIOS
......................................................................

Patch Set 1:

(2 comments)

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

https://review.coreboot.org/#/c/31235/1/src/soc/intel/common/smbios.h@24 
PS1, Line 24: 		u32 frequency, u8 rank_per_dim, u8 channel_id, u8 dimm_id,
prototype doesn't match implementation

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

https://review.coreboot.org/#/c/31235/1/src/soc/intel/common/smbios.c@23 
PS1, Line 23: 		u32 frequency, u8 rankInDimm, u8 channel_id, u8 dimm_id,
please don't use camel case