Re: [PATCH 8/8] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation

2015-06-23 Thread Viresh Kumar
On 23-06-15, 09:38, Lee Jones wrote: > Do you always write your documentation before implementing a > feature? > > Surely it goes; > Requirements Gathering > Plan and Prepare > Implement > Test > Document > Deliver > > ;) DT bindings aren't just simple documentation of how things are

Re: [PATCH 8/8] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation

2015-06-23 Thread Lee Jones
On Tue, 23 Jun 2015, Javier Martinez Canillas wrote: > Hello Lee and Viresh, > > On Tue, Jun 23, 2015 at 10:38 AM, Lee Jones wrote: > > On Tue, 23 Jun 2015, Viresh Kumar wrote: > >> On 23-06-15, 08:06, Lee Jones wrote: > >> > > >> > > Over that, this patch should have been present before any oth

Re: [PATCH 8/8] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation

2015-06-23 Thread Javier Martinez Canillas
Hello Lee and Viresh, On Tue, Jun 23, 2015 at 10:38 AM, Lee Jones wrote: > On Tue, 23 Jun 2015, Viresh Kumar wrote: >> On 23-06-15, 08:06, Lee Jones wrote: >> > >> > > Over that, this patch should have been present before any other >> > > patches using these bindings. >> > >> > I've never heard t

Re: [PATCH 8/8] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation

2015-06-23 Thread Lee Jones
On Tue, 23 Jun 2015, Viresh Kumar wrote: > On 23-06-15, 08:06, Lee Jones wrote: > > > [Adding Rob] > > > > Rob is not the only DT Maintainer, there are many of them. The DT > > list was CC'ed, which they are all part of. Adding them all > > separately is not required IMO. > > I didn't Cc him be

Re: [PATCH 8/8] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation

2015-06-23 Thread Viresh Kumar
On 23-06-15, 08:06, Lee Jones wrote: > > [Adding Rob] > > Rob is not the only DT Maintainer, there are many of them. The DT > list was CC'ed, which they are all part of. Adding them all > separately is not required IMO. I didn't Cc him because you missed him, but because we have been discussing

Re: [PATCH 8/8] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation

2015-06-23 Thread Lee Jones
> [Adding Rob] Rob is not the only DT Maintainer, there are many of them. The DT list was CC'ed, which they are all part of. Adding them all separately is not required IMO. > On 22-06-15, 16:43, Lee Jones wrote: > > At least some description was required here on why you need additional > bindi

Re: [PATCH 8/8] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation

2015-06-22 Thread Viresh Kumar
[Adding Rob] On 22-06-15, 16:43, Lee Jones wrote: At least some description was required here on why you need additional bindings are what are they. Over that, this patch should have been present before any other patches using these bindings. > Cc: devicet...@vger.kernel.org > Signed-off-by: Le

[PATCH 8/8] dt: cpufreq: st: Provide bindings for ST's CPUFreq implementation

2015-06-22 Thread Lee Jones
Cc: devicet...@vger.kernel.org Signed-off-by: Lee Jones --- .../devicetree/bindings/cpufreq/cpufreq-st.txt | 48 ++ 1 file changed, 48 insertions(+) create mode 100644 Documentation/devicetree/bindings/cpufreq/cpufreq-st.txt diff --git a/Documentation/devicetree/bindings