[PATCH v1] Revert "ARM: dts: imx7: Fix NAND controller size-cells"

Francesco Dolcini francesco at dolcini.it
Mon Dec 5 11:07:18 PST 2022


On Mon, Dec 05, 2022 at 07:52:08PM +0100, Marek Vasut wrote:
> On 12/5/22 19:18, Miquel Raynal wrote:
> > marex at denx.de wrote on Mon, 5 Dec 2022 19:07:14 +0100:
> > > On 12/5/22 18:58, Miquel Raynal wrote:
> > > > , it's not
> > > > complex to do, there are plenty of examples. This would be IMHO a
> > > > better step ahead rather than just a cell change. Anyway, I don't mind
> > > > reverting this once we've sorted this mess out and fixed U-Boot.
> > > 
> > > Won't we still have issues with older bootloader versions which
> > > paste partitions directly into this &gpmi {} node, and which needs
> > > to be fixed up in the parser in the end ?
> > 
> > I believe fdt_fixup_mtdparts() should be killed, so we should no longer
> > have this problem.
> 
> The fdt_fixup_mtdparts is U-Boot code. If contemporary Linux kernel is
> booted with ancient U-Boot, then you would still get defective DT passed to
> Linux, and that should be fixed up by Linux. Removing fdt_fixup_mtdparts()
> from current mainline U-Boot won't solve this problem.
> 
> I think this is also what Francesco is trying to convey (please correct me
> if I'm wrong).

Yes, exactly, thanks!

Francesco




More information about the linux-mtd mailing list