Attention is currently required from: Filip Brozovic, Matt DeVillier.
Angel Pons has posted comments on this change by Filip Brozovic. ( https://review.coreboot.org/c/coreboot/+/85987?usp=email )
Change subject: CFR: add dependencies based on specific option values ......................................................................
Patch Set 1:
(2 comments)
File src/drivers/option/cfr_frontend.h:
https://review.coreboot.org/c/coreboot/+/85987/comment/33ac5d07_4a4c3a6e?usp... : PS1, Line 115: .dep_values = ((const uint32_t[]) { __VA_ARGS__ }), \ Does this work? I'm not sure where the array will be stored.
https://review.coreboot.org/c/coreboot/+/85987/comment/dca9ecae_ff3667b5?usp... : PS1, Line 116: .num_dep_values = sizeof((uint32_t[]) { __VA_ARGS__ }) / sizeof(uint32_t) ARRAY_SIZE macro exists