Re: [PATCH RFT] regulator: ab8500: Set enable enable_time in regulator_desc

2012-08-07 Thread Mark Brown
On Tue, Aug 07, 2012 at 11:32:27AM +0200, Linus Walleij wrote: > I remember this now. Only that one regulator has a non-negligeble > rise time, so for that reason it's set to that value. Atleast so I was > told once upon a time when I implemented it, so this patch > should be fine. Please resend

Re: [PATCH RFT] regulator: ab8500: Set enable enable_time in regulator_desc

2012-08-07 Thread Linus Walleij
On Mon, Aug 6, 2012 at 5:50 PM, Linus Walleij wrote: > On Mon, Aug 6, 2012 at 5:46 PM, Axel Lin wrote: >> 2012/7/4 Mark Brown : >>> On Wed, Jul 04, 2012 at 10:31:45AM +0800, Axel Lin wrote: Signed-off-by: Axel Lin --- Hi Linus, While working on this patch, I found in original

Re: [PATCH RFT] regulator: ab8500: Set enable enable_time in regulator_desc

2012-08-06 Thread Linus Walleij
On Mon, Aug 6, 2012 at 5:46 PM, Axel Lin wrote: > 2012/7/4 Mark Brown : >> On Wed, Jul 04, 2012 at 10:31:45AM +0800, Axel Lin wrote: >>> Signed-off-by: Axel Lin >>> --- >>> Hi Linus, >>> While working on this patch, I found in original code: >>> ab8500_regulator_ops set enable_time callback to >

Re: [PATCH RFT] regulator: ab8500: Set enable enable_time in regulator_desc

2012-08-06 Thread Axel Lin
2012/7/4 Mark Brown : > On Wed, Jul 04, 2012 at 10:31:45AM +0800, Axel Lin wrote: >> Signed-off-by: Axel Lin >> --- >> Hi Linus, >> While working on this patch, I found in original code: >> ab8500_regulator_ops set enable_time callback to >> ab8500_regulator_enable_time, >> but does not set the d