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

Stefan Wahren stefan.wahren at i2se.com
Wed May 17 10:52:59 PDT 2017


> Fabio Estevam <festevam at gmail.com> hat am 17. Mai 2017 um 19:06 geschrieben:
> 
> 
> 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.
> 

Thanks for this explanation, this wasn't clear to me from the subject / commit log.



More information about the linux-arm-kernel mailing list