Re: [PATCH] nds32: Replace <linux/clk-provider.h> by <linux/of_clk.h>

[Geert Uytterhoeven]

Hi Greentime,

On Wed, Feb 12, 2020 at 11:52 AM Greentime Hu <green.hu@xxxxxxxxx> wrote:
> Geert Uytterhoeven <geert+renesas@xxxxxxxxx> æ 2020å2æ12æ éä äå6:16åéï
> > The Andes 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@xxxxxxxxx>
> > ---
> >  arch/nds32/kernel/time.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/nds32/kernel/time.c b/arch/nds32/kernel/time.c
> > index ac9d78ce3a818926..574a3d0a853980a9 100644
> > --- a/arch/nds32/kernel/time.c
> > +++ b/arch/nds32/kernel/time.c
> > @@ -2,7 +2,7 @@
> >  // Copyright (C) 2005-2017 Andes Technology Corporation
> >
> >  #include <linux/clocksource.h>
> > -#include <linux/clk-provider.h>
> > +#include <linux/of_clk.h>
> >
> >  void __init time_init(void)
> >  {
>
> Thank you, Geert.
>
> Let me know if you like to put in your tree or nds32's.
> Acked-by: Greentime Hu <green.hu@xxxxxxxxx>

Please take it in the nds32 tree.
Thanks!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

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