Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41209 )
Change subject: fw_config: Add firmware configuration interface ......................................................................
Patch Set 5:
(3 comments)
https://review.coreboot.org/c/coreboot/+/41209/5/Documentation/lib/fw_config... File Documentation/lib/fw_config.md:
https://review.coreboot.org/c/coreboot/+/41209/5/Documentation/lib/fw_config... PS5, Line 10: The initial implementation is designed to take advantage of a bitmask returned by the Embedded : Controller on Google Chrome OS devices which allows the manufacturer to use the same firmware : image across multiple devices by selecting various options at runtime.
Link to https://chromium.googlesource.com/chromiumos/docs/+/master/design_docs/firmw... […]
Done
https://review.coreboot.org/c/coreboot/+/41209/5/src/include/fw_config.h File src/include/fw_config.h:
https://review.coreboot.org/c/coreboot/+/41209/5/src/include/fw_config.h@17 PS5, Line 17: const char *field_name; : const char *option_name;
if we don't change matching logic, then we should document that these fields are required to be non- […]
Ack
https://review.coreboot.org/c/coreboot/+/41209/5/src/lib/fw_config.c File src/lib/fw_config.c:
https://review.coreboot.org/c/coreboot/+/41209/5/src/lib/fw_config.c@67 PS5, Line 67: match->field_name && match->option_name
Ah, I like the mask==0 better; that makes more sense to me.
Ack