linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARC: Replace <linux/clk-provider.h> by <linux/of_clk.h>
@ 2020-02-12 10:00 Geert Uytterhoeven
  2020-02-12 17:06 ` Stephen Boyd
  2020-02-12 17:30 ` Vineet Gupta
  0 siblings, 2 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2020-02-12 10:00 UTC (permalink / raw)
  To: Vineet Gupta; +Cc: linux-snps-arc, linux-clk, linux-kernel, Geert Uytterhoeven

The ARC 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/arc/kernel/setup.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arc/kernel/setup.c b/arch/arc/kernel/setup.c
index e1c647490f00e910..aa41af6ef4ac6ecc 100644
--- a/arch/arc/kernel/setup.c
+++ b/arch/arc/kernel/setup.c
@@ -8,11 +8,11 @@
 #include <linux/delay.h>
 #include <linux/root_dev.h>
 #include <linux/clk.h>
-#include <linux/clk-provider.h>
 #include <linux/clocksource.h>
 #include <linux/console.h>
 #include <linux/module.h>
 #include <linux/cpu.h>
+#include <linux/of_clk.h>
 #include <linux/of_fdt.h>
 #include <linux/of.h>
 #include <linux/cache.h>
-- 
2.17.1


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

* Re: [PATCH] ARC: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:00 [PATCH] ARC: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
@ 2020-02-12 17:06 ` Stephen Boyd
  2020-02-12 17:30 ` Vineet Gupta
  1 sibling, 0 replies; 4+ messages in thread
From: Stephen Boyd @ 2020-02-12 17:06 UTC (permalink / raw)
  To: Geert Uytterhoeven, Vineet Gupta
  Cc: linux-snps-arc, linux-clk, linux-kernel, Geert Uytterhoeven

Quoting Geert Uytterhoeven (2020-02-12 02:00:47)
> The ARC 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] 4+ messages in thread

* Re: [PATCH] ARC: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 10:00 [PATCH] ARC: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
  2020-02-12 17:06 ` Stephen Boyd
@ 2020-02-12 17:30 ` Vineet Gupta
  2020-02-12 18:24   ` Geert Uytterhoeven
  1 sibling, 1 reply; 4+ messages in thread
From: Vineet Gupta @ 2020-02-12 17:30 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: linux-snps-arc, linux-clk, linux-kernel

On 2/12/20 2:00 AM, Geert Uytterhoeven wrote:
> The ARC 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>

Thx for this Geert. Do you want me to pick this up.

Thx,
-Vineet


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

* Re: [PATCH] ARC: Replace <linux/clk-provider.h> by <linux/of_clk.h>
  2020-02-12 17:30 ` Vineet Gupta
@ 2020-02-12 18:24   ` Geert Uytterhoeven
  0 siblings, 0 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2020-02-12 18:24 UTC (permalink / raw)
  To: Vineet Gupta; +Cc: Geert Uytterhoeven, linux-snps-arc, linux-clk, linux-kernel

Hi Vineet,

On Wed, Feb 12, 2020 at 6:30 PM Vineet Gupta <Vineet.Gupta1@synopsys.com> wrote:
> On 2/12/20 2:00 AM, Geert Uytterhoeven wrote:
> > The ARC 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>
>
> Thx for this Geert. Do you want me to pick this up.

Yes please. Thanks!

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] 4+ messages in thread

end of thread, other threads:[~2020-02-12 18:25 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-12 10:00 [PATCH] ARC: Replace <linux/clk-provider.h> by <linux/of_clk.h> Geert Uytterhoeven
2020-02-12 17:06 ` Stephen Boyd
2020-02-12 17:30 ` Vineet Gupta
2020-02-12 18:24   ` Geert Uytterhoeven

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