Change in coreboot[master]: mb/prodrive/hermes: Fix eeprom reading

[Angel Pons (Code Review)]

Attention is currently required from: Patrick Rudolph, Christian Walter, Arthur Heymans.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52207 )

Change subject: mb/prodrive/hermes: Fix eeprom reading
......................................................................

Patch Set 2: Code-Review+1

(1 comment)

File src/mainboard/prodrive/hermes/eeprom.c:

https://review.coreboot.org/c/coreboot/+/52207/comment/fd3a2053_80740e5c 
PS2, Line 125: 		memcpy(writePointer, tmp, bytes_to_copy);
Shouldn't this be:

uint8_t writePointer = (uint8_t *)blob + write_offset + i;
 writePointer[0] = tmp[0];
 if (size - i > 1)
    writePointer[1] = tmp[1];

Otherwise, we will write one byte too many when size is odd and greater than 1 (e.g. 3).

I was writing the above for Patchset 1, but you pushed Patchset 2 in the meantime. Still, I'd rather not use memcpy here.