linux-mips.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h>
@ 2020-02-12 10:15 Geert Uytterhoeven
  2020-02-12 10:15 ` [PATCH 1/7] MIPS: ath79: " Geert Uytterhoeven
                   ` (8 more replies)
  0 siblings, 9 replies; 21+ messages in thread
From: Geert Uytterhoeven @ 2020-02-12 10:15 UTC (permalink / raw)
  To: Ralf Baechle, Paul Burton, Florian Fainelli, Paul Cercueil,
	James Hartley, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

	Hi all,

The OF clock helpers were moved to <linux/of_clk.h> a while ago.
Hence code that is not a clock provider, but just needs to call
of_clk_init(), can (and should) include <linux/of_clk.h> instead of
<linux/clk-provider.h>.

All these patches are independent of each others, and thus can be
applied by the corresponding subsystem maintainers.

Thanks!

Geert Uytterhoeven (7):
  MIPS: ath79: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  MIPS: BMIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  MIPS: generic: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  MIPS: jz4740: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  MIPS: pic32mzda: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  MIPS: Pistachio: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  MIPS: ralink: Replace <linux/clk-provider.h> by <linux/of_clk.h>

 arch/mips/ath79/setup.c          | 2 +-
 arch/mips/bmips/setup.c          | 2 +-
 arch/mips/generic/init.c         | 2 +-
 arch/mips/jz4740/time.c          | 2 +-
 arch/mips/pic32/pic32mzda/time.c | 2 +-
 arch/mips/pistachio/time.c       | 2 +-
 arch/mips/ralink/timer-gic.c     | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)

-- 
2.17.1

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [PATCH 1/7] MIPS: ath79: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
@ 2020-02-12 10:15 ` Geert Uytterhoeven
  2020-02-12 18:52   ` Stephen Boyd
  2020-02-12 10:15 ` [PATCH 2/7] MIPS: BMIPS: " Geert Uytterhoeven
                   ` (7 subsequent siblings)
  8 siblings, 1 reply; 21+ messages in thread
From: Geert Uytterhoeven @ 2020-02-12 10:15 UTC (permalink / raw)
  To: Ralf Baechle, Paul Burton, Florian Fainelli, Paul Cercueil,
	James Hartley, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

The Atheros 7/9xxx platform code is not a clock provider, and just needs
to call of_clk_init().

Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/mips/ath79/setup.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/ath79/setup.c b/arch/mips/ath79/setup.c
index 484ee28922a9ac3f..acb4fd647a304754 100644
--- a/arch/mips/ath79/setup.c
+++ b/arch/mips/ath79/setup.c
@@ -15,7 +15,7 @@
 #include <linux/memblock.h>
 #include <linux/err.h>
 #include <linux/clk.h>
-#include <linux/clk-provider.h>
+#include <linux/of_clk.h>
 #include <linux/of_fdt.h>
 #include <linux/irqchip.h>
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 2/7] MIPS: BMIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
  2020-02-12 10:15 ` [PATCH 1/7] MIPS: ath79: " Geert Uytterhoeven
@ 2020-02-12 10:15 ` Geert Uytterhoeven
  2020-02-12 17:45   ` Florian Fainelli
  2020-02-12 18:53   ` Stephen Boyd
  2020-02-12 10:15 ` [PATCH 3/7] MIPS: generic: " Geert Uytterhoeven
                   ` (6 subsequent siblings)
  8 siblings, 2 replies; 21+ messages in thread
From: Geert Uytterhoeven @ 2020-02-12 10:15 UTC (permalink / raw)
  To: Ralf Baechle, Paul Burton, Florian Fainelli, Paul Cercueil,
	James Hartley, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

The Broadcom BMIPS platform code is not a clock provider, and just needs
to call of_clk_init().

Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/mips/bmips/setup.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/bmips/setup.c b/arch/mips/bmips/setup.c
index 2f81a94c71a60f74..19308df5f577914d 100644
--- a/arch/mips/bmips/setup.c
+++ b/arch/mips/bmips/setup.c
@@ -10,11 +10,11 @@
 #include <linux/init.h>
 #include <linux/bitops.h>
 #include <linux/memblock.h>
-#include <linux/clk-provider.h>
 #include <linux/ioport.h>
 #include <linux/kernel.h>
 #include <linux/io.h>
 #include <linux/of.h>
