Raul Rangel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42522 )
Change subject: soc/amd/common: Drop ACPIMMIO GPIO bank separation ......................................................................
Patch Set 6:
(3 comments)
https://review.coreboot.org/c/coreboot/+/42522/6/src/soc/amd/common/block/gp... File src/soc/amd/common/block/gpio_banks/gpio.c:
https://review.coreboot.org/c/coreboot/+/42522/6/src/soc/amd/common/block/gp... PS6, Line 88: static void __gpio_update32(gpio_t gpio_num, uint32_t mask, uint32_t or) : { : uint32_t reg; : : reg = gpio_read32(gpio_num); : reg &= mask; : reg |= or; : gpio_write32(gpio_num, reg); : } In a different patch I had replaced all the invocations with mem_read_write32: https://chromium-review.googlesource.com/c/chromiumos/third_party/coreboot/+...
Guessing this patch wasn't pushed.
Is your goal to delete mem_read_write32?
https://review.coreboot.org/c/coreboot/+/42522/6/src/soc/amd/common/block/gp... PS6, Line 98: __ why the __? See https://stackoverflow.com/questions/25090635/use-and-in-c-programs
All identifiers that begin with an underscore and either an uppercase letter or another underscore are always reserved for any use.
https://review.coreboot.org/c/coreboot/+/42522/6/src/soc/amd/common/block/in... File src/soc/amd/common/block/include/amdblocks/acpimmio.h:
https://review.coreboot.org/c/coreboot/+/42522/6/src/soc/amd/common/block/in... PS6, Line 372: static inline void gpio_write32_rb(uint8_t gpio_num, uint32_t value) : { : write32(gpio_ctrl_ptr(gpio_num), value); : read32(gpio_ctrl_ptr(gpio_num)); : } : I don't think it's worth while to pollute the public API with this. If you really feel strongly about keeping it, I would add some documentation as to when you would want to use this.