[coreboot-gerrit] Change in ...coreboot[master]: util/inteltool: Improve GPIO specification

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31634 )

Change subject: util/inteltool: Improve GPIO specification
......................................................................

Patch Set 12:

(1 comment)

https://review.coreboot.org/#/c/31634/10/util/inteltool/gpio.h 
File util/inteltool/gpio.h:

https://review.coreboot.org/#/c/31634/10/util/inteltool/gpio.h@44 
PS10, Line 44: uint32_t config0;
             : 	uint32_t config1;
> These are meant as cache for the register contents. […]

`x` means the default value depends on more than the platform (e.g. pin
straps, soft straps in the IFD). It makes handling of default values
very tedious. Hard to tell if it would be worth the effort to add them.