Re: [PATCH RFC/RFT] regulator: core: Fix continuous_voltage_range case in regulator_can_change_voltage

2012-12-24 Thread Mark Brown
On Wed, Dec 12, 2012 at 09:22:46AM +0800, Axel Lin wrote: > Regulator drivers with continuous_voltage_range flag set allows not setting > n_voltages. Thus if continuous_voltage_range is set, check the constraint > range > instead. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH RFC/RFT] regulator: core: Fix continuous_voltage_range case in regulator_can_change_voltage

2012-12-24 Thread Mark Brown
On Wed, Dec 12, 2012 at 09:22:46AM +0800, Axel Lin wrote: Regulator drivers with continuous_voltage_range flag set allows not setting n_voltages. Thus if continuous_voltage_range is set, check the constraint range instead. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH RFC/RFT] regulator: core: Fix continuous_voltage_range case in regulator_can_change_voltage

2012-12-13 Thread Mark Brown
On Thu, Dec 13, 2012 at 09:56:25AM +0100, Marek Szyprowski wrote: > This change looks reasonable, I've missed the fact that there is a > support for regulators with continuous voltage range. This change also > shows that the n_voltages based workaround used earlier by mmc core was > indeed only a

Re: [PATCH RFC/RFT] regulator: core: Fix continuous_voltage_range case in regulator_can_change_voltage

2012-12-13 Thread Marek Szyprowski
Hello, On 12/12/2012 2:22 AM, Axel Lin wrote: Regulator drivers with continuous_voltage_range flag set allows not setting n_voltages. Thus if continuous_voltage_range is set, check the constraint range instead. Signed-off-by: Axel Lin --- drivers/regulator/core.c | 12 +--- 1

Re: [PATCH RFC/RFT] regulator: core: Fix continuous_voltage_range case in regulator_can_change_voltage

2012-12-13 Thread Marek Szyprowski
Hello, On 12/12/2012 2:22 AM, Axel Lin wrote: Regulator drivers with continuous_voltage_range flag set allows not setting n_voltages. Thus if continuous_voltage_range is set, check the constraint range instead. Signed-off-by: Axel Lin axel@ingics.com --- drivers/regulator/core.c | 12

Re: [PATCH RFC/RFT] regulator: core: Fix continuous_voltage_range case in regulator_can_change_voltage

2012-12-13 Thread Mark Brown
On Thu, Dec 13, 2012 at 09:56:25AM +0100, Marek Szyprowski wrote: This change looks reasonable, I've missed the fact that there is a support for regulators with continuous voltage range. This change also shows that the n_voltages based workaround used earlier by mmc core was indeed only a

[PATCH RFC/RFT] regulator: core: Fix continuous_voltage_range case in regulator_can_change_voltage

2012-12-11 Thread Axel Lin
Regulator drivers with continuous_voltage_range flag set allows not setting n_voltages. Thus if continuous_voltage_range is set, check the constraint range instead. Signed-off-by: Axel Lin --- drivers/regulator/core.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff

[PATCH RFC/RFT] regulator: core: Fix continuous_voltage_range case in regulator_can_change_voltage

2012-12-11 Thread Axel Lin
Regulator drivers with continuous_voltage_range flag set allows not setting n_voltages. Thus if continuous_voltage_range is set, check the constraint range instead. Signed-off-by: Axel Lin axel@ingics.com --- drivers/regulator/core.c | 12 +--- 1 file changed, 9 insertions(+), 3