Hi Maxime, On 5 July 2017 at 08:43, Maxime Ripard <maxime.rip...@free-electrons.com> wrote: > Hi, > > On Tue, Jul 04, 2017 at 01:31:28PM -0600, Simon Glass wrote: >> This is shown as active high in the schematics[1], so fix it. >> >> [1] https://patchwork.ozlabs.org/patch/777890/ >> >> Signed-off-by: Simon Glass <s...@chromium.org> >> Reported-by: Maxime Ripard <maxime.rip...@free-electrons.com> >> --- >> >> Changes in v2: >> - Add new patch to correct polarity of MMC card detect on Linksprite_pcDuino3 >> >> arch/arm/dts/sun7i-a20-pcduino3.dts | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arm/dts/sun7i-a20-pcduino3.dts >> b/arch/arm/dts/sun7i-a20-pcduino3.dts >> index 1a8b39be1d..37b1e0ee9b 100644 >> --- a/arch/arm/dts/sun7i-a20-pcduino3.dts >> +++ b/arch/arm/dts/sun7i-a20-pcduino3.dts >> @@ -164,7 +164,7 @@ >> pinctrl-0 = <&mmc0_pins_a>, <&mmc0_cd_pin_reference_design>; >> vmmc-supply = <®_vcc3v3>; >> bus-width = <4>; >> - cd-gpios = <&pio 7 1 GPIO_ACTIVE_HIGH>; /* PH1 */ >> + cd-gpios = <&pio 7 1 GPIO_ACTIVE_LOW>; /* PH1 */ > > Hmmm, are you sure? At least your commit log says otherwise :)
The commit message is wrong, unfortunately. The schematics suggest it is active low and it does seem to work with this setting. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
