]> git.baikalelectronics.ru Git - kernel.git/commit
ARM: nomadik: fix up double inversion in DT
authorLinus Walleij <linus.walleij@linaro.org>
Fri, 25 Jul 2014 10:18:42 +0000 (12:18 +0200)
committerOlof Johansson <olof@lixom.net>
Wed, 30 Jul 2014 19:47:17 +0000 (12:47 -0700)
commit9a45ba452fe97a8650b808004c162415b160542d
tree22a00c3af5f3c265cc5b382729a7d76a5bc93dca
parente9f20178ff2f52b848a9f3ce72ff7d70703eb6d0
ARM: nomadik: fix up double inversion in DT

The GPIO pin connected to card detect was inverted twice: once by
the argument to the GPIO line itself where it was magically marked
as active low by the flag GPIO_ACTIVE_LOW (0x01) in the third cell,
and also marked active low AGAIN by explicitly stating
"cd-inverted" (a deprecated method).

After commit 424dea917fe489377aaf6eb0923b5c91f7699ef7
"mmc: mmci: Use the common mmc DT parser" this results in the
line being inverted twice so it was effectively uninverted, while
the old code would not have this effect, instead disregarding the
flag on the GPIO line altogether, which is a bug. I admit the
semantics may be unclear but inverting twice is as good a
definition as any on how this should work.

So fix up the buggy device tree. Use proper #includes so the DTS
is clear and readable.

Cc: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Olof Johansson <olof@lixom.net>
arch/arm/boot/dts/ste-nomadik-s8815.dts
arch/arm/boot/dts/ste-nomadik-stn8815.dtsi