+#include <linux/of_clk.h>
 #include <linux/of_fdt.h>
 #include <linux/of_platform.h>
 #include <linux/libfdt.h>
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 3/7] MIPS: generic: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
  2020-02-12 10:15 ` [PATCH 1/7] MIPS: ath79: " Geert Uytterhoeven
  2020-02-12 10:15 ` [PATCH 2/7] MIPS: BMIPS: " Geert Uytterhoeven
@ 2020-02-12 10:15 ` Geert Uytterhoeven
  2020-02-12 20:52   ` Stephen Boyd
  2020-02-12 10:15 ` [PATCH 4/7] MIPS: jz4740: " Geert Uytterhoeven
                   ` (5 subsequent siblings)
  8 siblings, 1 reply; 21+ messages in thread
From: Geert Uytterhoeven @ 2020-02-12 10:15 UTC (permalink / raw)
  To: Ralf Baechle, Paul Burton, Florian Fainelli, Paul Cercueil,
	James Hartley, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

The generic MIPS platform code is not a clock provider, and just needs
to call of_clk_init().

Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/mips/generic/init.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/generic/init.c b/arch/mips/generic/init.c
index 1de215b283d6a058..805d0135a9f4ea0b 100644
--- a/arch/mips/generic/init.c
+++ b/arch/mips/generic/init.c
@@ -5,10 +5,10 @@
  */
 
 #include <linux/clk.h>
-#include <linux/clk-provider.h>
 #include <linux/clocksource.h>
 #include <linux/init.h>
 #include <linux/irqchip.h>
+#include <linux/of_clk.h>
 #include <linux/of_fdt.h>
 
 #include <asm/bootinfo.h>
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 4/7] MIPS: jz4740: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
                   ` (2 preceding siblings ...)
  2020-02-12 10:15 ` [PATCH 3/7] MIPS: generic: " Geert Uytterhoeven
@ 2020-02-12 10:15 ` Geert Uytterhoeven
  2020-02-12 20:52   ` Stephen Boyd
  2020-02-13 15:36   ` Paul Cercueil
  2020-02-12 10:15 ` [PATCH 5/7] MIPS: pic32mzda: " Geert Uytterhoeven
                   ` (4 subsequent siblings)
  8 siblings, 2 replies; 21+ messages in thread
From: Geert Uytterhoeven @ 2020-02-12 10:15 UTC (permalink / raw)
  To: Ralf Baechle, Paul Burton, Florian Fainelli, Paul Cercueil,
	James Hartley, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

The Ingenic JZ4740 platform code is not a clock provider, and just needs
to call of_clk_init().

Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/mips/jz4740/time.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/jz4740/time.c b/arch/mips/jz4740/time.c
index 5476899f0882a4b4..605a84a250bfb299 100644
--- a/arch/mips/jz4740/time.c
+++ b/arch/mips/jz4740/time.c
@@ -4,8 +4,8 @@
  *  JZ4740 platform time support
  */
 
-#include <linux/clk-provider.h>
 #include <linux/clocksource.h>
+#include <linux/of_clk.h>
 
 #include <asm/mach-jz4740/timer.h>
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 5/7] MIPS: pic32mzda: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
                   ` (3 preceding siblings ...)
  2020-02-12 10:15 ` [PATCH 4/7] MIPS: jz4740: " Geert Uytterhoeven
@ 2020-02-12 10:15 ` Geert Uytterhoeven
  2020-02-12 20:52   ` Stephen Boyd
  2020-02-12 10:15 ` [PATCH 6/7] MIPS: Pistachio: " Geert Uytterhoeven
                   ` (3 subsequent siblings)
  8 siblings, 1 reply; 21+ messages in thread
From: Geert Uytterhoeven @ 2020-02-12 10:15 UTC (permalink / raw)
  To: Ralf Baechle, Paul Burton, Florian Fainelli, Paul Cercueil,
	James Hartley, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

The Microchip PIC32MZDA platform code is not a clock provider, and just
needs to call of_clk_init().

Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/mips/pic32/pic32mzda/time.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/pic32/pic32mzda/time.c b/arch/mips/pic32/pic32mzda/time.c
index 905ec1d8692873a1..754924f260891b0e 100644
--- a/arch/mips/pic32/pic32mzda/time.c
+++ b/arch/mips/pic32/pic32mzda/time.c
@@ -3,11 +3,11 @@
  * Joshua Henderson <joshua.henderson@microchip.com>
  * Copyright (C) 2015 Microchip Technology Inc.  All rights reserved.
  */
-#include <linux/clk-provider.h>
 #include <linux/clocksource.h>
 #include <linux/init.h>
 #include <linux/irqdomain.h>
 #include <linux/of.h>
+#include <linux/of_clk.h>
 #include <linux/of_irq.h>
 
 #include <asm/time.h>
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 6/7] MIPS: Pistachio: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
                   ` (4 preceding siblings ...)
  2020-02-12 10:15 ` [PATCH 5/7] MIPS: pic32mzda: " Geert Uytterhoeven
@ 2020-02-12 10:15 ` Geert Uytterhoeven
  2020-02-12 20:53   ` Stephen Boyd
  2020-02-12 22:08   ` James Hartley
  2020-02-12 10:15 ` [PATCH 7/7] MIPS: ralink: " Geert Uytterhoeven
                   ` (2 subsequent siblings)
  8 siblings, 2 replies; 21+ messages in thread
