Re: [PATCH 1/2] regulator: Put mutex lock down to mc13783_powermisc_rmw function

[Axel Lin <axel.lin@gmail.com>]

Hi Mark,
2011/5/4 Mark Brown <broonie@opensource.wolfsonmicro.com>:
> On Wed, May 04, 2011 at 12:40:30AM +0800, Axel Lin wrote:
>> mc13783_powermisc_rmw function is doing read/modify/write bitmask operations,
>> thus add the lock to protect it.
>> Then we can remove the lock/unlock from the caller.
>
> This should be squashed in with the MFD patch I think otherwise we'll
> get bisection points with either no lock or double locking (which upsets
> lockdep IIRC).  The change itself looks good, though, and similarly for
> patch 2
>

I think these 2 patches are independent from the MFD patch.

The MFD patch modified the mc13xxx_reg_rmw which is not called by
mc13783_powermisc_rmw and mc13783_powermisc_rmw functions.
I double checked the usage of mc13783_powermisc_rmw and mc13783_powermisc_rmw,
I think these 2 patches can be applied either with or without the MFD patch.

Regards,
Axel