Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44534 )
Change subject: soc/amd/common: add gpio subsystem event reporting ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44534/1/src/soc/amd/common/block/gp... File src/soc/amd/common/block/gpio_banks/gpio.c:
https://review.coreboot.org/c/coreboot/+/44534/1/src/soc/amd/common/block/gp... PS1, Line 354: check_gpios(state->wake_stat[0], 32, 0, state); : check_gpios(state->wake_stat[1], 14, 128, state); I'm wondering if I should move this whole patch to be under picasso. I don't think this logic applies to stoney, and I'm not really wanting to try and sort that out. I have a feeling many things are not "common".