[PATCH] cmd: mmc: Drop trailing space in Name: before newline

[PATCH] cmd: mmc: Drop trailing space in Name: before newline

[Marek Vasut]

The Name: line of 'mmc info' command prints a trailing space before
newline. This is not useful and shows up as trailing space e.g. when
the output is checked into documentation. Remove the trailing space.

Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Jerome Forissier <jerome.forissier@arm.com> <jerome@forissier.org>
Cc: Mattijs Korpershoek <mkorpershoek@kernel.org>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: Yao Zi <me@ziyao.cc>
Cc: u-boot@lists.denx.de
---
 cmd/mmc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cmd/mmc.c b/cmd/mmc.c
index 6cb1ef5dc23..81b1ca4ad84 100644
--- a/cmd/mmc.c
+++ b/cmd/mmc.c
@@ -29,12 +29,12 @@ static void print_mmcinfo(struct mmc *mmc)
 	printf("Manufacturer ID: %x\n", mmc->cid[0] >> 24);
 	if (IS_SD(mmc)) {
 		printf("OEM: %x\n", (mmc->cid[0] >> 8) & 0xffff);
-		printf("Name: %c%c%c%c%c \n", mmc->cid[0] & 0xff,
+		printf("Name: %c%c%c%c%c\n", mmc->cid[0] & 0xff,
 		(mmc->cid[1] >> 24), (mmc->cid[1] >> 16) & 0xff,
 		(mmc->cid[1] >> 8) & 0xff, mmc->cid[1] & 0xff);
 	} else {
 		printf("OEM: %x\n", (mmc->cid[0] >> 8) & 0xff);
-		printf("Name: %c%c%c%c%c%c \n", mmc->cid[0] & 0xff,
+		printf("Name: %c%c%c%c%c%c\n", mmc->cid[0] & 0xff,
 		(mmc->cid[1] >> 24), (mmc->cid[1] >> 16) & 0xff,
 		(mmc->cid[1] >> 8) & 0xff, mmc->cid[1] & 0xff,
 		(mmc->cid[2] >> 24));
-- 
2.51.0

Re: [PATCH] cmd: mmc: Drop trailing space in Name: before newline

[Tom Rini]

[-- Attachment #1: Type: text/plain, Size: 405 bytes --]

On Sun, Feb 08, 2026 at 03:51:03PM +0100, Marek Vasut wrote:

> The Name: line of 'mmc info' command prints a trailing space before
> newline. This is not useful and shows up as trailing space e.g. when
> the output is checked into documentation. Remove the trailing space.
> 
> Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: [PATCH] cmd: mmc: Drop trailing space in Name: before newline

[Mattijs Korpershoek]

Hi Marek,

Thank you for the patch.

On Sun, Feb 08, 2026 at 15:51, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:

> The Name: line of 'mmc info' command prints a trailing space before
> newline. This is not useful and shows up as trailing space e.g. when
> the output is checked into documentation. Remove the trailing space.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>

Reviewed-by: Mattijs Korpershoek <mkorpershoek@kernel.org>

> ---
> Cc: Jaehoon Chung <jh80.chung@samsung.com>
> Cc: Jerome Forissier <jerome.forissier@arm.com> <jerome@forissier.org>
> Cc: Mattijs Korpershoek <mkorpershoek@kernel.org>
> Cc: Peng Fan <peng.fan@nxp.com>
> Cc: Tom Rini <trini@konsulko.com>
> Cc: Yao Zi <me@ziyao.cc>
> Cc: u-boot@lists.denx.de
> ---
>  cmd/mmc.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/cmd/mmc.c b/cmd/mmc.c
> index 6cb1ef5dc23..81b1ca4ad84 100644
> --- a/cmd/mmc.c
> +++ b/cmd/mmc.c
> @@ -29,12 +29,12 @@ static void print_mmcinfo(struct mmc *mmc)
>  	printf("Manufacturer ID: %x\n", mmc->cid[0] >> 24);
>  	if (IS_SD(mmc)) {
>  		printf("OEM: %x\n", (mmc->cid[0] >> 8) & 0xffff);
> -		printf("Name: %c%c%c%c%c \n", mmc->cid[0] & 0xff,
> +		printf("Name: %c%c%c%c%c\n", mmc->cid[0] & 0xff,
>  		(mmc->cid[1] >> 24), (mmc->cid[1] >> 16) & 0xff,
>  		(mmc->cid[1] >> 8) & 0xff, mmc->cid[1] & 0xff);
>  	} else {
>  		printf("OEM: %x\n", (mmc->cid[0] >> 8) & 0xff);
> -		printf("Name: %c%c%c%c%c%c \n", mmc->cid[0] & 0xff,
> +		printf("Name: %c%c%c%c%c%c\n", mmc->cid[0] & 0xff,
>  		(mmc->cid[1] >> 24), (mmc->cid[1] >> 16) & 0xff,
>  		(mmc->cid[1] >> 8) & 0xff, mmc->cid[1] & 0xff,
>  		(mmc->cid[2] >> 24));
> -- 
> 2.51.0

Re: [PATCH] cmd: mmc: Drop trailing space in Name: before newline

[Peng Fan (OSS)]

From: Peng Fan <peng.fan@nxp.com>


On Sun, 08 Feb 2026 15:51:03 +0100, Marek Vasut wrote:
> The Name: line of 'mmc info' command prints a trailing space before
> newline. This is not useful and shows up as trailing space e.g. when
> the output is checked into documentation. Remove the trailing space.
> 
> 

Applied to mmc-next-2026-02-26, thanks!

[1/1] cmd: mmc: Drop trailing space in Name: before newline
      commit: 6f7d976317c59aad1af0b8bb169adca9b5802ca0

Best regards,
-- 
Peng Fan <peng.fan@nxp.com>