Change in coreboot[master]: arch/x86/smbios.c: Allow creating entries for empty DIMM slots

[Nico Huber (Code Review)]

Attention is currently required from: Arthur Heymans, Marc Jones, Jonathan Zhang, Angel Pons, lichenchen.carl, Nill Ge.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/64035 )

Change subject: arch/x86/smbios.c: Allow creating entries for empty DIMM slots
......................................................................

Patch Set 1:

(2 comments)

File src/arch/x86/smbios.c:

https://review.coreboot.org/c/coreboot/+/64035/comment/78c36aa5_5ad615c8 
PS1, Line 281: *current
> I just reused the signature of `create_smbios_type17_for_dimm()`. […]

Ack. I just noticed the whole file is full of such odd patterns.

https://review.coreboot.org/c/coreboot/+/64035/comment/b144c163_46969ae6 
PS1, Line 291: 	t->form_factor = 0x2; /* Unknown */
> It could be SO-DIMM in some cases.

Hmmm, if we expect that location info is filled for empty slots, should
we expect `form_factor` to be filled too?