Change in coreboot[master]: arch/x86: Refactor the SMBIOS type 17 write function

[Angel Pons (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Tim Wawrzynczak, Subrata Banik, Patrick Rudolph, Felix Held.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56628 )

Change subject: arch/x86: Refactor the SMBIOS type 17 write function
......................................................................

Patch Set 11:

(4 comments)

File src/device/dram/spd.c:

https://review.coreboot.org/c/coreboot/+/56628/comment/47780440_80ac8068 
PS11, Line 49: uint8_t
Why not use enum types instead of plain integer types? It's easy to forget which enum goes with which variable.

(Also applies to other cases like this in this patch)

https://review.coreboot.org/c/coreboot/+/56628/comment/ea295f19_d644cbe7 
PS11, Line 171: sizeof(struct spd_info)
sizeof(*info)

https://review.coreboot.org/c/coreboot/+/56628/comment/69df4029_aae82748 
PS11, Line 171: (struct spd_info *) 
cast not needed

https://review.coreboot.org/c/coreboot/+/56628/comment/5cd8057e_30f4cfbc 
PS11, Line 171: malloc
Why not let the caller pass a pointer to a `struct spd_info` instead? This way, there's no need to deal with dynamic memory in here.