Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42920 )
Change subject: mb/supermicro/x11ssh-tf: set Host SW Pad Ownership to DRIVER ......................................................................
Patch Set 10:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42920/10/src/mainboard/supermicro/x... File src/mainboard/supermicro/x11-lga1151-series/variants/x11ssh-tf/include/variant/gpio.h:
https://review.coreboot.org/c/coreboot/+/42920/10/src/mainboard/supermicro/x... PS10, Line 48: PAD_CFG_GPO_GPIO_DRIVER
you mean the macro itself but not it's use here, don't you?
Hmm, tbh I'm a bit confused...
doc#332691-003EN says that the ownership bit decides which interrupt bits get masked. Comments in fsp/Include/GpioConfig.h confirm that. Those bits are only relevant for input/interrupt pads.
fsp/Include/GpioConfig.h also states that ownership must be set to driver if the OS driver shall be able to control the pad, but an ACPI entry (GpioIo/GpioInt) is required.
I have no idea if "control the pad" translates to "change direction" or only "change state if set as output".