Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42690 )
Change subject: soc/amd/common: Complete ACPIMMIO GPIO bank separation ......................................................................
Patch Set 4:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42690/4/src/soc/amd/common/block/gp... File src/soc/amd/common/block/gpio_banks/gpio.c:
https://review.coreboot.org/c/coreboot/+/42690/4/src/soc/amd/common/block/gp... PS4, Line 83: return acpimmio_gpio0 + gpio_num * sizeof(uint32_t); I skipped ahead a few patches to this one. I thought we already established this function? Why is it reintroduced? Or am I just confused now?