From: Geert Uytterhoeven @ 2020-02-12 10:15 UTC (permalink / raw)
  To: Ralf Baechle, Paul Burton, Florian Fainelli, Paul Cercueil,
	James Hartley, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

The Pistachio platform code is not a clock provider, and just needs to
call of_clk_init().

Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/mips/pistachio/time.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/pistachio/time.c b/arch/mips/pistachio/time.c
index 3c9235c7710a608f..de64751dec401990 100644
--- a/arch/mips/pistachio/time.c
+++ b/arch/mips/pistachio/time.c
@@ -6,10 +6,10 @@
  */
 
 #include <linux/clk.h>
-#include <linux/clk-provider.h>
 #include <linux/clocksource.h>
 #include <linux/init.h>
 #include <linux/of.h>
+#include <linux/of_clk.h>
 
 #include <asm/mips-cps.h>
 #include <asm/time.h>
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 7/7] MIPS: ralink: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
                   ` (5 preceding siblings ...)
  2020-02-12 10:15 ` [PATCH 6/7] MIPS: Pistachio: " Geert Uytterhoeven
@ 2020-02-12 10:15 ` Geert Uytterhoeven
  2020-02-12 20:53   ` Stephen Boyd
  2020-02-13  4:35   ` John Crispin
  2020-02-12 18:41 ` [PATCH 0/7] MIPS: " Philippe Mathieu-Daudé
  2020-02-19 19:04 ` Paul Burton
  8 siblings, 2 replies; 21+ messages in thread
From: Geert Uytterhoeven @ 2020-02-12 10:15 UTC (permalink / raw)
  To: Ralf Baechle, Paul Burton, Florian Fainelli, Paul Cercueil,
	James Hartley, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

The Ralink platform code is not a clock provider, and just needs to call
of_clk_init().

Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/mips/ralink/timer-gic.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/ralink/timer-gic.c b/arch/mips/ralink/timer-gic.c
index 944fbe0fc7416f5a..dcf2a44ac51eede6 100644
--- a/arch/mips/ralink/timer-gic.c
+++ b/arch/mips/ralink/timer-gic.c
@@ -8,7 +8,7 @@
 #include <linux/init.h>
 
 #include <linux/of.h>
-#include <linux/clk-provider.h>
+#include <linux/of_clk.h>
 #include <linux/clocksource.h>
 
 #include "common.h"
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* Re: [PATCH 2/7] MIPS: BMIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 2/7] MIPS: BMIPS: " Geert Uytterhoeven
@ 2020-02-12 17:45   ` Florian Fainelli
  2020-02-12 18:53   ` Stephen Boyd
  1 sibling, 0 replies; 21+ messages in thread
From: Florian Fainelli @ 2020-02-12 17:45 UTC (permalink / raw)
  To: Geert Uytterhoeven, Ralf Baechle, Paul Burton, Paul Cercueil,
	James Hartley, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel

On 2/12/20 2:15 AM, Geert Uytterhoeven wrote:
> The Broadcom BMIPS platform code is not a clock provider, and just needs
> to call of_clk_init().
> 
> Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Acked-by: Florian Fainelli <f.fainelli@gmail.com>
-- 
Florian

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
                   ` (6 preceding siblings ...)
  2020-02-12 10:15 ` [PATCH 7/7] MIPS: ralink: " Geert Uytterhoeven
@ 2020-02-12 18:41 ` Philippe Mathieu-Daudé
  2020-02-19 19:04 ` Paul Burton
  8 siblings, 0 replies; 21+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-02-12 18:41 UTC (permalink / raw)
  To: Geert Uytterhoeven, Ralf Baechle, Paul Burton, Florian Fainelli,
	Paul Cercueil, James Hartley, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel

On 2/12/20 11:15 AM, Geert Uytterhoeven wrote:
> 	Hi all,
> 
> The OF clock helpers were moved to <linux/of_clk.h> a while ago.
> Hence code that is not a clock provider, but just needs to call
> of_clk_init(), can (and should) include <linux/of_clk.h> instead of
> <linux/clk-provider.h>.
> 
> All these patches are independent of each others, and thus can be
> applied by the corresponding subsystem maintainers.
> 
> Thanks!
> 
> Geert Uytterhoeven (7):
>   MIPS: ath79: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: BMIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: generic: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: jz4740: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: pic32mzda: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: Pistachio: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: ralink: Replace <linux/clk-provider.h> by <linux/of_clk.h>
> 
>  arch/mips/ath79/setup.c          | 2 +-
>  arch/mips/bmips/setup.c          | 2 +-
>  arch/mips/generic/init.c         | 2 +-
>  arch/mips/jz4740/time.c          | 2 +-
>  arch/mips/pic32/pic32mzda/time.c | 2 +-
>  arch/mips/pistachio/time.c       | 2 +-
>  arch/mips/ralink/timer-gic.c     | 2 +-
>  7 files changed, 7 insertions(+), 7 deletions(-)
> 

Series:
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 1/7] MIPS: ath79: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 1/7] MIPS: ath79: " Geert Uytterhoeven
@ 2020-02-12 18:52   ` Stephen Boyd
  0 siblings, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-02-12 18:52 UTC (permalink / raw)
  To: Florian Fainelli, Geert Uytterhoeven, James Hartley,
	John Crispin, Paul Burton, Paul Cercueil, Ralf Baechle
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

