[coreboot-gerrit] Change in ...coreboot[master]: mb/google/hatch: Reconfigure SD card detect GPIO host software ownership

[Rizwan Qureshi (Code Review)]

Rizwan Qureshi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31178 )

Change subject: mb/google/hatch: Reconfigure SD card detect GPIO host software ownership
......................................................................

Patch Set 1:

(1 comment)
> Patch Set 1:
>
> (1 comment)

https://review.coreboot.org/#/c/31178/1/src/mainboard/google/hatch/variants/... 
File src/mainboard/google/hatch/variants/baseboard/gpio.c:

https://review.coreboot.org/#/c/31178/1/src/mainboard/google/hatch/variants/... 
PS1, Line 141: PAD_CFG_GPI_GPIO_DRIVER
> What is the impact of doing this on the SD controller? Wouldn't it want the pin to be in native mode?

We are checking this.
> Since the kernel driver wants to configure this in a certain way, would it make sense to dual-route this signal?

Or, does the kernel really need to program this pin for IRQ?