Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-05-17 Thread Maxime Ripard
On Fri, May 08, 2015 at 08:22:46AM +0100, Lee Jones wrote: > On Thu, 07 May 2015, Maxime Ripard wrote: > > > On Fri, May 01, 2015 at 07:44:05AM +0100, Lee Jones wrote: > > > > > Does Sascha's antidote patch change your opinion? We can use DT to > > > > > declare critical clocks, and in the rare

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-05-17 Thread Maxime Ripard
On Fri, May 08, 2015 at 08:22:46AM +0100, Lee Jones wrote: On Thu, 07 May 2015, Maxime Ripard wrote: On Fri, May 01, 2015 at 07:44:05AM +0100, Lee Jones wrote: Does Sascha's antidote patch change your opinion? We can use DT to declare critical clocks, and in the rare case of the

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-05-08 Thread Lee Jones
On Thu, 07 May 2015, Maxime Ripard wrote: > On Fri, May 01, 2015 at 07:44:05AM +0100, Lee Jones wrote: > > > > Does Sascha's antidote patch change your opinion? We can use DT to > > > > declare critical clocks, and in the rare case of the introduction of a > > > > new DDRFreq-like feature, which

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-05-08 Thread Lee Jones
On Thu, 07 May 2015, Maxime Ripard wrote: On Fri, May 01, 2015 at 07:44:05AM +0100, Lee Jones wrote: Does Sascha's antidote patch change your opinion? We can use DT to declare critical clocks, and in the rare case of the introduction of a new DDRFreq-like feature, which doesn't

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-05-07 Thread Maxime Ripard
On Fri, May 01, 2015 at 07:44:05AM +0100, Lee Jones wrote: > > > Does Sascha's antidote patch change your opinion? We can use DT to > > > declare critical clocks, and in the rare case of the introduction of a > > > new DDRFreq-like feature, which doesn't adapt the DT will still be > > > able to

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-05-07 Thread Maxime Ripard
On Fri, May 01, 2015 at 07:44:05AM +0100, Lee Jones wrote: Does Sascha's antidote patch change your opinion? We can use DT to declare critical clocks, and in the rare case of the introduction of a new DDRFreq-like feature, which doesn't adapt the DT will still be able to unlock the

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-05-04 Thread Maxime Ripard
On Wed, Apr 29, 2015 at 04:05:49PM -0700, Michael Turquette wrote: > > > Could you elaborate why you still want to have the clk-always-on in the > > > device tree instead of in the kernel where it can be removed when > > > necessary? What's your problem with enabling the critical clocks using > >

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-05-04 Thread Maxime Ripard
On Wed, Apr 29, 2015 at 04:05:49PM -0700, Michael Turquette wrote: Could you elaborate why you still want to have the clk-always-on in the device tree instead of in the kernel where it can be removed when necessary? What's your problem with enabling the critical clocks using

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-05-01 Thread Lee Jones
On Fri, 01 May 2015, Sascha Hauer wrote: > On Thu, Apr 30, 2015 at 10:57:22AM +0100, Lee Jones wrote: > > On Wed, 29 Apr 2015, Maxime Ripard wrote: > > > > > On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: > > > > On Wed, 22 Apr 2015, Maxime Ripard wrote: > > > > > > > > > On Wed, Apr

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-05-01 Thread Lee Jones
On Fri, 01 May 2015, Sascha Hauer wrote: On Thu, Apr 30, 2015 at 10:57:22AM +0100, Lee Jones wrote: On Wed, 29 Apr 2015, Maxime Ripard wrote: On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: On Wed, 22 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-30 Thread Sascha Hauer
On Thu, Apr 30, 2015 at 10:57:22AM +0100, Lee Jones wrote: > On Wed, 29 Apr 2015, Maxime Ripard wrote: > > > On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: > > > On Wed, 22 Apr 2015, Maxime Ripard wrote: > > > > > > > On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: > > > >

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-30 Thread Lee Jones
On Wed, 29 Apr 2015, Maxime Ripard wrote: > On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: > > On Wed, 22 Apr 2015, Maxime Ripard wrote: > > > > > On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: > > > > On Wed, 08 Apr 2015, Maxime Ripard wrote: > > > > > > > > > On Wed,

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-30 Thread Lee Jones
On Wed, 29 Apr 2015, Maxime Ripard wrote: On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: On Wed, 22 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-30 Thread Sascha Hauer
On Thu, Apr 30, 2015 at 10:57:22AM +0100, Lee Jones wrote: On Wed, 29 Apr 2015, Maxime Ripard wrote: On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: On Wed, 22 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: On Wed, 08 Apr

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Michael Turquette
Quoting Lee Jones (2015-04-29 09:07:13) > On Wed, 29 Apr 2015, Sascha Hauer wrote: > > > On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: > > > On Wed, 22 Apr 2015, Maxime Ripard wrote: > > > > > > > On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: > > > > > On Wed, 08 Apr

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Maxime Ripard
On Wed, Apr 29, 2015 at 04:11:47PM +0100, Lee Jones wrote: > On Wed, 29 Apr 2015, Geert Uytterhoeven wrote: > > On Wed, Apr 29, 2015 at 4:17 PM, Lee Jones wrote: > > > This useful binding should be accepted and people should not abuse > > > it. If they do and the vendor Maintainer's review and

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Maxime Ripard
On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: > On Wed, 22 Apr 2015, Maxime Ripard wrote: > > > On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: > > > On Wed, 08 Apr 2015, Maxime Ripard wrote: > > > > > > > On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: > > > >

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Lee Jones
On Wed, 29 Apr 2015, Sascha Hauer wrote: > On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: > > On Wed, 22 Apr 2015, Maxime Ripard wrote: > > > > > On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: > > > > On Wed, 08 Apr 2015, Maxime Ripard wrote: > > > > > > > > > On Wed,

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Lee Jones
On Wed, 29 Apr 2015, Geert Uytterhoeven wrote: > On Wed, Apr 29, 2015 at 4:17 PM, Lee Jones wrote: > > This useful binding should be accepted and people should not abuse > > it. If they do and the vendor Maintainer's review and accept then > > they have no foundation for recourse. > > > > Would

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Sascha Hauer
On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: > On Wed, 22 Apr 2015, Maxime Ripard wrote: > > > On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: > > > On Wed, 08 Apr 2015, Maxime Ripard wrote: > > > > > > > On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: > > > >

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Geert Uytterhoeven
Hi Lee, On Wed, Apr 29, 2015 at 4:17 PM, Lee Jones wrote: > This useful binding should be accepted and people should not abuse > it. If they do and the vendor Maintainer's review and accept then > they have no foundation for recourse. > > Would you prefer it if I made the warning starker?

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Lee Jones
On Wed, 22 Apr 2015, Maxime Ripard wrote: > On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: > > On Wed, 08 Apr 2015, Maxime Ripard wrote: > > > > > On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: > > > > On Wed, 08 Apr 2015, Maxime Ripard wrote: > > > > > > > > > On Wed,

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Lee Jones
On Wed, 29 Apr 2015, Sascha Hauer wrote: On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: On Wed, 22 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Maxime Ripard
On Wed, Apr 29, 2015 at 04:11:47PM +0100, Lee Jones wrote: On Wed, 29 Apr 2015, Geert Uytterhoeven wrote: On Wed, Apr 29, 2015 at 4:17 PM, Lee Jones lee.jo...@linaro.org wrote: This useful binding should be accepted and people should not abuse it. If they do and the vendor Maintainer's

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Maxime Ripard
On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: On Wed, 22 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: On Wed, 08 Apr

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Michael Turquette
Quoting Lee Jones (2015-04-29 09:07:13) On Wed, 29 Apr 2015, Sascha Hauer wrote: On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: On Wed, 22 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Lee Jones
On Wed, 22 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Geert Uytterhoeven
Hi Lee, On Wed, Apr 29, 2015 at 4:17 PM, Lee Jones lee.jo...@linaro.org wrote: This useful binding should be accepted and people should not abuse it. If they do and the vendor Maintainer's review and accept then they have no foundation for recourse. Would you prefer it if I made the warning

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Sascha Hauer
On Wed, Apr 29, 2015 at 03:17:51PM +0100, Lee Jones wrote: On Wed, 22 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: On Wed, 08 Apr

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-29 Thread Lee Jones
On Wed, 29 Apr 2015, Geert Uytterhoeven wrote: On Wed, Apr 29, 2015 at 4:17 PM, Lee Jones lee.jo...@linaro.org wrote: This useful binding should be accepted and people should not abuse it. If they do and the vendor Maintainer's review and accept then they have no foundation for recourse.

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-22 Thread Maxime Ripard
On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: > On Wed, 08 Apr 2015, Maxime Ripard wrote: > > > On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: > > > On Wed, 08 Apr 2015, Maxime Ripard wrote: > > > > > > > On Wed, Apr 08, 2015 at 09:14:50AM +0100, Lee Jones wrote: > > > >

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-22 Thread Maxime Ripard
On Wed, Apr 08, 2015 at 06:23:44PM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 09:14:50AM +0100, Lee Jones wrote: + +

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Lee Jones
On Wed, 08 Apr 2015, Maxime Ripard wrote: > On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: > > On Wed, 08 Apr 2015, Maxime Ripard wrote: > > > > > On Wed, Apr 08, 2015 at 09:14:50AM +0100, Lee Jones wrote: > > > > > > + > > > > > > + This property is not to be abused.

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Maxime Ripard
On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: > On Wed, 08 Apr 2015, Maxime Ripard wrote: > > > On Wed, Apr 08, 2015 at 09:14:50AM +0100, Lee Jones wrote: > > > > > + > > > > > + This property is not to be abused. It is only to > > > > > be used to > > > > > +

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Lee Jones
On Wed, 08 Apr 2015, Maxime Ripard wrote: > On Wed, Apr 08, 2015 at 09:14:50AM +0100, Lee Jones wrote: > > > > + > > > > + This property is not to be abused. It is only to > > > > be used to > > > > + protect platforms from being crippled by gated > > > >

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Maxime Ripard
On Wed, Apr 08, 2015 at 09:14:50AM +0100, Lee Jones wrote: > > > + > > > + This property is not to be abused. It is only to be used to > > > + protect platforms from being crippled by gated clocks, not > > > + as a convenience function to avoid using the

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Lee Jones
On Tue, 07 Apr 2015, Maxime Ripard wrote: > On Tue, Apr 07, 2015 at 07:43:59PM +0100, Lee Jones wrote: > > Signed-off-by: Lee Jones > > --- > > .../devicetree/bindings/clock/clock-bindings.txt | 38 > > ++ > > 1 file changed, 38 insertions(+) > > > > diff --git

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Maxime Ripard
Hi Lee, On Tue, Apr 07, 2015 at 07:43:59PM +0100, Lee Jones wrote: > Signed-off-by: Lee Jones > --- > .../devicetree/bindings/clock/clock-bindings.txt | 38 > ++ > 1 file changed, 38 insertions(+) > > diff --git

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Lee Jones
On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 09:14:50AM +0100, Lee Jones wrote: + + This property is not to be abused. It is only to be used to + protect platforms from being crippled by gated clocks, not +

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Maxime Ripard
Hi Lee, On Tue, Apr 07, 2015 at 07:43:59PM +0100, Lee Jones wrote: Signed-off-by: Lee Jones lee.jo...@linaro.org --- .../devicetree/bindings/clock/clock-bindings.txt | 38 ++ 1 file changed, 38 insertions(+) diff --git

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Lee Jones
On Tue, 07 Apr 2015, Maxime Ripard wrote: On Tue, Apr 07, 2015 at 07:43:59PM +0100, Lee Jones wrote: Signed-off-by: Lee Jones lee.jo...@linaro.org --- .../devicetree/bindings/clock/clock-bindings.txt | 38 ++ 1 file changed, 38 insertions(+) diff --git

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Maxime Ripard
On Wed, Apr 08, 2015 at 09:14:50AM +0100, Lee Jones wrote: + + This property is not to be abused. It is only to be used to + protect platforms from being crippled by gated clocks, not + as a convenience function to avoid using the framework +

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Lee Jones
On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 09:14:50AM +0100, Lee Jones wrote: + + This property is not to be abused. It is only to

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-08 Thread Maxime Ripard
On Wed, Apr 08, 2015 at 11:38:32AM +0100, Lee Jones wrote: On Wed, 08 Apr 2015, Maxime Ripard wrote: On Wed, Apr 08, 2015 at 09:14:50AM +0100, Lee Jones wrote: + + This property is not to be abused. It is only to be used to + protect

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-07 Thread Stephen Boyd
On 04/07, Lee Jones wrote: > Signed-off-by: Lee Jones > --- Can you please add some commit text here? Why did we make this change? > .../devicetree/bindings/clock/clock-bindings.txt | 38 > ++ > 1 file changed, 38 insertions(+) > > diff --git

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-07 Thread Rob Herring
On Tue, Apr 7, 2015 at 1:43 PM, Lee Jones wrote: > Signed-off-by: Lee Jones Please use get_maintainers.pl in the future. Acked-by: Rob Herring > --- > .../devicetree/bindings/clock/clock-bindings.txt | 38 > ++ > 1 file changed, 38 insertions(+) > > diff --git

[PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-07 Thread Lee Jones
Signed-off-by: Lee Jones --- .../devicetree/bindings/clock/clock-bindings.txt | 38 ++ 1 file changed, 38 insertions(+) diff --git a/Documentation/devicetree/bindings/clock/clock-bindings.txt b/Documentation/devicetree/bindings/clock/clock-bindings.txt index

[PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-07 Thread Lee Jones
Signed-off-by: Lee Jones lee.jo...@linaro.org --- .../devicetree/bindings/clock/clock-bindings.txt | 38 ++ 1 file changed, 38 insertions(+) diff --git a/Documentation/devicetree/bindings/clock/clock-bindings.txt b/Documentation/devicetree/bindings/clock/clock-bindings.txt

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-07 Thread Rob Herring
On Tue, Apr 7, 2015 at 1:43 PM, Lee Jones lee.jo...@linaro.org wrote: Signed-off-by: Lee Jones lee.jo...@linaro.org Please use get_maintainers.pl in the future. Acked-by: Rob Herring r...@kernel.org --- .../devicetree/bindings/clock/clock-bindings.txt | 38 ++ 1

Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

2015-04-07 Thread Stephen Boyd
On 04/07, Lee Jones wrote: Signed-off-by: Lee Jones lee.jo...@linaro.org --- Can you please add some commit text here? Why did we make this change? .../devicetree/bindings/clock/clock-bindings.txt | 38 ++ 1 file changed, 38 insertions(+) diff --git