[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede: Add SD card support

[Aamir Bohra (Code Review)]

Aamir Bohra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39237 )

Change subject: mb/google/dedede: Add SD card support
......................................................................

Patch Set 28:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39237/23/src/mainboard/google/deded... 
File src/mainboard/google/dedede/variants/baseboard/gpio.c:

https://review.coreboot.org/c/coreboot/+/39237/23/src/mainboard/google/deded... 
PS23, Line 284: GPP_G7
> Thanks Aamir! I still don't understand why the WP creates problems only at init time. […]

ok, yes. based on current observations the NF configuration seems to be must for resolving the dmesg errors.

Below are still open:

1. Does controller expects to be configured WP as NF, even if not connected.
2. Why are tranfer request not getting completed and card status is set busy with NC configuration.

We can update the bug based on the findings.