Re: [PATCH 1/5] regulator: of: Add regulator-initial-mode parse support

2014-10-10 Thread Mark Rutland
On Thu, Oct 09, 2014 at 04:19:47PM +0100, Javier Martinez Canillas wrote: > Hello Mark, > > On 10/09/2014 12:27 PM, Mark Rutland wrote: > >> > >> Well, is not fairly obvious to me. One can also say the opposite, why the > >> kernel is documenting a DT binding that is not (currently) implemented?

Re: [PATCH 1/5] regulator: of: Add regulator-initial-mode parse support

2014-10-09 Thread Javier Martinez Canillas
Hello Mark, On 10/09/2014 12:27 PM, Mark Rutland wrote: >> >> Well, is not fairly obvious to me. One can also say the opposite, why the >> kernel is documenting a DT binding that is not (currently) implemented? > > Checkpatch will complain regarding undocumented bindings, so from a > pragmatic p

Re: [PATCH 1/5] regulator: of: Add regulator-initial-mode parse support

2014-10-09 Thread Mark Rutland
On Wed, Oct 08, 2014 at 05:29:26PM +0100, Javier Martinez Canillas wrote: > On 10/08/2014 05:12 PM, Mark Brown wrote: > > On Wed, Oct 08, 2014 at 04:38:53PM +0200, Javier Martinez Canillas wrote: > >> On 10/08/2014 04:25 PM, Mark Brown wrote: > > > >> > That doesn't mean that the definition of tho

Re: [PATCH 1/5] regulator: of: Add regulator-initial-mode parse support

2014-10-08 Thread Javier Martinez Canillas
On 10/08/2014 05:12 PM, Mark Brown wrote: > On Wed, Oct 08, 2014 at 04:38:53PM +0200, Javier Martinez Canillas wrote: >> On 10/08/2014 04:25 PM, Mark Brown wrote: > >> > That doesn't mean that the definition of those modes is something we can >> > sensibly provide in generic code, especially in a

Re: [PATCH 1/5] regulator: of: Add regulator-initial-mode parse support

2014-10-08 Thread Mark Brown
On Wed, Oct 08, 2014 at 04:38:53PM +0200, Javier Martinez Canillas wrote: > On 10/08/2014 04:25 PM, Mark Brown wrote: > > That doesn't mean that the definition of those modes is something we can > > sensibly provide in generic code, especially in a completely > > undocumented fashion (perhaps you'

Re: [PATCH 1/5] regulator: of: Add regulator-initial-mode parse support

2014-10-08 Thread Javier Martinez Canillas
Hello Mark, Thanks for the feedback. On 10/08/2014 04:25 PM, Mark Brown wrote: > On Wed, Oct 08, 2014 at 03:44:03PM +0200, Javier Martinez Canillas wrote: > >> But currently there isn't a way to do the same with DeviceTrees. Argubly >> the operating modes are Linux-specific so that information s

Re: [PATCH 1/5] regulator: of: Add regulator-initial-mode parse support

2014-10-08 Thread Mark Brown
On Wed, Oct 08, 2014 at 03:44:03PM +0200, Javier Martinez Canillas wrote: > But currently there isn't a way to do the same with DeviceTrees. Argubly > the operating modes are Linux-specific so that information should not be > in the DT which should be used to only describe hardware. But regulators