diff options
author | Tuomas Tynkkynen <tuomas@tuxera.com> | 2017-12-23 00:30:56 +0200 |
---|---|---|
committer | Jagan Teki <jagan@amarulasolutions.com> | 2018-01-10 11:37:09 +0530 |
commit | 34439f73d92171cff478bd454cd812a5a50fbdd0 (patch) | |
tree | d594712a27c07cdef0a1e2175ac9b2e961fd453f /arch/arm/dts/sun7i-a20-pcduino3.dts | |
parent | 3afdfd8ddc0a90e76c0a33be4352293c95f3be2a (diff) |
ARM: sunxi: Remove left-over cd-inverted property from pcDuino3
Commit 8620f384098b ("dm: sunxi: Linksprite_pcDuino3: Correct polarity
of MMC card detect") claims that the Pcduino3 device tree had an
incorrect polarity for the card detect pin and thus changed the polarity
flag of the cd-gpios from GPIO_ACTIVE_HIGH to GPIO_ACTIVE_LOW.
Actually the DT was correct since according to the mmc binding, a
combination of GPIO_ACTIVE_HIGH + cd-inverted results in an active-low
polarity. But because the U-Boot driver lacks the code to look at the
cd-inverted property (unlike the Linux driver) it interpreted the
polarity of active-high. Thus, after that commit the DT is actually
wrong from the binding/Linux point of view.
To make both Linux and U-Boot interpret the DT in the same way, just
drop the left-over cd-inverted property. I've sent a Linux patch to
switch all sunxi DTs over to not using the cd-inverted property, so
eventually all sunxi boards in U-Boot will be consistent in not using
cd-inverted.
Signed-off-by: Tuomas Tynkkynen <tuomas@tuxera.com>
Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Reviewed-by: Jagan Teki <jagan@amarulasolutions.com>
Diffstat (limited to 'arch/arm/dts/sun7i-a20-pcduino3.dts')
-rw-r--r-- | arch/arm/dts/sun7i-a20-pcduino3.dts | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/arch/arm/dts/sun7i-a20-pcduino3.dts b/arch/arm/dts/sun7i-a20-pcduino3.dts index 37b1e0ee9b..7f449b6a28 100644 --- a/arch/arm/dts/sun7i-a20-pcduino3.dts +++ b/arch/arm/dts/sun7i-a20-pcduino3.dts @@ -165,7 +165,6 @@ vmmc-supply = <®_vcc3v3>; bus-width = <4>; cd-gpios = <&pio 7 1 GPIO_ACTIVE_LOW>; /* PH1 */ - cd-inverted; status = "okay"; }; |