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

[Felix Held (Code Review)]

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

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

Patch Set 3:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/48564/2/src/soc/amd/cezanne/include... 
PS2, Line 3: AMD_PICASSO_GPIO_H
> change name

Done

https://review.coreboot.org/c/coreboot/+/48564/2/src/soc/amd/cezanne/include... 
PS2, Line 116: #define GPIO_10_IOMUX_S0A3 1
> nit, it looks like 2 is missing.  PPR calls it reserved.  could consider a comment for "reserved".

done for all comments and one more