Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47369 )
Change subject: mb/google/zork: Add func fpmcu_needs_delay ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/47369/1/src/mainboard/google/zork/v... File src/mainboard/google/zork/variants/baseboard/include/baseboard/variants.h:
https://review.coreboot.org/c/coreboot/+/47369/1/src/mainboard/google/zork/v... PS1, Line 37: fpmcu_needs_delay A thought: Instead of having a function call back, does it make sense to have a Kconfig here like https://source.chromium.org/chromiumos/chromiumos/codesearch/+/master:src/th....
* VARIANT_HAS_BROKEN_FPMCU_POWER * VARIANT_MAX_BOARD_ID_BROKEN_FMPCU_POWER
and these can be used directly in gpio_baseboard_trembyle.c. It is one less variant function to care about and newer boards using newer schematics don't need to worry about this at all. (I know weak function provides the same functionality, but having the Kconfig allows easier audit to decide when to drop the support for these quirks).