[coreboot-gerrit] [S] Change in coreboot[master]: mb/google/skyrim/var/markarth: Change to read the eMMC clkreq instead

[Martin L Roth (Code Review)]

Attention is currently required from: John Su, Jason Nien, Jon Murphy, Jason Glenesk, Chris Wang, Martin Roth, Tim Van Patten, Karthik Ramasubramanian, Felix Held.

Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74583 )

Change subject: mb/google/skyrim/var/markarth: Change to read the eMMC clkreq instead
......................................................................

Patch Set 3:

(1 comment)

File src/mainboard/google/skyrim/variants/markarth/port_descriptors.c:

https://review.coreboot.org/c/coreboot/+/74583/comment/f812c728_c2b30f33 
PS3, Line 88: 	if (gpio_get(EMMC_CLKREQ_GPIO)) {
> Since each storage devices are behaving differently w.r. […]

Great.  Thank you Karthik.

For now, I propose that we file a bug to update and refactor the identification for this process after we have more information. Right now, I'd like to get this merged to close the P1 issue that's causing Markarth devices with an SSD to not boot.