Patchwork [U-Boot] cmd_mmc.c: Fix typo, "dislay" -> "display"

login
register
mail settings
Submitter Robert P. J. Day
Date Nov. 11, 2012, 1:24 p.m.
Message ID <alpine.DEB.2.02.1211110822160.1083@oneiric>
Download mbox | patch
Permalink /patch/198283/
State Accepted
Delegated to: Anatolij Gustschin
Headers show

Comments

Robert P. J. Day - Nov. 11, 2012, 1:24 p.m.
Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

---
Benoît Thébaudeau - Nov. 11, 2012, 2:17 p.m.
Dear Robert P. J. Day,

On Sunday, November 11, 2012 2:24:44 PM, Robert P. J. Day wrote:
> Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> 
> ---
> 
> diff --git a/common/cmd_mmc.c b/common/cmd_mmc.c
> index 62a1c22..4c19df7 100644
> --- a/common/cmd_mmc.c
> +++ b/common/cmd_mmc.c
> @@ -144,7 +144,7 @@ static int do_mmcinfo(cmd_tbl_t *cmdtp, int flag,
> int argc, char * const argv[])
>  U_BOOT_CMD(
>  	mmcinfo, 1, 0, do_mmcinfo,
>  	"display MMC info",
> -	"- dislay info of the current MMC device"
> +	"- display info of the current MMC device"
>  );
> 
>  static int do_mmcops(cmd_tbl_t *cmdtp, int flag, int argc, char *
>  const argv[])

Thanks for fixing my typo! ;)

Cc-ing Andy (MMC custodian).

Best regards,
Benoît
Robert P. J. Day - Nov. 11, 2012, 2:45 p.m.
On Sun, 11 Nov 2012, Benoît Thébaudeau wrote:

... snip ...

> Thanks for fixing my typo! ;)
>
> Cc-ing Andy (MMC custodian).

  i checked the MAINTAINERS file and didn't see an official MMC
maintainer, else i would have CCed my original post.  andy fleming's
name doesn't appear in that file.  should it?

rday
Wolfgang Denk - Nov. 11, 2012, 3:20 p.m.
Dear Robert,

In message <alpine.DEB.2.02.1211110943450.23306@oneiric> you wrote:
>
>   i checked the MAINTAINERS file and didn't see an official MMC
> maintainer, else i would have CCed my original post.  andy fleming's
> name doesn't appear in that file.  should it?

No.  MAINTAINERS is the list of _board_ maintainers, as the header
should explain ("Maintainers for U-Boot board support").  In case of
sub-systems like MMC you need to look up the responsible custodian -
see here: http://www.denx.de/wiki/U-Boot/Custodians

Best regards,

Wolfgang Denk
Robert P. J. Day - Nov. 11, 2012, 3:24 p.m.
On Sun, 11 Nov 2012, Wolfgang Denk wrote:

> Dear Robert,
>
> In message <alpine.DEB.2.02.1211110943450.23306@oneiric> you wrote:
> >
> >   i checked the MAINTAINERS file and didn't see an official MMC
> > maintainer, else i would have CCed my original post.  andy fleming's
> > name doesn't appear in that file.  should it?
>
> No.  MAINTAINERS is the list of _board_ maintainers, as the header
> should explain ("Maintainers for U-Boot board support").  In case of
> sub-systems like MMC you need to look up the responsible custodian -
> see here: http://www.denx.de/wiki/U-Boot/Custodians

  ah, my mistake, apologies.  too used to kernel MAINTAINERS file ...

rday
Anatolij Gustschin - Nov. 14, 2012, 1:35 p.m.
Hi,

On Sun, 11 Nov 2012 08:24:44 -0500 (EST)
"Robert P. J. Day" <rpjday@crashcourse.ca> wrote:

> 
> Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> 
> ---
> 
> diff --git a/common/cmd_mmc.c b/common/cmd_mmc.c
> index 62a1c22..4c19df7 100644
> --- a/common/cmd_mmc.c
> +++ b/common/cmd_mmc.c
> @@ -144,7 +144,7 @@ static int do_mmcinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>  U_BOOT_CMD(
>  	mmcinfo, 1, 0, do_mmcinfo,
>  	"display MMC info",
> -	"- dislay info of the current MMC device"
> +	"- display info of the current MMC device"
>  );
> 
>  static int do_mmcops(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])

Applied to staging/agust@denx.de, thanks! Andy, I hope it is okay if
this patch goes via staging tree.

Anatolij

Patch

diff --git a/common/cmd_mmc.c b/common/cmd_mmc.c
index 62a1c22..4c19df7 100644
--- a/common/cmd_mmc.c
+++ b/common/cmd_mmc.c
@@ -144,7 +144,7 @@  static int do_mmcinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 U_BOOT_CMD(
 	mmcinfo, 1, 0, do_mmcinfo,
 	"display MMC info",
-	"- dislay info of the current MMC device"
+	"- display info of the current MMC device"
 );

 static int do_mmcops(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])