From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Date: Thu, 6 Sep 2018 11:57:57 +0200 Subject: [U-Boot] [PATCH v7 12/12] cmd: mtdparts: describe as legacy In-Reply-To: <20180906070854.9717-13-miquel.raynal@bootlin.com> References: <20180906070854.9717-1-miquel.raynal@bootlin.com> <20180906070854.9717-13-miquel.raynal@bootlin.com> Message-ID: <20180906115757.49859925@bbrezillon> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, 6 Sep 2018 09:08:54 +0200 Miquel Raynal wrote: > The 'mtdparts' command is not needed anymore. While the environment > variable is still valid (and useful, along with the 'mtdids' one), the > command has been replaced by 'mtd' which is much more close to the MTD > stack and do not add its own specific glue. > > Signed-off-by: Miquel Raynal > Reviewed-by: Stefan Roese > --- > cmd/Kconfig | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/cmd/Kconfig b/cmd/Kconfig > index f82addca0a..51dd058128 100644 > --- a/cmd/Kconfig > +++ b/cmd/Kconfig > @@ -1680,7 +1680,11 @@ config CMD_MTDPARTS > bool "MTD partition support" > select MTD_DEVICE if (CMD_NAND || NAND) > help > - MTD partition support > + MTD partitioning tool support. > + It is strongly encouraged to avoid using this command > + anymore. One can still declare the partitions in the > + mtdparts environment variable but better use the MTD stack > + and the mtd command instead than this one. Well, that's only working if people stop using nand,onenand,sf,... commands in favor the mtd ones, otherwise they still need mtdparts, unfortunately :-(. I guess a long term goal could be to define aliases between flash specific commands and the generic mtd ones so that we can get rid of cmd/.c drivers. Regards, Boris