[U-Boot] [PATCH v7 13/13] cmd: mtdparts: describe as legacy

[Stefan Roese]

On 31.08.2018 16:57, Miquel Raynal wrote:
> The 'mtdparts' command is not needed anymore. While the environment
> variable is still valid (and useful), the command has been replaced by
> 'mtd' which is much more close to the MTD stack and do not add its own
> specific glue. The 'mtdids' variable, only used by the 'mtdparts'
> command is also useless if the right MTD device name is used in the
> 'mtdparts' variable.
> 
> Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
> ---
>   cmd/Kconfig | 10 +++++++++-
>   1 file changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/cmd/Kconfig b/cmd/Kconfig
> index 4deec0b238..0786663f4a 100644
> --- a/cmd/Kconfig
> +++ b/cmd/Kconfig
> @@ -1673,7 +1673,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.
>   
>   config MTDIDS_DEFAULT
>   	string "Default MTD IDs"
> @@ -1681,6 +1685,10 @@ config MTDIDS_DEFAULT
>   	help
>   	  Defines a default MTD IDs list for use with MTD partitions in the
>   	  Linux MTD command line partitions format.
> +	  Declaration of this environment variable is not useful
> +	  anymore when using the right MTD names in mtdparts along
> +	  with the use of the 'mtd' command instead of the legacy
> +	  'mtdparts'.
>   
>   config MTDPARTS_DEFAULT
>   	string "Default MTD partition scheme"
> 

Very nice work overall. :)

Reviewed-by: Stefan Roese <sr at denx.de>

Thanks,
Stefan