Messages in this thread | | | From | "J, KEERTHY" <> | Subject | RE: [PATCH] Regulator: Reorder the min max assignment in the sequence of regulator_set_voltage function | Date | Mon, 11 Feb 2013 05:26:20 +0000 |
| |
Hi Mark,
> -----Original Message----- > From: Mark Brown [mailto:broonie@opensource.wolfsonmicro.com] > Sent: Friday, February 08, 2013 4:59 PM > To: J, KEERTHY > Cc: linux-kernel@vger.kernel.org; lgirdwood@gmail.com > Subject: Re: [PATCH] Regulator: Reorder the min max assignment in the > sequence of regulator_set_voltage function > > On Fri, Feb 01, 2013 at 10:34:58AM +0530, J Keerthy wrote: > > > +++ b/drivers/regulator/core.c > > @@ -81,6 +81,8 @@ struct regulator { > > int uA_load; > > int min_uV; > > int max_uV; > > + int curr_min_uV; > > + int curr_max_uV; > > This change makes sense but I don't see why we need to add these to > struct regulator - why not just store the values locally for unwinding?
Ok. I will send an updated patch. Thanks for the response And sorry for the top posting.
Regards, Keerthy
| |