Hi Mark, > -----Original Message----- > From: Mark Brown [mailto:broo...@opensource.wolfsonmicro.com] > Sent: Friday, February 08, 2013 4:59 PM > To: J, KEERTHY > Cc: linux-kernel@vger.kernel.org; lgirdw...@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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
