Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34189 )
Change subject: gpio: Add a function to get current output value of a GPIO. ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/34189/1/src/include/gpio.h File src/include/gpio.h:
https://review.coreboot.org/c/coreboot/+/34189/1/src/include/gpio.h@27 PS1, Line 27: int gpio_get_output(gpio_t gpio); This is a pretty odd use case, I think it might be better to just have this function at the platform level. I can see how it makes sense for x86 SMM, but in other architectures without runtime components it doesn't really come up that you wouldn't know what value a certain pin is set to.
If you do want to add this to the generic API, please move it away from those other definitions (because the comment above says these are required), and add a comment clarifying that this one is optional.