Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-24 Thread Mark Brown
On Thu, Mar 23, 2017 at 02:40:16PM -0700, Matthias Kaehlcke wrote: > El Mon, Mar 20, 2017 at 12:06:15PM + Mark Brown ha dit: > > On Fri, Mar 17, 2017 at 05:03:30PM -0700, Matthias Kaehlcke wrote: > > > You are right that my case is very specialist, however I think it is > > > a general

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-24 Thread Mark Brown
On Thu, Mar 23, 2017 at 02:40:16PM -0700, Matthias Kaehlcke wrote: > El Mon, Mar 20, 2017 at 12:06:15PM + Mark Brown ha dit: > > On Fri, Mar 17, 2017 at 05:03:30PM -0700, Matthias Kaehlcke wrote: > > > You are right that my case is very specialist, however I think it is > > > a general

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-23 Thread Matthias Kaehlcke
El Mon, Mar 20, 2017 at 12:06:15PM + Mark Brown ha dit: > On Fri, Mar 17, 2017 at 05:03:30PM -0700, Matthias Kaehlcke wrote: > > > In principle I totally agree with you that consumers should be able > > to enumerate the supported voltages with the existing functions. And > > they can, as

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-23 Thread Matthias Kaehlcke
El Mon, Mar 20, 2017 at 12:06:15PM + Mark Brown ha dit: > On Fri, Mar 17, 2017 at 05:03:30PM -0700, Matthias Kaehlcke wrote: > > > In principle I totally agree with you that consumers should be able > > to enumerate the supported voltages with the existing functions. And > > they can, as

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-20 Thread Mark Brown
On Fri, Mar 17, 2017 at 05:03:30PM -0700, Matthias Kaehlcke wrote: > In principle I totally agree with you that consumers should be able > to enumerate the supported voltages with the existing functions. And > they can, as long as they already know (or assume) that the regulator > they are using

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-20 Thread Mark Brown
On Fri, Mar 17, 2017 at 05:03:30PM -0700, Matthias Kaehlcke wrote: > In principle I totally agree with you that consumers should be able > to enumerate the supported voltages with the existing functions. And > they can, as long as they already know (or assume) that the regulator > they are using

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-17 Thread Matthias Kaehlcke
Hi Mark, El Fri, Mar 17, 2017 at 09:15:47PM + Mark Brown ha dit: > On Thu, Mar 09, 2017 at 11:40:54AM -0800, Matthias Kaehlcke wrote: > > El Thu, Mar 09, 2017 at 11:28:19AM +0100 Mark Brown ha dit: > > > On Wed, Mar 08, 2017 at 12:02:45PM -0800, Matthias Kaehlcke wrote: > > > > > The new

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-17 Thread Matthias Kaehlcke
Hi Mark, El Fri, Mar 17, 2017 at 09:15:47PM + Mark Brown ha dit: > On Thu, Mar 09, 2017 at 11:40:54AM -0800, Matthias Kaehlcke wrote: > > El Thu, Mar 09, 2017 at 11:28:19AM +0100 Mark Brown ha dit: > > > On Wed, Mar 08, 2017 at 12:02:45PM -0800, Matthias Kaehlcke wrote: > > > > > The new

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-17 Thread Mark Brown
On Thu, Mar 09, 2017 at 11:40:54AM -0800, Matthias Kaehlcke wrote: > El Thu, Mar 09, 2017 at 11:28:19AM +0100 Mark Brown ha dit: > > On Wed, Mar 08, 2017 at 12:02:45PM -0800, Matthias Kaehlcke wrote: > > > The new function allows consumers to determine if a regulator is > > > continuous or

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-17 Thread Mark Brown
On Thu, Mar 09, 2017 at 11:40:54AM -0800, Matthias Kaehlcke wrote: > El Thu, Mar 09, 2017 at 11:28:19AM +0100 Mark Brown ha dit: > > On Wed, Mar 08, 2017 at 12:02:45PM -0800, Matthias Kaehlcke wrote: > > > The new function allows consumers to determine if a regulator is > > > continuous or

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-09 Thread Matthias Kaehlcke
Hi Mark, El Thu, Mar 09, 2017 at 11:28:19AM +0100 Mark Brown ha dit: > On Wed, Mar 08, 2017 at 12:02:45PM -0800, Matthias Kaehlcke wrote: > > > The new function allows consumers to determine if a regulator is > > continuous or discrete, and whether the results of > > Why? As we discussed at

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-09 Thread Matthias Kaehlcke
Hi Mark, El Thu, Mar 09, 2017 at 11:28:19AM +0100 Mark Brown ha dit: > On Wed, Mar 08, 2017 at 12:02:45PM -0800, Matthias Kaehlcke wrote: > > > The new function allows consumers to determine if a regulator is > > continuous or discrete, and whether the results of > > Why? As we discussed at

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-09 Thread Mark Brown
On Wed, Mar 08, 2017 at 12:02:45PM -0800, Matthias Kaehlcke wrote: > The new function allows consumers to determine if a regulator is > continuous or discrete, and whether the results of Why? As we discussed at ELC this is an implementation detail of the regulator and it's to an extent a taste

Re: [PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-09 Thread Mark Brown
On Wed, Mar 08, 2017 at 12:02:45PM -0800, Matthias Kaehlcke wrote: > The new function allows consumers to determine if a regulator is > continuous or discrete, and whether the results of Why? As we discussed at ELC this is an implementation detail of the regulator and it's to an extent a taste

[PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-08 Thread Matthias Kaehlcke
The new function allows consumers to determine if a regulator is continuous or discrete, and whether the results of regulator_count_voltages() and regulator_list_voltage() correspond to the regulator itself or its supply. Change-Id: I1198cee9fff60dc747a02860e9652034f4d5da33 Signed-off-by:

[PATCH v2 1/2] regulator: core: add regulator_has_continuous_voltage_range()

2017-03-08 Thread Matthias Kaehlcke
The new function allows consumers to determine if a regulator is continuous or discrete, and whether the results of regulator_count_voltages() and regulator_list_voltage() correspond to the regulator itself or its supply. Change-Id: I1198cee9fff60dc747a02860e9652034f4d5da33 Signed-off-by: