Re: [PATCH 1/1] as3645a: Remove set_power() from platform data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Laurent,

On Wed, May 23, 2012 at 01:00:08PM +0200, Laurent Pinchart wrote:
> Hi Sakari,
> 
> Thanks for the patch.
> 
> On Wednesday 16 May 2012 06:12:49 Sakari Ailus wrote:
> > The chip is typically powered constantly and no board uses the set_power()
> > callback. Remove it.
> > 
> > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxx>
> > ---
> >  drivers/media/video/as3645a.c |   39 +++++++++----------------------------
> >  include/media/as3645a.h       |    1 -
> >  2 files changed, 9 insertions(+), 31 deletions(-)
> > 
> > diff --git a/drivers/media/video/as3645a.c b/drivers/media/video/as3645a.c
> > index c4b0357..7454660 100644
> > --- a/drivers/media/video/as3645a.c
> > +++ b/drivers/media/video/as3645a.c
> > @@ -512,31 +512,6 @@ static int as3645a_setup(struct as3645a *flash)
> >  	return ret & ~AS_FAULT_INFO_LED_AMOUNT ? -EIO : 0;
> >  }
> > 
> > -static int __as3645a_set_power(struct as3645a *flash, int on)
> > -{
> > -	int ret;
> > -
> > -	if (!on)
> > -		as3645a_set_control(flash, AS_MODE_EXT_TORCH, false);
> > -
> > -	if (flash->pdata->set_power) {
> > -		ret = flash->pdata->set_power(&flash->subdev, on);
> > -		if (ret < 0)
> > -			return ret;
> > -	}
> > -
> > -	if (!on)
> > -		return 0;
> > -
> > -	ret = as3645a_setup(flash);
> > -	if (ret < 0) {
> > -		if (flash->pdata->set_power)
> > -			flash->pdata->set_power(&flash->subdev, 0);
> > -	}
> > -
> > -	return ret;
> > -}
> > -
> >  static int as3645a_set_power(struct v4l2_subdev *sd, int on)
> >  {
> >  	struct as3645a *flash = to_as3645a(sd);
> > @@ -545,9 +520,13 @@ static int as3645a_set_power(struct v4l2_subdev *sd,
> > int on) mutex_lock(&flash->power_lock);
> > 
> >  	if (flash->power_count == !on) {
> > -		ret = __as3645a_set_power(flash, !!on);
> > -		if (ret < 0)
> > -			goto done;
> > +		if (!on) {
> > +			as3645a_set_control(flash, AS_MODE_EXT_TORCH, false);
> > +		} else {
> > +			ret = as3645a_setup(flash);
> > +			if (ret < 0)
> > +				goto done;
> > +		}
> >  	}
> > 
> 
> If the chip is powered on constantly, why do we need a .s_power() subdev 
> operation at all ?

I don't know why was it there in the first place. Probably to make it easier
to use the driver on boards that required e.g. a regulator for the chip.

But typically they're connected to battery directly. The idle power
consumption is just some tens of µA.

Cheers,

-- 
Sakari Ailus
e-mail: sakari.ailus@xxxxxx	jabber/XMPP/Gmail: sailus@xxxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux