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 -- [email protected]
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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 <[email protected]>
* 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
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 <[email protected]>
Acked-by: Florian Fainelli <[email protected]>
--
Florian
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é <[email protected]>
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 <[email protected]>
> ---
Reviewed-by: Stephen Boyd <[email protected]>
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 <[email protected]>
> ---
Reviewed-by: Stephen Boyd <[email protected]>
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 <[email protected]>
> ---
Reviewed-by: Stephen Boyd <[email protected]>
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 <[email protected]>
> ---
Reviewed-by: Stephen Boyd <[email protected]>
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 <[email protected]>
> ---
Reviewed-by: Stephen Boyd <[email protected]>
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 <[email protected]>
> ---
Reviewed-by: Stephen Boyd <[email protected]>
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 <[email protected]>
Acked-by: John Crispin <[email protected]>
> ---
> 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"
>
Hi,
Le mer., f?vr. 12, 2020 at 11:15, Geert Uytterhoeven
<[email protected]> 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 <[email protected]>
Looks good to me.
Reviewed-by: Paul Cercueil <[email protected]>
> ---
> 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
>
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 <[email protected]>
> Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
> Reviewed-by: Stephen Boyd <[email protected]>
> Signed-off-by: Paul Burton <[email protected]>
>
> 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 <[email protected]>
> Acked-by: Florian Fainelli <[email protected]>
> Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
> Reviewed-by: Stephen Boyd <[email protected]>
> Signed-off-by: Paul Burton <[email protected]>
>
> 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 <[email protected]>
> Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
> Reviewed-by: Stephen Boyd <[email protected]>
> Signed-off-by: Paul Burton <[email protected]>
>
> 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 <[email protected]>
> Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
> Reviewed-by: Stephen Boyd <[email protected]>
> Reviewed-by: Paul Cercueil <[email protected]>
> Signed-off-by: Paul Burton <[email protected]>
>
> 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 <[email protected]>
> Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
> Reviewed-by: Stephen Boyd <[email protected]>
> Signed-off-by: Paul Burton <[email protected]>
>
> 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 <[email protected]>
> Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
> Reviewed-by: Stephen Boyd <[email protected]>
> Acked-by: James Hartley <[email protected]>
> Signed-off-by: Paul Burton <[email protected]>
>
> 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 <[email protected]>
> Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
> Reviewed-by: Stephen Boyd <[email protected]>
> Acked-by: John Crispin <[email protected]>
> Signed-off-by: Paul Burton <[email protected]>
Thanks,
Paul
[ This message was auto-generated; if you believe anything is incorrect
then please email [email protected] to report it. ]