[U-Boot] [RFC PATCH 17/20] cmd: mtd: add 'mtd' command

Miquel Raynal miquel.raynal at bootlin.com
Fri Jul 6 12:26:49 UTC 2018


Hi Jagan,

Jagan Teki <jagannadh.teki at gmail.com> wrote on Fri, 6 Jul 2018 17:08:57
+0530:

> On Wed, Jun 6, 2018 at 9:00 PM, Miquel Raynal <miquel.raynal at bootlin.com> wrote:
> > There should not be a 'nand' command, a 'sf' command and certainly not
> > another 'spi-nand'. Write a 'mtd' command instead to manage all MTD
> > devices at once. This should be the preferred way to access any MTD
> > device.  
> 
> So are you planning to integrate sf, nand command in future, adding
> them dm conversion of many stuff. I would like to reommend to go with
> spinand command. I have been through this and finally added spinor
> command, which I will going to in soon. once all are converted to dm,
> it would be easy and meaningfull to have common command.

I'm not sure we are in sync about this. The whole point of the previous
discussion was to decide whether or not we should make full use of
the MTD stack or not. I think it was pretty clear on the fact that
people prefer to be close to Linux's architecture on this regard.

MTD being an abstraction of the type of memory, I don't get the
point in creating yet another command each time a new type of
device is supported. The fact that all the drivers of these devices
register to the MTD layer makes it trivial to interact with. So why
should we wait?


Thanks,
Miquèl


More information about the U-Boot mailing list