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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



+ Arnd

On Thu, Dec 08, 2022 at 11:51:24AM +0100, Miquel Raynal wrote:
> marex@xxxxxxx wrote on Mon, 5 Dec 2022 17:26:53 +0100:
> > On 12/5/22 16:23, Francesco Dolcini wrote:
> > > From: Francesco Dolcini <francesco.dolcini@xxxxxxxxxxx>
> > > 
> > > This reverts commit 753395ea1e45c724150070b5785900b6a44bd5fb.
> > > 
> > > It introduced a boot regression on colibri-imx7, and potentially any
> > > other i.MX7 boards with MTD partition list generated into the fdt by
> > > U-Boot.
> > > 
> > > While the commit we are reverting here is not obviously wrong, it fixes
> > > only a dt binding checker warning that is non-functional, while it
> > > introduces a boot regression and there is no obvious fix ready.
> > > 
> > > Cc: stable@xxxxxxxxxxxxxxx
> > > Fixes: 753395ea1e45 ("ARM: dts: imx7: Fix NAND controller size-cells")
> > > Link: https://lore.kernel.org/all/Y4dgBTGNWpM6SQXI@xxxxxxxxxxxxxxxxxxxxxxxxxxxx/
> > > Link: https://lore.kernel.org/all/20221205144917.6514168a@xps-13/
> > > Signed-off-by: Francesco Dolcini <francesco.dolcini@xxxxxxxxxxx>
> [...]
> > Reviewed-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
> [...]
> > Acked-by: Marek Vasut <marex@xxxxxxx>
> [...]
> 
> As discussed in the above links, boot is broken on imx7 Colibri boards,
> this revert was the most quick and straightforward fix we agreed upon
> with the hope (~ duty?) it would make it in v6.1. Any chance you could
> pick this up rapidly and forward it to Linus? Or should we involve
> him directly (Thorsten?).

Hello Arnd,
FYI - see Miquel explanation above.

Francesco



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux