Re: Would devm_regulator_enable be useful ?

2014-02-04 Thread Mark Brown
On Tue, Feb 04, 2014 at 06:22:30AM -0800, Guenter Roeck wrote: > On 02/04/2014 03:10 AM, Mark Brown wrote: > >Sort of. They're there but that doesn't mean that they should be used > >in normal operation - they should be special cases, not normal things. > >Managed resources are supposed to for

Re: Would devm_regulator_enable be useful ?

2014-02-04 Thread Guenter Roeck
On 02/04/2014 03:10 AM, Mark Brown wrote: On Mon, Feb 03, 2014 at 02:27:26PM -0800, Guenter Roeck wrote: On Mon, Feb 03, 2014 at 06:21:52PM +, Mark Brown wrote: As previously mentioned please fix your mailer to word wrap at a sensible limit. I thought I did ;-). I'll try to make sure

Re: Would devm_regulator_enable be useful ?

2014-02-04 Thread Mark Brown
On Mon, Feb 03, 2014 at 02:27:26PM -0800, Guenter Roeck wrote: > On Mon, Feb 03, 2014 at 06:21:52PM +, Mark Brown wrote: > > As previously mentioned please fix your mailer to word wrap at a > > sensible limit. > I thought I did ;-). I'll try to make sure I only send e-mail to you > using

Re: Would devm_regulator_enable be useful ?

2014-02-04 Thread Mark Brown
On Mon, Feb 03, 2014 at 02:27:26PM -0800, Guenter Roeck wrote: On Mon, Feb 03, 2014 at 06:21:52PM +, Mark Brown wrote: As previously mentioned please fix your mailer to word wrap at a sensible limit. I thought I did ;-). I'll try to make sure I only send e-mail to you using mutt in

Re: Would devm_regulator_enable be useful ?

2014-02-04 Thread Guenter Roeck
On 02/04/2014 03:10 AM, Mark Brown wrote: On Mon, Feb 03, 2014 at 02:27:26PM -0800, Guenter Roeck wrote: On Mon, Feb 03, 2014 at 06:21:52PM +, Mark Brown wrote: As previously mentioned please fix your mailer to word wrap at a sensible limit. I thought I did ;-). I'll try to make sure

Re: Would devm_regulator_enable be useful ?

2014-02-04 Thread Mark Brown
On Tue, Feb 04, 2014 at 06:22:30AM -0800, Guenter Roeck wrote: On 02/04/2014 03:10 AM, Mark Brown wrote: Sort of. They're there but that doesn't mean that they should be used in normal operation - they should be special cases, not normal things. Managed resources are supposed to for things

Re: Would devm_regulator_enable be useful ?

2014-02-03 Thread Guenter Roeck
On Mon, Feb 03, 2014 at 06:21:52PM +, Mark Brown wrote: > On Sat, Feb 01, 2014 at 04:23:59PM -0800, Guenter Roeck wrote: > > As previously mentioned please fix your mailer to word wrap at a > sensible limit. > I thought I did ;-). I'll try to make sure I only send e-mail to you using mutt in

Re: Would devm_regulator_enable be useful ?

2014-02-03 Thread Mark Brown
On Sat, Feb 01, 2014 at 04:23:59PM -0800, Guenter Roeck wrote: As previously mentioned please fix your mailer to word wrap at a sensible limit. > Seems to me it would be useful to have it, but then devm_clk_enable() > doesn't exist either, so I wonder if there is a reason for not having > it.

Re: Would devm_regulator_enable be useful ?

2014-02-03 Thread Mark Brown
On Sat, Feb 01, 2014 at 04:23:59PM -0800, Guenter Roeck wrote: As previously mentioned please fix your mailer to word wrap at a sensible limit. Seems to me it would be useful to have it, but then devm_clk_enable() doesn't exist either, so I wonder if there is a reason for not having it. In

Re: Would devm_regulator_enable be useful ?

2014-02-03 Thread Guenter Roeck
On Mon, Feb 03, 2014 at 06:21:52PM +, Mark Brown wrote: On Sat, Feb 01, 2014 at 04:23:59PM -0800, Guenter Roeck wrote: As previously mentioned please fix your mailer to word wrap at a sensible limit. I thought I did ;-). I'll try to make sure I only send e-mail to you using mutt in the

Would devm_regulator_enable be useful ?

2014-02-01 Thread Guenter Roeck
Hi all, while working with regulators, I noticed that there is no devm_regulator_enable() API. Seems to me it would be useful to have it, but then devm_clk_enable() doesn't exist either, so I wonder if there is a reason for not having it. I'll be happy to submit a patch if people think it is

Would devm_regulator_enable be useful ?

2014-02-01 Thread Guenter Roeck
Hi all, while working with regulators, I noticed that there is no devm_regulator_enable() API. Seems to me it would be useful to have it, but then devm_clk_enable() doesn't exist either, so I wonder if there is a reason for not having it. I'll be happy to submit a patch if people think it is