Re: [PATCH] regulator: MAX8998: set_voltage bugfix. ramp_up delay and min/max voltage

2010-07-17 Thread Liam Girdwood
On Fri, 2010-07-16 at 17:09 +0900, MyungJoo Ham wrote: Two issues are addressed for max8998_set_voltage function. 1. Min/Max Voltage. max8998_set_voltage had been using the voltage value of min ( voltage[i] = max_vol , i ) This is corrected to use: min (

[PATCH] regulator: MAX8998: set_voltage bugfix. ramp_up delay and min/max voltage

2010-07-16 Thread MyungJoo Ham
Two issues are addressed for max8998_set_voltage function. 1. Min/Max Voltage. max8998_set_voltage had been using the voltage value of min ( voltage[i] = max_vol , i ) This is corrected to use: min ( voltage[i] = min_vol , i ) 2. Ramp Up Delay.

Re: [PATCH] regulator: MAX8998: set_voltage bugfix. ramp_up delay and min/max voltage

2010-07-16 Thread Mark Brown
On Fri, Jul 16, 2010 at 05:09:20PM +0900, MyungJoo Ham wrote: Two issues are addressed for max8998_set_voltage function. Acked-by: Mark Brown broo...@opensource.wolfsonmicro.com Ideally you should've submitted this as two separate patches. -- To unsubscribe from this list: send the line

RE: [PATCH] regulator: MAX8998: set_voltage bugfix. ramp_up delay and min/max voltage

2010-07-16 Thread Marek Szyprowski
Hello, On Friday, July 16, 2010 10:09 AM MyungJoo Ham wrote: Two issues are addressed for max8998_set_voltage function. 1. Min/Max Voltage. max8998_set_voltage had been using the voltage value of min ( voltage[i] = max_vol , i ) This is corrected to use: