Re: [PATCH 6/9] ARM: dts: wheat: Drop MTD partitioning from DT

[Marek Vasut <marek.vasut@gmail.com>]

On 05/22/2018 04:43 PM, Geert Uytterhoeven wrote:
> On Tue, May 22, 2018 at 2:02 PM, Marek Vasut <marek.vasut@gmail.com> 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@gmail.com>
> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 


-- 
Best regards,
Marek Vasut