Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36298 )
Change subject: vendorcode/secunet: Add driver to read DMI info from I2C EEPROM ......................................................................
Patch Set 1:
Patch Set 1:
Not sure if this belongs into vendorcode/ or drivers/. Beside the header file, everything is tied into coreboot. Though, there is also a related user-space tool pending, that might fit better into vendorcode/ (if it should be upstream at all).
user-space tools belong in util/* (and I think we already have the odd instance there of userland including headers directly out of coreboot).
For the decision vendorcode vs. drivers, I think the more important piece of information is if this stuff is (and remains) maintained elsewhere. If coreboot.git isn't the upstream, that's a signal for vendorcode. Otherwise, consider drivers.