[PATCH] ARM: pcm037: convert custom GPIO-based power function to a regulator
Guennadi Liakhovetski
g.liakhovetski at gmx.de
Tue Nov 5 16:56:58 EST 2013
Hi Sascha,
On Mon, 4 Nov 2013, Sascha Hauer wrote:
> Hi Guennadi,
>
> On Thu, Oct 31, 2013 at 11:20:46AM +0100, Guennadi Liakhovetski wrote:
> > Add a fixed-voltage GPIO-enabled regulator to switch the camera on and off
> > instead of using a .power() platform callback.
> >
> > Signed-off-by: Guennadi Liakhovetski <g.liakhovetski at gmx.de>
> > ---
> >
> > Only tested to grab the regulator during probing, couldn't test
> > functionality due to the lack of an mt9t031 sensor.
> >
> > arch/arm/mach-imx/mach-pcm037.c | 50 ++++++++++++++++++++++++++++-----------
> > 1 files changed, 36 insertions(+), 14 deletions(-)
>
> What's the motivation for this patch? It probably does the right thing,
> but I think we need a plan for converting this to devicetree rather than
> adding more stuff.
It's not adding stuff. It's getting rid of a custom callback and replacing
it with a proper regulator. Sure, if there's a plan to port this platform
to DT in a foreseeable future - feel free to drop this. But such a port
would require adding DT support to the mx3_camera driver / the soc-camera
framework, which might take some time, and this board isn't too new
either, so, maybe not many customers will be prepared to pay for that kind
of work ;) Anyway, I find this useful in its present form, but if there
are plans to switch to DT soon enough - no problem with dropping it
either.
Thanks
Guennadi
> Sascha
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
More information about the linux-arm-kernel
mailing list