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

[Furquan Shaikh (Code Review)]

Furquan Shaikh 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)

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
> > 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?

I believe the kernel needs it to enable/disable wake from SD card insert. I will have to go back and check the kernel driver to see its use. Rajat might be able to provide more input on this.