build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32307 )
Change subject: [WIP]sconfig: Add SMBIOS type 9 entries ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/#/c/32307/1/src/include/device/device.h File src/include/device/device.h:
https://review.coreboot.org/#/c/32307/1/src/include/device/device.h@147 PS1, Line 147: const char* smbios_slot_designation; "foo* bar" should be "foo *bar"
https://review.coreboot.org/#/c/32307/1/util/sconfig/main.c File util/sconfig/main.c:
https://review.coreboot.org/#/c/32307/1/util/sconfig/main.c@643 PS1, Line 643: printf("ERROR: 'slot_data_width' only allowed for PCI devices\n"); line over 80 characters
https://review.coreboot.org/#/c/32307/1/util/sconfig/main.c@655 PS1, Line 655: printf("ERROR: 'slot_designation' only allowed for PCI devices\n"); line over 80 characters