Re: [PATCH v4 5/6] arm: dts: pxa: use reworked NAND controller driver

[Boris Brezillon <boris.brezillon@bootlin.com>]

On Thu, 1 Mar 2018 10:04:46 +0100
Miquel Raynal <miquel.raynal@bootlin.com> wrote:

> Hi Robert,
> 
> On Tue, 27 Feb 2018 21:42:45 +0100, Boris Brezillon
> <boris.brezillon@bootlin.com> wrote:
> 
> > Hi Robert,
> > 
> > On Tue, 27 Feb 2018 21:09:51 +0100
> > Robert Jarzmik <robert.jarzmik@free.fr> wrote:
> >   
> > > Boris Brezillon <boris.brezillon@bootlin.com> writes:
> > >     
> > > > +Arnd and the LAKML
> > > >
> > > > Hi Miquel,
> > > >
> > > > Please try to Cc the LAKML and the arm-soc maintainers when you send DT
> > > > changes.
> > > >
> > > > Arnd, Robert, I'd like to apply this patch through the MTD tree since
> > > > it depends on patch 1 of this series ("mtd: nand: use Marvell reworked
> > > > NAND controller driver with all platforms").
> > > > Otherwise, I can leave it to Robert for 4.18.      
> > > 
> > > This one looks conflict free, so I have no objection if you carry it through
> > > your tree.    
> > 
> > Already prepared an immutable branch with all patches of this series
> > for you to pull in the pxa tree:
> > 
> > The following changes since commit 7928b2cbe55b2a410a0f5c1f154610059c57b1b2:
> > 
> >   Linux 4.16-rc1 (2018-02-11 15:04:29 -0800)
> > 
> > are available in the git repository at:
> > 
> >   git://git.infradead.org/linux-mtd.git nand/pxa3xx-removal
> > 
> > for you to fetch changes up to fa4d26b6f00e2c3460c2cc070f467c74d163bf40:
> > 
> >   MAINTAINERS: remove entry for deleted pxa3xx_nand driver (2018-02-23 22:12:15 +0100)
> >   
> 
> As you may know, this commit:
> 
>     mtd: nand: remove useless fields from pxa3xx NAND platform data
> 
> currently breaks linux-next build as I forgot to patch files in the
> mach-mmp directory.
> 
> I sent a fix for that yesterday to linux-arm-kernel@ and Boris:
> 
>     arm: mmp: Fix build issue due to PXA NAND pdata structure changes
> 
> but maybe, it would be better to squash the fix directly in Boris'
> immutable branch if you did not pull it already?

I guess Miquel meant squash it into the commit introducing the
regression ("mtd: nand: remove useless fields from pxa3xx NAND platform
data"), and I agree, if you haven't pulled the branch yet, I'd prefer to
create a new one addressing the issue. I could also add your Acked-by
at the same time ;-).

Thanks,

Boris

-- 
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com