Re: [PATCH v4 2/2] MIPS: ingenic: Enable JZ4780_NEMC manually

From: Krzysztof Kozlowski
Date: Tue Jul 28 2020 - 08:53:15 EST


On Tue, Jul 28, 2020 at 02:51:23PM +0200, Paul Cercueil wrote:
>
>
> Le mar. 28 juil. 2020 Ã 14:00, Krzysztof Kozlowski <krzk@xxxxxxxxxx> a Ãcrit
> :
> > On Tue, Jul 28, 2020 at 01:37:02PM +0200, Thomas Bogendoerfer wrote:
> > > On Tue, Jul 28, 2020 at 01:19:35PM +0200, Krzysztof Kozlowski wrote:
> > > > On Tue, Jul 28, 2020 at 01:12:11PM +0200, Paul Cercueil wrote:
> > > > > Hi Krzysztof,
> > > > >
> > > > > Le mar. 28 juil. 2020 Ã 12:45, Krzysztof Kozlowski
> > > <krzk@xxxxxxxxxx> a Ãcrit
> > > > > :
> > > > > > The CONFIG_JZ4780_NEMC was previously a default on MIPS but
> > > now it has
> > > > > > to be enabled manually.
> > > > > >
> > > > > > Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> > > > >
> > > > > I think you should swap the two so that there are no problems
> > > when
> > > > > bisecting.
> > > >
> > > > Good point. I was thinking that it will go via some of MIPS trees
> > > and
> > > > the patch #1 will just wait a cycle. However with acks, I can
> > > take it
> > > > through drivers/memory tree.
> > >
> > > I've acked the patch.
> > >
> > > Thomas.
> >
> > Thanks but now I noticed that one of changed configs
> > (arch/mips/configs/rs90_defconfig) is only in MIPS tree.
> >
> > I think it is easier then to take the patch #2 (configs) via MIPS and
> > wait with #1 for the next cycle or also take it via MIPS if it applies
> > cleanly.
>
> Why not take them both in the MIPS tree then? Would that conflict with
> changes in your tree?

Exactly (last part of my sentence). There should be no conflicts.

Best regards,
Krzysztof