On Sun, Apr 22, 2012 at 11:21:30PM +0200, Martin Jansa wrote:
> On Sat, Apr 21, 2012 at 04:28:27PM +0100, Stefan Schmidt wrote:
> > Hello.
> >
> > On Sat, 2012-04-21 at 11:05, benjamin deering wrote:
> > > Protect calls in bq27x00_external_power_changed with the mutex that
> > > is used to protect the same calls in other places in the driver.
> > > ---
> > > drivers/power/bq27x00_battery.c | 2 ++
> > > 1 files changed, 2 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/drivers/power/bq27x00_battery.c
> > > b/drivers/power/bq27x00_battery.c
> > > index 59e68db..346b489 100644
> > > --- a/drivers/power/bq27x00_battery.c
> > > +++ b/drivers/power/bq27x00_battery.c
> > > @@ -513,8 +513,10 @@ static void bq27x00_external_power_changed(struct
> > > power_supply *psy)
> > > {
> > > struct bq27x00_device_info *di = to_bq27x00_device_info(psy);
> > >
> > > + mutex_lock(&di->lock);
> > > cancel_delayed_work_sync(&di->work);
> > > schedule_delayed_work(&di->work, 0);
> > > + mutex_unlock(&di->lock);
> > > }
> > >
> > > static int bq27x00_powersupply_init(struct bq27x00_device_info *di)
> >
> > This one looks better. :)
> >
> > Reviewed-by: Stefan Schmidt <[email protected]>
>
> can you test it with shr-core?
>
> last 2 patches (now)
> http://git.shr-project.org/git/?p=meta-smartphone.git;a=shortlog;h=refs/heads/jansa/test
>
> on my FR I'm usually using 3.2, but if you can confirm it works, I'll
> push it to shr branch and also to shr-3.2-nodrm.Works with 3.2, so I'll push it now, will be in next staging. Thanks! Cheers, -- Martin 'JaMa' Jansa jabber: [email protected]
signature.asc
Description: Digital signature
_______________________________________________ Shr-devel mailing list [email protected] http://lists.shr-project.org/mailman/listinfo/shr-devel
