Messages in this thread | | | Subject | Re: [PATCH] regulator: lock supply in regulator enable | From | Liam Girdwood <> | Date | Thu, 04 Nov 2010 11:11:52 +0000 |
| |
On Thu, 2010-11-04 at 11:49 +0100, Mattias Wallin wrote: > As I wrote, the problem that I had is solved with my previous patch. > Right now I have no visible problem but I still think there is locks missing > and would like your opinion on it. > > /Wallin > > On 11/04/2010 11:18 AM, Liam Girdwood wrote: > > On Thu, 2010-11-04 at 11:01 +0100, Mattias Wallin wrote: > >> This patch add locks around regulator supply enable. > >> > >> Signed-off-by: Mattias Wallin <mattias.wallin@stericsson.com> > >> --- > >> The previous patch I sent solves a problem seen in our system. > >> This patch does not solve a problem I have seen but I still think > >> it should be there. Or at least some locking of the supply in regulator enable. > >> What do you guys think? > > > > This sounds like guesswork. What exactly is the problem in your system ? > >
Sorry, got a busy schedule atm. Can you give us your reasoning behind why you think we need a lock here ?
Thanks
Liam -- Freelance Developer, SlimLogic Ltd ASoC and Voltage Regulator Maintainer. http://www.slimlogic.co.uk
| |