[PATCH 1/2] ARM: dts: imx23-sansa: Remove 'enable-active-low' property

Fabio Estevam festevam at gmail.com
Wed May 17 10:06:56 PDT 2017


Hi Stefan,

On Wed, May 17, 2017 at 1:59 PM, Stefan Wahren <stefan.wahren at i2se.com> wrote:

> GPIO_ACTIVE_LOW = 1, GPIO_ACTIVE_HIGH = 0

Correct.

> So this doesn't look save to me.

gpio regulator driver does not look for the gpio flag field, so this
change is safe.

That's the reason there is a property called 'enable-active-high'.

When 'enable-active-high' is passed the gpio regulator is active high.
If the property is absent then it is active low.

This is explained in
Documentation/devicetree/bindings/regulator/fixed-regulator.txt

Passing GPIO_ACTIVE_LOW describes the gpio regulator correctly in this case.



More information about the linux-arm-kernel mailing list