RE: [lm-sensors] [PATCH] mfd: twl4030: Fix mismatch in mutex lock-unlock

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

 



> -----Original Message-----
> From: Jean Delvare [mailto:khali@xxxxxxxxxxxx] 
> Sent: Saturday, July 09, 2011 12:21 PM
> To: Premi, Sanjeev
> Cc: lm-sensors@xxxxxxxxxxxxxx; linux-omap@xxxxxxxxxxxxxxx
> Subject: Re: [lm-sensors] [PATCH] mfd: twl4030: Fix mismatch 
> in mutex lock-unlock
> 
> On Fri, 8 Jul 2011 18:27:59 +0530, Sanjeev Premi wrote:
> > A mutex is locked on entry into twl4030_madc_conversion().
> > Immediate return on some error conditions leaves the
> > mutex locked.
> > 
> > This patch ensures that mutex is always unlocked before
> > leaving the function.
> > 
> > Signed-off-by: Sanjeev Premi <premi@xxxxxx>
> > Cc: Keerthy <j-keerthy@xxxxxx>
> > ---
> > 
> >  Compile tested only.
> > 
> > 
> >  drivers/mfd/twl4030-madc.c |    8 ++++----
> >  1 files changed, 4 insertions(+), 4 deletions(-)
> > 
> > diff --git a/drivers/mfd/twl4030-madc.c b/drivers/mfd/twl4030-madc.c
> > index 3941ddc..b5d598c 100644
> > --- a/drivers/mfd/twl4030-madc.c
> > +++ b/drivers/mfd/twl4030-madc.c
> > (...)
> 
> Wrong list. This isn't a hwmon driver, so lm-sensors isn't suitable.
> scripts/get_maintainer.pl says that you should send this patch to:
> Samuel Ortiz <sameo@xxxxxxxxxxxxxxx>

[sp] I picked the list from the patch where this functionality was first
     introduced. Assumed that Samuel would on this list itself...

     sorry for noise :(

~sanjeev

> 
> Thanks,
> -- 
> Jean Delvare
> --
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux