[S] Change in coreboot[master]: mb/google/skyrim: Implement touchscreen power sequencing

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Raul Rangel, Jason Nien, Matt DeVillier, Martin Roth.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67778 )

Change subject: mb/google/skyrim: Implement touchscreen power sequencing
......................................................................

Patch Set 4:

(1 comment)

File src/mainboard/google/skyrim/variants/baseboard/gpio.c:

https://review.coreboot.org/c/coreboot/+/67778/comment/ced03567_af0cf4ec 
PS3, Line 116: 	/* EN_PP3300_TCHSCR */
             : 	PAD_GPO(GPIO_131, LOW),
             : 	/* TCHSCR_RESET_L */
             : 	PAD_GPO(GPIO_136, LOW),
> Yeah, I think that sounds fine. I don't think on the AMD platforms we ever had a bit.

I think this definitely makes a good reason to have a FW_CONFIG bit to indicate the presence of touch screen.

An alternate is like Raul mentioned, we should assume that touch screen is present. When all the touchscreen modules defined in devicetree could not be probed, we can put these GPIOs as NC?