[coreboot-gerrit] [S] Change in coreboot[main]: vendorcode/google/chromeos: Add API to read factory config

[Julius Werner (Code Review)]

Attention is currently required from: Subrata Banik.

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79737?usp=email )

Change subject: vendorcode/google/chromeos: Add API to read factory config
......................................................................

Patch Set 8:

(1 comment)

File src/vendorcode/google/chromeos/tpm_factory_config.c:

https://review.coreboot.org/c/coreboot/+/79737/comment/d90e81ca_4a4b2b2a :
PS8, Line 7: int64_t
I don't think it's a good idea to make this signed everywhere, it's not a signed number, it's a bit field. Even if the top bit will never be allocated, making it signed is confusing.

You can still use `~(uint64_t)0` as a sentinel value even if it's an unsigned number (e.g. like `UNDEFINED_FW_CONFIG`).