[coreboot-gerrit] [L] Change in coreboot[master]: [WIP] mb/prodrive/atlas: Populate smbios table with VPD from ECs EMI

[Lean Sheng Tan (Code Review)]

Attention is currently required from: Maximilian Brune, Angel Pons, David Milosevic.

Lean Sheng Tan has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68137 )

Change subject: [WIP] mb/prodrive/atlas: Populate smbios table with VPD from ECs EMI
......................................................................

Patch Set 8:

(1 comment)

File src/mainboard/prodrive/atlas/emi.c:

https://review.coreboot.org/c/coreboot/+/68137/comment/13790bdf_6d9fc98b 
PS6, Line 83: {
> It does not worsen readability at all. Those are mostly independent functions. […]

personally I think it makes sense to keep the write function, because:
1. it can be used for testing/debug when needed
2. since this is quite generic to EC we could also move it to common code/lib later on
3. a good reference