* [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).