[coreboot-gerrit] [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:
> I've got your thoughts now. […]

we can create 5 byte mask 0xffffffffff and &serial[4], I think there is no number will contain all 0 in the next 5 bytes. Is that work?