On 05/22/2018 04:43 PM, Geert Uytterhoeven wrote: > On Tue, May 22, 2018 at 2:02 PM, Marek Vasut <marek.vasut@xxxxxxxxx> wrote: >> Drop the MTD partitioning from DT, since it does not describe HW >> and to give way to a more flexible kernel command line partition >> passing. >> >> To retain the original partitioning, assure you have enabled >> CONFIG_MTD_CMDLINE_PARTS in your kernel config and add the >> following to your kernel command line: >> >> mtdparts=spi0.0:256k@0(loader),4096k(user),-(flash) > > I think the "@0" can be dropped, as it's optional? > 4m? My take on this is that the loader is actually at offset 0x0 of the MTD device and we explicitly state that in the mtdparts to anchor the first partition within the MTD device and all the other partitions are at offset +(sum of the sizes of all partitions listed before the current one) relative to that first partition. Removing the @0 feels fragile at best and it seems to depend on the current behavior of the code. > (Gaining more knowledge during reviewing ;-) > >> Signed-off-by: Marek Vasut <marek.vasut+renesas@xxxxxxxxx> > > Gr{oetje,eeting}s, > > Geert > -- Best regards, Marek Vasut