Quoting Geert Uytterhoeven (2020-02-12 02:15:38)
> The Atheros 7/9xxx platform code is not a clock provider, and just needs
> to call of_clk_init().
> 
> Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 2/7] MIPS: BMIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 2/7] MIPS: BMIPS: " Geert Uytterhoeven
  2020-02-12 17:45   ` Florian Fainelli
@ 2020-02-12 18:53   ` Stephen Boyd
  1 sibling, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-02-12 18:53 UTC (permalink / raw)
  To: Florian Fainelli, Geert Uytterhoeven, James Hartley,
	John Crispin, Paul Burton, Paul Cercueil, Ralf Baechle
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

Quoting Geert Uytterhoeven (2020-02-12 02:15:39)
> The Broadcom BMIPS platform code is not a clock provider, and just needs
> to call of_clk_init().
> 
> Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 3/7] MIPS: generic: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 3/7] MIPS: generic: " Geert Uytterhoeven
@ 2020-02-12 20:52   ` Stephen Boyd
  0 siblings, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-02-12 20:52 UTC (permalink / raw)
  To: Florian Fainelli, Geert Uytterhoeven, James Hartley,
	John Crispin, Paul Burton, Paul Cercueil, Ralf Baechle
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

Quoting Geert Uytterhoeven (2020-02-12 02:15:40)
> The generic MIPS platform code is not a clock provider, and just needs
> to call of_clk_init().
> 
> Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 4/7] MIPS: jz4740: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 4/7] MIPS: jz4740: " Geert Uytterhoeven
@ 2020-02-12 20:52   ` Stephen Boyd
  2020-02-13 15:36   ` Paul Cercueil
  1 sibling, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-02-12 20:52 UTC (permalink / raw)
  To: Florian Fainelli, Geert Uytterhoeven, James Hartley,
	John Crispin, Paul Burton, Paul Cercueil, Ralf Baechle
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

Quoting Geert Uytterhoeven (2020-02-12 02:15:41)
> The Ingenic JZ4740 platform code is not a clock provider, and just needs
> to call of_clk_init().
> 
> Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 5/7] MIPS: pic32mzda: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 5/7] MIPS: pic32mzda: " Geert Uytterhoeven
@ 2020-02-12 20:52   ` Stephen Boyd
  0 siblings, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-02-12 20:52 UTC (permalink / raw)
  To: Florian Fainelli, Geert Uytterhoeven, James Hartley,
	John Crispin, Paul Burton, Paul Cercueil, Ralf Baechle
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

Quoting Geert Uytterhoeven (2020-02-12 02:15:42)
> The Microchip PIC32MZDA platform code is not a clock provider, and just
> needs to call of_clk_init().
> 
> Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 6/7] MIPS: Pistachio: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 6/7] MIPS: Pistachio: " Geert Uytterhoeven
@ 2020-02-12 20:53   ` Stephen Boyd
  2020-02-12 22:08   ` James Hartley
  1 sibling, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-02-12 20:53 UTC (permalink / raw)
  To: Florian Fainelli, Geert Uytterhoeven, James Hartley,
	John Crispin, Paul Burton, Paul Cercueil, Ralf Baechle
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

Quoting Geert Uytterhoeven (2020-02-12 02:15:43)
> The Pistachio platform code is not a clock provider, and just needs to
> call of_clk_init().
> 
> Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 7/7] MIPS: ralink: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 7/7] MIPS: ralink: " Geert Uytterhoeven
@ 2020-02-12 20:53   ` Stephen Boyd
  2020-02-13  4:35   ` John Crispin
  1 sibling, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-02-12 20:53 UTC (permalink / raw)
  To: Florian Fainelli, Geert Uytterhoeven, James Hartley,
	John Crispin, Paul Burton, Paul Cercueil, Ralf Baechle
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven

Quoting Geert Uytterhoeven (2020-02-12 02:15:44)
> The Ralink platform code is not a clock provider, and just needs to call
> of_clk_init().
> 
> Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 6/7] MIPS: Pistachio: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 6/7] MIPS: Pistachio: " Geert Uytterhoeven
  2020-02-12 20:53   ` Stephen Boyd
@ 2020-02-12 22:08   ` James Hartley
  1 sibling, 0 replies; 21+ messages in thread
From: James Hartley @ 2020-02-12 22:08 UTC (permalink / raw)
  To: Geert Uytterhoeven, Ralf Baechle, Paul Burton, Florian Fainelli,
	Paul Cercueil, John Crispin
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel


On 12/02/2020 10:15, Geert Uytterhoeven wrote:
> The Pistachio platform code is not a clock provider, and just needs to
> call of_clk_init().
>
> Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>   arch/mips/pistachio/time.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/mips/pistachio/time.c b/arch/mips/pistachio/time.c
> index 3c9235c7710a608f..de64751dec401990 100644
> --- a/arch/mips/pistachio/time.c
> +++ b/arch/mips/pistachio/time.c
> @@ -6,10 +6,10 @@
>    */
>   
>   #include <linux/clk.h>
> -#include <linux/clk-provider.h>
>   #include <linux/clocksource.h>
>   #include <linux/init.h>
>   #include <linux/of.h>
> +#include <linux/of_clk.h>
>   
>   #include <asm/mips-cps.h>
>   #include <asm/time.h>

Thanks for the update Geert.

Acked-by: James Hartley <james.hartley@sondrel.com>


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 7/7] MIPS: ralink: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 7/7] MIPS: ralink: " Geert Uytterhoeven
  2020-02-12 20:53   ` Stephen Boyd
@ 2020-02-13  4:35   ` John Crispin
  1 sibling, 0 replies; 21+ messages in thread
From: John Crispin @ 2020-02-13  4:35 UTC (permalink / raw)
  To: Geert Uytterhoeven, Ralf Baechle, Paul Burton, Florian Fainelli,
	Paul Cercueil, James Hartley
  Cc: linux-mips, bcm-kernel-feedback-list, linux-clk, linux-kernel

On 12/02/2020 11:15, Geert Uytterhoeven wrote:
> The Ralink platform code is not a clock provider, and just needs to call
> of_clk_init().
> 
> Hence it can include <linux/of_clk.h> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: John Crispin <john@phrozen.org>

