On Fri, Jan 13, 2023 at 07:45:44PM +0100, Francesco Dolcini wrote: > From: Francesco Dolcini <[email protected]> > > Recently we had a boot regression on colibri-imx7 because of a cleanup change > on Linux imx7.dtsi setting nand controller node #size-cells from 1 to 0. > > Because of that Linux partition parser was no longer able to properly > parse the OF partitions leading to a boot failure, the above change was > reverted in the meantime as an immediate workaround, but some improvement > is required on both Linux and U-Boot. > > This change improve the U-Boot part of it, #size-cell is set to 1 when > it has an invalid value. This has the limitation to work only with devices > smaller than 4GiB. In general the suggestion from the Linux MTD maintainer > would > be to just deprecate using this U-Boot function and pass the MTD partitions > from the command line, unless they are statically defined in the DTS file > in the first place. > > This series therefore convert colibri-imx6ull and colibri-imx7 to pass the > partition list from the command line instead of fixing up the DT. > > Link: > https://lore.kernel.org/all/[email protected]/ > Link: > https://lore.kernel.org/all/[email protected]/
My higher level question / concern here is, is using one of the dts partition schemes still valid / preferred, or should everyone now have reverted to passing via the kernel command line? If device tree still, is mtd/partitions/fixed-partitions.yaml the one to follow or something else? -- Tom
signature.asc
Description: PGP signature
