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

[David Milosevic (Code Review)]

Attention is currently required from: Maximilian Brune, Angel Pons, Arthur Heymans, Lean Sheng Tan.

David Milosevic 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 11:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/68137/comment/6830cd9d_ca138722 
PS11, Line 68: if (idx == n)
             : 				return;
             : 
             : 			buff[idx++]
> I could do that but then we would need the same branch right after the for-loop in order to determin […]

Even if we put this condition into the outer loop, we would have one branch/check more than actually needed

while(idx != n) { // check idx != n
     [...]
     for(u8 i = 0; i < 4 && idx != n; i++) // check idx != n
         buff[idx++] = bytes[i];
 }