> ---
>   arch/mips/ralink/timer-gic.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/mips/ralink/timer-gic.c b/arch/mips/ralink/timer-gic.c
> index 944fbe0fc7416f5a..dcf2a44ac51eede6 100644
> --- a/arch/mips/ralink/timer-gic.c
> +++ b/arch/mips/ralink/timer-gic.c
> @@ -8,7 +8,7 @@
>   #include <linux/init.h>
>   
>   #include <linux/of.h>
> -#include <linux/clk-provider.h>
> +#include <linux/of_clk.h>
>   #include <linux/clocksource.h>
>   
>   #include "common.h"
> 


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 4/7] MIPS: jz4740: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 ` [PATCH 4/7] MIPS: jz4740: " Geert Uytterhoeven
  2020-02-12 20:52   ` Stephen Boyd
@ 2020-02-13 15:36   ` Paul Cercueil
  1 sibling, 0 replies; 21+ messages in thread
From: Paul Cercueil @ 2020-02-13 15:36 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Ralf Baechle, Paul Burton, Florian Fainelli, James Hartley,
	John Crispin, linux-mips, bcm-kernel-feedback-list, linux-clk,
	linux-kernel

Hi,


Le mer., févr. 12, 2020 at 11:15, Geert Uytterhoeven 
<geert+renesas@glider.be> a écrit :
> The Ingenic JZ4740 platform code is not a clock provider, and just 
> needs
> to call of_clk_init().
> 
> Hence it can include <linux/of_clk.h> instead of 
> <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Looks good to me.
Reviewed-by: Paul Cercueil <paul@crapouillou.net>


> ---
>  arch/mips/jz4740/time.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/mips/jz4740/time.c b/arch/mips/jz4740/time.c
> index 5476899f0882a4b4..605a84a250bfb299 100644
> --- a/arch/mips/jz4740/time.c
> +++ b/arch/mips/jz4740/time.c
> @@ -4,8 +4,8 @@
>   *  JZ4740 platform time support
>   */
> 
> -#include <linux/clk-provider.h>
>  #include <linux/clocksource.h>
> +#include <linux/of_clk.h>
> 
>  #include <asm/mach-jz4740/timer.h>
> 
> --
> 2.17.1
> 



^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:15 [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
                   ` (7 preceding siblings ...)
  2020-02-12 18:41 ` [PATCH 0/7] MIPS: " Philippe Mathieu-Daudé
@ 2020-02-19 19:04 ` Paul Burton
  8 siblings, 0 replies; 21+ messages in thread
From: Paul Burton @ 2020-02-19 19:04 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Ralf Baechle, Paul Burton, Florian Fainelli, Paul Cercueil,
	James Hartley, John Crispin, linux-mips,
	bcm-kernel-feedback-list, linux-clk, linux-kernel,
	Geert Uytterhoeven, linux-mips

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 4290 bytes --]

Hello,

Geert Uytterhoeven wrote:
> Hi all,
> 
> The OF clock helpers were moved to <linux/of_clk.h> a while ago.
> Hence code that is not a clock provider, but just needs to call
> of_clk_init(), can (and should) include <linux/of_clk.h> instead of
> <linux/clk-provider.h>.
> 
> All these patches are independent of each others, and thus can be
> applied by the corresponding subsystem maintainers.
> 
> Thanks!
> 
> Geert Uytterhoeven (7):
>   MIPS: ath79: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: BMIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: generic: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: jz4740: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: pic32mzda: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: Pistachio: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   MIPS: ralink: Replace <linux/clk-provider.h> by <linux/of_clk.h>
> 
>  arch/mips/ath79/setup.c          | 2 +-
>  arch/mips/bmips/setup.c          | 2 +-
>  arch/mips/generic/init.c         | 2 +-
>  arch/mips/jz4740/time.c          | 2 +-
>  arch/mips/pic32/pic32mzda/time.c | 2 +-
>  arch/mips/pistachio/time.c       | 2 +-
>  arch/mips/ralink/timer-gic.c     | 2 +-
>  7 files changed, 7 insertions(+), 7 deletions(-)

Series applied to mips-next.

> MIPS: ath79: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   commit d2936bd02b19
>   https://git.kernel.org/mips/c/d2936bd02b19
>   
>   Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>   Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>   Reviewed-by: Stephen Boyd <sboyd@kernel.org>
>   Signed-off-by: Paul Burton <paulburton@kernel.org>
> 
> MIPS: BMIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   commit e40b3deff7af
>   https://git.kernel.org/mips/c/e40b3deff7af
>   
>   Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>   Acked-by: Florian Fainelli <f.fainelli@gmail.com>
>   Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>   Reviewed-by: Stephen Boyd <sboyd@kernel.org>
>   Signed-off-by: Paul Burton <paulburton@kernel.org>
> 
> MIPS: generic: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   commit 089a792c750d
>   https://git.kernel.org/mips/c/089a792c750d
>   
>   Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>   Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>   Reviewed-by: Stephen Boyd <sboyd@kernel.org>
>   Signed-off-by: Paul Burton <paulburton@kernel.org>
> 
> MIPS: jz4740: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   commit 3a94afc68947
>   https://git.kernel.org/mips/c/3a94afc68947
>   
>   Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>   Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>   Reviewed-by: Stephen Boyd <sboyd@kernel.org>
>   Reviewed-by: Paul Cercueil <paul@crapouillou.net>
>   Signed-off-by: Paul Burton <paulburton@kernel.org>
> 
> MIPS: pic32mzda: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   commit 071cec1bfe1f
>   https://git.kernel.org/mips/c/071cec1bfe1f
>   
>   Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>   Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>   Reviewed-by: Stephen Boyd <sboyd@kernel.org>
>   Signed-off-by: Paul Burton <paulburton@kernel.org>
> 
> MIPS: Pistachio: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   commit 97e04ea15fd5
>   https://git.kernel.org/mips/c/97e04ea15fd5
>   
>   Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>   Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>   Reviewed-by: Stephen Boyd <sboyd@kernel.org>
>   Acked-by: James Hartley <james.hartley@sondrel.com>
>   Signed-off-by: Paul Burton <paulburton@kernel.org>
> 
> MIPS: ralink: Replace <linux/clk-provider.h> by <linux/of_clk.h>
>   commit 9926108f799a
>   https://git.kernel.org/mips/c/9926108f799a
>   
>   Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>   Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>   Reviewed-by: Stephen Boyd <sboyd@kernel.org>
>   Acked-by: John Crispin <john@phrozen.org>
>   Signed-off-by: Paul Burton <paulburton@kernel.org>

Thanks,
    Paul

[ This message was auto-generated; if you believe anything is incorrect
  then please email paulburton@kernel.org to report it. ]

^ permalink raw reply	[flat|nested] 21+ messages in thread

end of thread, other threads:[~2020-02-19 19:04 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-12 10:15 [PATCH 0/7] MIPS: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
2020-02-12 10:15 ` [PATCH 1/7] MIPS: ath79: " Geert Uytterhoeven
2020-02-12 18:52   ` Stephen Boyd
2020-02-12 10:15 ` [PATCH 2/7] MIPS: BMIPS: " Geert Uytterhoeven
2020-02-12 17:45   ` Florian Fainelli
2020-02-12 18:53   ` Stephen Boyd
2020-02-12 10:15 ` [PATCH 3/7] MIPS: generic: " Geert Uytterhoeven
2020-02-12 20:52   ` Stephen Boyd
2020-02-12 10:15 ` [PATCH 4/7] MIPS: jz4740: " Geert Uytterhoeven
2020-02-12 20:52   ` Stephen Boyd
2020-02-13 15:36   ` Paul Cercueil
2020-02-12 10:15 ` [PATCH 5/7] MIPS: pic32mzda: " Geert Uytterhoeven
2020-02-12 20:52   ` Stephen Boyd
2020-02-12 10:15 ` [PATCH 6/7] MIPS: Pistachio: " Geert Uytterhoeven
2020-02-12 20:53   ` Stephen Boyd
2020-02-12 22:08   ` James Hartley
2020-02-12 10:15 ` [PATCH 7/7] MIPS: ralink: " Geert Uytterhoeven
2020-02-12 20:53   ` Stephen Boyd
2020-02-13  4:35   ` John Crispin
2020-02-12 18:41 ` [PATCH 0/7] MIPS: " Philippe Mathieu-Daudé
2020-02-19 19:04 ` Paul Burton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).