Hi Patrick. > I'm not sure about Siemens' hwilib: if that's imported, it remains there, if > this version is written for coreboot, src/drivers/siemens perhaps?
This code is basically a pure software library to access defined fields within a binary image kept inside CBFS. Since there is no hardware this code especially deals with I am not sure about src/drivers/ as the host directory. But I am totally fine with moving it into the tree and out of vendorcode, maybe into src/lib/siemens? Werner Von: Patrick Georgi via coreboot <coreboot@coreboot.org> Gesendet: Mittwoch, 7. April 2021 10:59 An: Peter Stuge <pe...@stuge.se> Cc: coreboot <coreboot@coreboot.org> Betreff: [coreboot] Re: On handling vendorcode Am Di., 6. Apr. 2021 um 21:21 Uhr schrieb Peter Stuge <mailto:pe...@stuge.se>: Patrick Georgi via coreboot wrote: > Any objections to moving the code out there that has no other upstream > (e.g. src/vendorcode/google/chromeos or src/vendorcode/eltan, I think?) > while moving in code from elsewhere in the tree that fits the "it has a > foreign upstream" description (e.g. the lzma library)? I think that can make sense, but where would the chromeos, eltan and other directories go instead? Someplace existing, or someplace new? I first wanted to make sure that there are no major issues with the overall idea, but to further the discussion: - src/vendorcode/eltan/security -> src/security/mboot - src/vendorcode/google/chromeos is a mixed bag: some src/drivers/tpm/cr50, some src/lib (e.g. elog, vpd), some src/acpi/chromeos (acpi*) I'm not sure about Siemens' hwilib: if that's imported, it remains there, if this version is written for coreboot, src/drivers/siemens perhaps? The other things look to be "real" vendorcode, even if we're probably the last codebase by now that still ships (our versions of) amd/agesa. Patrick -- Google Germany GmbH, ABC-Str. 19, 20354 Hamburg Registergericht und -nummer: Hamburg, HRB 86891, Sitz der Gesellschaft: Hamburg Geschäftsführer: Paul Manicle, Halimah DeLaine Prado _______________________________________________ coreboot mailing list -- coreboot@coreboot.org To unsubscribe send an email to coreboot-le...@coreboot.org