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

[Thomas Bogendoerfer <tsbogend@alpha.franken.de>]

On Thu, May 07, 2020 at 11:11:10AM +0200, Geert Uytterhoeven wrote:
> Hi Thomas,
> 
> On Thu, May 7, 2020 at 11:05 AM Thomas Bogendoerfer
> <tsbogend@alpha.franken.de> wrote:
> > On Wed, May 06, 2020 at 11:04:52AM +0200, Geert Uytterhoeven wrote:
> > > The JZ4740 setup 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>.
> > >
> > > Fixes: f932449c11dabb4b ("MIPS: ingenic: Drop obsolete code, merge the rest in setup.c")
> > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > ---
> > >  arch/mips/jz4740/setup.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Does this need to go via mips-fixes ? Or is mips-next good enough ?
> 
> The commit fixed is not yet upstream, only in next, and the fix is
> non-critical.  It could even be merged into the original commit,
> depending on your rebase policy.

no rebase on mips-next. I've applied your patch to mips-next.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]