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

[Subrata Banik (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Tim Wawrzynczak, Angel Pons, Patrick Rudolph, Felix Held.
Subrata Banik 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/9b0d10cb_0590c5d9 
PS11, Line 49: uint8_t
> Why not use enum types instead of plain integer types? It's easy to forget which enum goes with whic […]

Ack

https://review.coreboot.org/c/coreboot/+/56628/comment/15a6dff5_da95c304 
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 d […]

Ack

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

Ack

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

Ack