From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCH V3 07/16] PM / OPP: Add support to parse "operating-points-v2" bindings Date: Thu, 30 Jul 2015 23:20:13 -0700 Message-ID: <20150731062013.GR3159@codeaurora.org> References: <6d8d9cd0c3ae2d3fa8c3aadedb5ac403785b7675.1438166099.git.viresh.kumar@linaro.org> <20150731055106.GJ3159@codeaurora.org> <20150731055806.GN17794@linux> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20150731055806.GN17794@linux> Sender: linux-kernel-owner@vger.kernel.org To: Viresh Kumar Cc: Rafael Wysocki , linaro-kernel@lists.linaro.org, linux-pm@vger.kernel.org, rob.herring@linaro.org, arnd.bergmann@linaro.org, nm@ti.com, broonie@kernel.org, mturquette@baylibre.com, Sudeep.Holla@arm.com, viswanath.puttagunta@linaro.org, l.stach@pengutronix.de, thomas.petazzoni@free-electrons.com, linux-arm-kernel@lists.infradead.org, ta.omasab@gmail.com, kesavan.abhilash@gmail.com, khilman@linaro.org, santosh.shilimkar@oracle.com, b.zolnierkie@samsung.com, Greg Kroah-Hartman , Len Brown , open list , Pavel Machek List-Id: linux-pm@vger.kernel.org On 07/31, Viresh Kumar wrote: > On 30-07-15, 22:51, Stephen Boyd wrote: > > > + opp->u_volt = microvolt[0]; > > > + opp->u_volt_min = microvolt[1]; > > > + opp->u_volt_max = microvolt[2]; > > > > Should the default be 0 and ULONG_MAX for volt_min/volt_max when > > there's on element? > > I am not still sure how the regulator API is going to look like for > this target/min/max thing. So, lets defer it until that is resolved. > For now they are initialized to 0. > > And, because the user has just passed in a target voltage, maybe all > three must be == u_volt. :) > Ok. Sounds fair enough. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project