On 9/3/24 9:42 AM, Mattijs Korpershoek wrote:
On lun., sept. 02, 2024 at 15:41, Quentin Schulz <quentin.sch...@cherry.de> 
wrote:

Hi Marek,

On 9/2/24 3:37 PM, Marek Vasut wrote:
The mmc speed command configuration option keeps showing up in
Kconfig directly in 'Command line interface'. Move MMC_SPEED_MODE_SET
under CMD_MMC to make it show up alongside the MMC command.

Signed-off-by: Marek Vasut <marek.vasut+rene...@mailbox.org>
---
Cc: AKASHI Takahiro <akashi.tk...@gmail.com>
Cc: Francis Laniel <francis.lan...@amarulasolutions.com>
Cc: Heinrich Schuchardt <xypron.g...@gmx.de>
Cc: Mattijs Korpershoek <mkorpersh...@baylibre.com>
Cc: Michal Simek <michal.si...@amd.com>
Cc: Quentin Schulz <quentin.sch...@cherry.de>
Cc: Simon Glass <s...@chromium.org>
Cc: Tom Rini <tr...@konsulko.com>
Cc: u-boot@lists.denx.de
---
V2: Rework the commit message

Better, thanks :)

---
   cmd/Kconfig | 19 +++++++++----------
   1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/cmd/Kconfig b/cmd/Kconfig
index 43f78a5aeb1..953e1f0ef2a 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -1406,6 +1406,15 @@ config CMD_MMC_SWRITE
          Enable support for the "mmc swrite" command to write Android sparse
          images to eMMC.
+config MMC_SPEED_MODE_SET

I'm wondering if we should prefix this with CMD_ maybe? It's used in
drivers/mmc/ though.

Anyway, not a blocker for this.

Reviewed-by: Quentin Schulz <quentin.sch...@cherry.de>

(You forgot Mattijs's Rb from v1 BTW)

Giving it again, thank you for noticing :)

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

Thank you both.

Reply via email to