[U-Boot] [PATCH v13 5/7] cmd: mtd: add 'mtd' command

[Miquel Raynal]

Hi Adam,

Adam Ford <aford173 at gmail.com> wrote on Wed, 3 Oct 2018 07:35:15 -0500:

> On Mon, Oct 1, 2018 at 8:48 AM Miquel Raynal <miquel.raynal at bootlin.com> wrote:
> >
> > There should not be a 'nand' command, a 'sf' command and certainly not
> > a new 'spi-nand' command. Write a 'mtd' command instead to manage all
> > MTD devices/partitions at once. This should be the preferred way to
> > access any MTD device.  
> 
> What is the expected behavior when I type 'mtd list' on my omap37
> board, it just hangs.

What do you mean "hangs", does U-Boot crashes? Or is it really hanging
with no more on the console? Can you Ctrl-C to cancel the command or is
it really stuck?

> 
> I can use the nand read/write functions and mtdparts lists the
> partitions, so I know nand works.  My defconfig
> lists the partitions, so if we're not supposed to use mtdparts, where
> I do store the partition information?

You are not supposed to use the mtdpart _command_, but the mtdparts
_variable_ must be used in order to declare the partitions.

> 
> I intentionally removed it from the device tree a while ago, because
> U-Boot was passing the partition info to Linux.

Indeed, that's his primary role.

Thanks,
Miquèl