Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30545 ) Change subject: mb/google/hatch: Enable SD card support for hatch ...................................................................... Patch Set 20: (2 comments) https://review.coreboot.org/#/c/30545/3/src/mainboard/google/hatch/variants/... File src/mainboard/google/hatch/variants/baseboard/gpio.c: https://review.coreboot.org/#/c/30545/3/src/mainboard/google/hatch/variants/... PS3, Line 106: };
Yes that was the case earlier, if you take a look at Dragonegg we configure only required GPIOs […] I have included all GPIOs related to SD card in this patch itself.
https://review.coreboot.org/#/c/30545/18/src/mainboard/google/hatch/variants... File src/mainboard/google/hatch/variants/baseboard/gpio.c: https://review.coreboot.org/#/c/30545/18/src/mainboard/google/hatch/variants... PS18, Line 103: UP_20K
Let's test the change properly and then push it in. I have tested changes locally on RVP board and SD card detection works without pull up. So I have removed pull up
-- To view, visit https://review.coreboot.org/c/coreboot/+/30545 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I8ccaa28323b1e1fcc192e245347a96309227660b Gerrit-Change-Number: 30545 Gerrit-PatchSet: 20 Gerrit-Owner: Maulik V Vaghela <maulik.v.vaghela@intel.com> Gerrit-Reviewer: Aamir Bohra <aamir.bohra@intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Furquan Shaikh <furquan@google.com> Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela@intel.com> Gerrit-Reviewer: Naresh Solanki <naresh.solanki@intel.com> Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi@intel.com> Gerrit-Reviewer: Ronak Kanabar <ronak.kanabar@intel.com> Gerrit-Reviewer: Shelley Chen <shchen@google.com> Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-Comment-Date: Fri, 18 Jan 2019 04:38:04 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Maulik V Vaghela <maulik.v.vaghela@intel.com> Comment-In-Reply-To: Rizwan Qureshi <rizwan.qureshi@intel.com> Comment-In-Reply-To: Furquan Shaikh <furquan@google.com> Gerrit-MessageType: comment