[coreboot-gerrit] Change in coreboot[master]: soc/amd/cezanne: add GPIO definitions

[Raul Rangel (Code Review)]

Raul Rangel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48564 )

Change subject: soc/amd/cezanne: add GPIO definitions
......................................................................

Patch Set 4: Code-Review+2

(2 comments)

https://review.coreboot.org/c/coreboot/+/48564/4/src/soc/amd/cezanne/include... 
File src/soc/amd/cezanne/include/soc/gpio.h:

https://review.coreboot.org/c/coreboot/+/48564/4/src/soc/amd/cezanne/include... 
PS4, Line 48:
> oh, those aren't in the IOMUX table and are also missing in picasso. […]

Sounds good. Can you file a bug so we don't lose track of the work. We might need to refactor some of the gpio code to handle these cases.

https://review.coreboot.org/c/coreboot/+/48564/4/src/soc/amd/cezanne/include... 
PS4, Line 93:
> same situation as with the ones i commented about above

Ack