[coreboot-gerrit] Change in coreboot[master]: drivers/vpd: Add support to read device serial from VPD

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49050 )

Change subject: drivers/vpd: Add support to read device serial from VPD
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/49050/1/src/drivers/vpd/vpd.c 
File src/drivers/vpd/vpd.c:

https://review.coreboot.org/c/coreboot/+/49050/1/src/drivers/vpd/vpd.c@306 
PS1, Line 306: har *smbios_system_serial_number(void)
             : {
             : 	static char serial[VPD_SERIAL_LEN];
             : 	if (vpd_gets(VPD_KEY_SYSTEM_SERIAL, serial, VPD_SERIAL_LEN, VPD_RO))
             : 		return serial;
             : 	return "";
             : }
             : 
             : const char *smbios_mainboard_serial_number(void)
             : {
             : 	static char serial[VPD_SERIAL_LEN];
             : 	if (vpd_gets(VPD_KEY_MAINBOARD_SERIAL, serial, VPD_SERIAL_LEN, VPD_RO))
             : 		return serial;
             : 	return "";
             : }
             : #endif
> IDK, should it? seems like these methods are located in the overriding entity, so would make sense t […]

Seems right to have the VPD backed override in `vpd/`. Probably not in this
file, though, see other comment.