Re: [PATCH] mmc: Hide mmc speed command under mmc command

[Mattijs Korpershoek <mkorpershoek@baylibre.com>]

Hi Marek,

Thank you for the patch.

On jeu., août 29, 2024 at 03:22, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:

> The mmc speed command configuration option keeps showing up in
> Kconfig even if CMD_MMC is not set. Move MMC_SPEED_MODE_SET under
> CMD_MMC to show up the Kconfig option only if MMC command is enabled.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>

Reviewed-by: Mattijs Korpershoek <mkorpershoek@baylibre.com>

> ---
> Cc: AKASHI Takahiro <akashi.tkhro@gmail.com>
> Cc: Francis Laniel <francis.laniel@amarulasolutions.com>
> Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Cc: Mattijs Korpershoek <mkorpershoek@baylibre.com>
> Cc: Michal Simek <michal.simek@amd.com>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Tom Rini <trini@konsulko.com>
> Cc: u-boot@lists.denx.de
> ---
>  cmd/Kconfig | 19 +++++++++----------
>  1 file changed, 9 insertions(+), 10 deletions(-)
>
> diff --git a/cmd/Kconfig b/cmd/Kconfig
> index 978f44eda42..efe2357d044 100644
> --- a/cmd/Kconfig
> +++ b/cmd/Kconfig
> @@ -1399,6 +1399,15 @@ config CMD_MMC_SWRITE
>  	  Enable support for the "mmc swrite" command to write Android sparse
>  	  images to eMMC.
>  
> +config MMC_SPEED_MODE_SET
> +	bool "set speed mode using mmc command"
> +	help
> +	  Enable setting speed mode using mmc rescan and mmc dev commands.
> +	  The speed mode is provided as the last argument in these commands
> +	  and is indicated using the index from enum bus_mode in
> +	  include/mmc.h. A speed mode can be set only if it has already
> +	  been enabled in the device tree.
> +
>  endif
>  
>  config CMD_CLONE
> @@ -2963,16 +2972,6 @@ config CMD_UBIFS
>  	help
>  	  UBIFS is a file system for flash devices which works on top of UBI.
>  
> -config MMC_SPEED_MODE_SET
> -	bool "set speed mode using mmc command"
> -	depends on CMD_MMC
> -	help
> -	  Enable setting speed mode using mmc rescan and mmc dev commands.
> -	  The speed mode is provided as the last argument in these commands
> -	  and is indicated using the index from enum bus_mode in
> -	  include/mmc.h. A speed mode can be set only if it has already
> -	  been enabled in the device tree.
> -
>  config CMD_MESON
>  	bool "Amlogic Meson commands"
>  	depends on ARCH_MESON
> -- 
> 2.45.2