[coreboot-gerrit] Change in coreboot[master]: mb/google/deltaur: Add initial GPIO configuration

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39674 )

Change subject: mb/google/deltaur: Add initial GPIO configuration
......................................................................

Patch Set 11:

(2 comments)

https://review.coreboot.org/c/coreboot/+/39674/7/src/mainboard/google/deltau... 
File src/mainboard/google/deltaur/variants/baseboard/gpio.c:

https://review.coreboot.org/c/coreboot/+/39674/7/src/mainboard/google/deltau... 
PS7, Line 56: programmed in early table
> Maybe we can leave as comment? That should be clear enough. […]

An X-Macro would be a decent way to only have to define in one place, but I don't think that would fly on coreboot 😊

https://review.coreboot.org/c/coreboot/+/39674/11/src/mainboard/google/delta... 
File src/mainboard/google/deltaur/variants/baseboard/gpio.c:

https://review.coreboot.org/c/coreboot/+/39674/11/src/mainboard/google/delta... 
PS11, Line 201: NONE
> Here is the Drallion and Sarien device tree for this pin. […]

Ack