[PATCH V2 5/6] regulator: add mxs on-chip regulator driver
stefan.wahren at i2se.com
Thu May 7 12:25:19 PDT 2015
> Mark Brown <broonie at kernel.org> hat am 7. Mai 2015 um 20:52 geschrieben:
> On Thu, May 07, 2015 at 08:37:18PM +0200, Stefan Wahren wrote:
> > > Stefan Wahren <stefan.wahren at i2se.com> hat am 4. Mai 2015 um 22:34
> > > i'm sorry, i forgot the following point in the changelog:
> > > - move dcdc handling from mxs_power to mxs-regulator
> > > I hope the change is okay. I implemented only is_enabled() to have at
> > > least 1
> > > operation.
> > > But it's not really necessary.
> > do we need enable / disable operation?
> > In the bootloader code enabling the DC-DC is critical. There is a known
> > issue
> > which falsely triggers the brownout detection.
> Why would the driver not provide these operations?
I didn't implement this operation because the suggested workaround for the
hardware issue isn't very nice
(disable brownout detection during enabling DC-DC). I was afraid of the
> > > > I'm also not 100% clear what code the DCDCs and LDOs are
> > > > sharing here...
> > > Actually only probing and the regulator info structure.
> > How can i improve that?
> If the devices aren't sharing code then they should be separate drivers.
What do you think of?
More information about the linux-arm-kernel