Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-27 Thread Matthias Kaehlcke
El Mon, Mar 27, 2017 at 01:54:50PM -0400 Javier Martinez Canillas ha dit: > On 03/27/2017 01:39 PM, Matthias Kaehlcke wrote: > > +if (ops->get_voltage || ops->get_voltage_sel) > >> > >> It's valid to have a .get_voltage_sel callback without a .list_voltage? > >> > >> At least

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-27 Thread Matthias Kaehlcke
El Mon, Mar 27, 2017 at 01:54:50PM -0400 Javier Martinez Canillas ha dit: > On 03/27/2017 01:39 PM, Matthias Kaehlcke wrote: > > +if (ops->get_voltage || ops->get_voltage_sel) > >> > >> It's valid to have a .get_voltage_sel callback without a .list_voltage? > >> > >> At least

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-27 Thread Mark Brown
On Mon, Mar 27, 2017 at 01:54:50PM -0400, Javier Martinez Canillas wrote: > On 03/27/2017 01:39 PM, Matthias Kaehlcke wrote: > > I don't know for sure if this condition is superfluous or if there are > > cases where it makes sense to have a .list_voltage but not > > .get_voltage_sel. > I don't

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-27 Thread Mark Brown
On Mon, Mar 27, 2017 at 01:54:50PM -0400, Javier Martinez Canillas wrote: > On 03/27/2017 01:39 PM, Matthias Kaehlcke wrote: > > I don't know for sure if this condition is superfluous or if there are > > cases where it makes sense to have a .list_voltage but not > > .get_voltage_sel. > I don't

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-27 Thread Javier Martinez Canillas
Hello Matthias, On 03/27/2017 01:39 PM, Matthias Kaehlcke wrote: > Thanks for the reviews and testing! > You are welcome. [snip] + if (ops->get_voltage || ops->get_voltage_sel) >> >> It's valid to have a .get_voltage_sel callback without a .list_voltage? >> >> At least it seems

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-27 Thread Javier Martinez Canillas
Hello Matthias, On 03/27/2017 01:39 PM, Matthias Kaehlcke wrote: > Thanks for the reviews and testing! > You are welcome. [snip] + if (ops->get_voltage || ops->get_voltage_sel) >> >> It's valid to have a .get_voltage_sel callback without a .list_voltage? >> >> At least it seems

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-27 Thread Matthias Kaehlcke
Thanks for the reviews and testing! El Sat, Mar 25, 2017 at 02:05:47AM -0300 Javier Martinez Canillas ha dit: On 03/24/2017 05:38 PM, Brian Norris wrote: > > On Fri, Mar 24, 2017 at 01:09:52PM -0700, Matthias Kaehlcke wrote: > >> diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-27 Thread Matthias Kaehlcke
Thanks for the reviews and testing! El Sat, Mar 25, 2017 at 02:05:47AM -0300 Javier Martinez Canillas ha dit: On 03/24/2017 05:38 PM, Brian Norris wrote: > > On Fri, Mar 24, 2017 at 01:09:52PM -0700, Matthias Kaehlcke wrote: > >> diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-27 Thread Mark Brown
On Sat, Mar 25, 2017 at 02:05:47AM -0300, Javier Martinez Canillas wrote: > On 03/24/2017 05:38 PM, Brian Norris wrote: > > > >> + if (ops->get_voltage || ops->get_voltage_sel) > It's valid to have a .get_voltage_sel callback without a .list_voltage? No. signature.asc Description:

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-27 Thread Mark Brown
On Sat, Mar 25, 2017 at 02:05:47AM -0300, Javier Martinez Canillas wrote: > On 03/24/2017 05:38 PM, Brian Norris wrote: > > > >> + if (ops->get_voltage || ops->get_voltage_sel) > It's valid to have a .get_voltage_sel callback without a .list_voltage? No. signature.asc Description:

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-24 Thread Javier Martinez Canillas
Hello Matthias, On 03/24/2017 05:38 PM, Brian Norris wrote: > On Fri, Mar 24, 2017 at 01:09:52PM -0700, Matthias Kaehlcke wrote: >> diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c >> index 53d4fc70dbd0..121838e0125b 100644 >> --- a/drivers/regulator/core.c >> +++

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-24 Thread Javier Martinez Canillas
Hello Matthias, On 03/24/2017 05:38 PM, Brian Norris wrote: > On Fri, Mar 24, 2017 at 01:09:52PM -0700, Matthias Kaehlcke wrote: >> diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c >> index 53d4fc70dbd0..121838e0125b 100644 >> --- a/drivers/regulator/core.c >> +++

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-24 Thread Brian Norris
On Fri, Mar 24, 2017 at 01:09:52PM -0700, Matthias Kaehlcke wrote: > diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c > index 53d4fc70dbd0..121838e0125b 100644 > --- a/drivers/regulator/core.c > +++ b/drivers/regulator/core.c > @@ -2487,6 +2487,10 @@ static int

Re: [PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-24 Thread Brian Norris
On Fri, Mar 24, 2017 at 01:09:52PM -0700, Matthias Kaehlcke wrote: > diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c > index 53d4fc70dbd0..121838e0125b 100644 > --- a/drivers/regulator/core.c > +++ b/drivers/regulator/core.c > @@ -2487,6 +2487,10 @@ static int

[PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-24 Thread Matthias Kaehlcke
Commit 26988efe11b1 ("regulator: core: Allow to get voltage count and list from parent") introduces the propagation of the parent voltage count and list for regulators that don't provide this information themselves. The goal is to support simple switch regulators, however as a side effect normal

[PATCH] regulator: core: Limit propagation of parent voltage count and list

2017-03-24 Thread Matthias Kaehlcke
Commit 26988efe11b1 ("regulator: core: Allow to get voltage count and list from parent") introduces the propagation of the parent voltage count and list for regulators that don't provide this information themselves. The goal is to support simple switch regulators, however as a side effect normal