Re: [RFC 04/15] regulator: add restrack support

2014-12-12 Thread Andrzej Hajda
On 12/11/2014 02:43 PM, Russell King - ARM Linux wrote: > On Thu, Dec 11, 2014 at 12:58:37PM +, Mark Brown wrote: >> I'd expect someone reading the change in the regulator API to have at >> least some idea how this fits in with the rest of the API and how to use >> it, and probably more

Re: [RFC 04/15] regulator: add restrack support

2014-12-12 Thread Andrzej Hajda
On 12/11/2014 02:43 PM, Russell King - ARM Linux wrote: On Thu, Dec 11, 2014 at 12:58:37PM +, Mark Brown wrote: I'd expect someone reading the change in the regulator API to have at least some idea how this fits in with the rest of the API and how to use it, and probably more importantly

Re: [RFC 04/15] regulator: add restrack support

2014-12-11 Thread Russell King - ARM Linux
On Thu, Dec 11, 2014 at 12:58:37PM +, Mark Brown wrote: > I'd expect someone reading the change in the regulator API to have at > least some idea how this fits in with the rest of the API and how to use > it, and probably more importantly I'd expect to be able to understand > why this is DT

Re: [RFC 04/15] regulator: add restrack support

2014-12-11 Thread Mark Brown
On Thu, Dec 11, 2014 at 11:49:54AM +0100, Andrzej Hajda wrote: > On 12/10/2014 05:07 PM, Mark Brown wrote: > > On Wed, Dec 10, 2014 at 04:48:22PM +0100, Andrzej Hajda wrote: > >> Regulators supports various methods of lookup. > >> The patch adds restrack support only to DT based regulators. > >

Re: [RFC 04/15] regulator: add restrack support

2014-12-11 Thread Andrzej Hajda
Hi Mark, On 12/10/2014 05:07 PM, Mark Brown wrote: > On Wed, Dec 10, 2014 at 04:48:22PM +0100, Andrzej Hajda wrote: >> Regulators supports various methods of lookup. >> The patch adds restrack support only to DT based regulators. > Why, what does this mean and how might one use it? I've not

Re: [RFC 04/15] regulator: add restrack support

2014-12-11 Thread Andrzej Hajda
Hi Mark, On 12/10/2014 05:07 PM, Mark Brown wrote: On Wed, Dec 10, 2014 at 04:48:22PM +0100, Andrzej Hajda wrote: Regulators supports various methods of lookup. The patch adds restrack support only to DT based regulators. Why, what does this mean and how might one use it? I've not looked at

Re: [RFC 04/15] regulator: add restrack support

2014-12-11 Thread Mark Brown
On Thu, Dec 11, 2014 at 11:49:54AM +0100, Andrzej Hajda wrote: On 12/10/2014 05:07 PM, Mark Brown wrote: On Wed, Dec 10, 2014 at 04:48:22PM +0100, Andrzej Hajda wrote: Regulators supports various methods of lookup. The patch adds restrack support only to DT based regulators. Why, what

Re: [RFC 04/15] regulator: add restrack support

2014-12-11 Thread Russell King - ARM Linux
On Thu, Dec 11, 2014 at 12:58:37PM +, Mark Brown wrote: I'd expect someone reading the change in the regulator API to have at least some idea how this fits in with the rest of the API and how to use it, and probably more importantly I'd expect to be able to understand why this is DT only.

Re: [RFC 04/15] regulator: add restrack support

2014-12-10 Thread Mark Brown
On Wed, Dec 10, 2014 at 04:48:22PM +0100, Andrzej Hajda wrote: > Regulators supports various methods of lookup. > The patch adds restrack support only to DT based regulators. Why, what does this mean and how might one use it? I've not looked at the code since I don't know what it's supposed to

[RFC 04/15] regulator: add restrack support

2014-12-10 Thread Andrzej Hajda
Regulators supports various methods of lookup. The patch adds restrack support only to DT based regulators. Signed-off-by: Andrzej Hajda --- drivers/regulator/core.c | 77 ++ include/linux/regulator/consumer.h | 10 + include/linux/restrack.h

[RFC 04/15] regulator: add restrack support

2014-12-10 Thread Andrzej Hajda
Regulators supports various methods of lookup. The patch adds restrack support only to DT based regulators. Signed-off-by: Andrzej Hajda a.ha...@samsung.com --- drivers/regulator/core.c | 77 ++ include/linux/regulator/consumer.h | 10 +

Re: [RFC 04/15] regulator: add restrack support

2014-12-10 Thread Mark Brown
On Wed, Dec 10, 2014 at 04:48:22PM +0100, Andrzej Hajda wrote: Regulators supports various methods of lookup. The patch adds restrack support only to DT based regulators. Why, what does this mean and how might one use it? I've not looked at the code since I don't know what it's supposed to