[coreboot-gerrit] Change in coreboot[master]: mb/google/hatch: Refactor override_early_gpio_table

[Tim Wawrzynczak (Code Review)]

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

Change subject: mb/google/hatch: Refactor override_early_gpio_table
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34782/4/src/mainboard/google/hatch/... 
File src/mainboard/google/hatch/variants/hatch/gpio.c:

https://review.coreboot.org/c/coreboot/+/34782/4/src/mainboard/google/hatch/... 
PS4, Line 56: 	PAD_CFG_GPI(GPP_F2, NONE, PLTRST),
> Not using gpio_base2_value() wouldn't help since it is getting configured in gpio_input(). […]

Sure, gpio_base2_value() calls gpio_input() before reading the GPIOs, which, from gpio_base2_value()'s name and description, is misleading.  I'll add some comments to the tables, explaining the reasoning for dropping the MEM_STRAP_* pins.