On Wed, Oct 21, 2020 at 1:20 PM Tim Wawrzynczak via coreboot <coreboot@coreboot.org> wrote:

Hi coreboot community,

Currently there are 3 different "strapping" entries in the coreboot tables, and with the recent addition of fw_config (https://review.coreboot.org/c/coreboot/+/41209), we would also like to add the 64-bit fw_config field (updated here https://review.coreboot.org/c/coreboot/+/45939) to the coreboot table as well.

In this patch (https://review.coreboot.org/c/coreboot/+/46605), I am proposing to deprecate the 3 current "strapping" entries (board ID, ram code and SKU ID), and add them all to 1 entry, containing board ID, ram code, SKU ID as well as fw_config. This saves the overhead of parsing 4 different entries to obtain board configuration information.

Not all of these entities are sourced through the same mechanism (gpios vs cbi). As such any time you want to read one of those fields you'll be unconditionally needing to obtain all of those while at the time the user may only want one field. And then this can happen from multiple stages.

I would say that since these things aren't all related to answering the question someone may want to answer that we shouldn't go down this path. Width of fields, e.g., could be different across platforms since not everything is consistent.

Would like to hear any thoughts on this,
- Tim
_______________________________________________
coreboot mailing list -- coreboot@coreboot.org
To unsubscribe send an email to coreboot-leave@coreboot.org