[S] Change in coreboot[master]: src/arch/x86/smbios.c: Change smbios_fill_dimm_serial_number function...

[Eric Lai (Code Review)]

Attention is currently required from: Arthur Heymans, Jeff Li, Johnny Lin, Jonathan Zhang, Lean Sheng Tan, Nill Ge, Subrata Banik, TangYiwei, lichenchen.carl, [e-mail].

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

Change subject: src/arch/x86/smbios.c: Change smbios_fill_dimm_serial_number function to weak
......................................................................

Patch Set 1:

(1 comment)

Patchset:

PS1:
> The 'serial' defined in dimm_info structure means 'SPD serial number' which conforms to SPD spec. […]

I've checked the spec.

The supplier shall include a unique serial number for the module. The supplier may use whatever decode method
desired to maintain a unique serial number for each module.
One method of achieving this is by assigning a byte in the field from 517~520 as a tester ID byte and using the
remaining bytes as a sequential serial number. Bytes 512~520 will then result in a nine-byte unique module identifier.
Note that part number is not included in this identifier: the supplier may not give the same value for Bytes 517~520 to
more than one DIMM even if the DIMMs have different part numbers.