Hi Marek,

Thank you for the patch.

On Sun, Feb 08, 2026 at 15:51, Marek Vasut <[email protected]> 
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 <[email protected]>

Reviewed-by: Mattijs Korpershoek <[email protected]>

> ---
> Cc: Jaehoon Chung <[email protected]>
> Cc: Jerome Forissier <[email protected]> <[email protected]>
> Cc: Mattijs Korpershoek <[email protected]>
> Cc: Peng Fan <[email protected]>
> Cc: Tom Rini <[email protected]>
> Cc: Yao Zi <[email protected]>
> Cc: [email protected]
> ---
>  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

Reply via email to