Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34337 )
Change subject: soc/intel/common: add PAD_CFG_NF_BUF_TRIG macro ......................................................................
Patch Set 4:
Patch Set 4:
Patch Set 3:
I personally find the GPIO macros very cryptic, and would prefer something more like sandybridge and ivybridge's gpio.c if possible.
Did you mean to use separate structures instead of macros? As it is done in https://github.com/coreboot/coreboot/blob/master/src/mainboard/asrock/b75pro...
Honestly, macros look better to me ) But that's my personal opinion
One of the advantages of using structures like the one you linked to is that you have to assign values to named members. If you forget one of the elements in an array, the rest of the values would get shifted downwards. And that can be potentially fatal to the GPIO pins: if a pin used as input gets configured as an output, short-circuits can happen!