[coreboot-gerrit] Change in coreboot[master]: fw_config: Add firmware configuration interface

[Duncan Laurie (Code Review)]

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:

(1 comment)

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
> If the debug string aren't present, shouldn't we still match (but not print out as much in the probe […]

I could key off "mask=0" and make that an invalid probe instead of needing the pointers to be a list terminator.