[PATCH] omap_hsmmc: fix missing parenthesis in pr_info

[PATCH] omap_hsmmc: fix missing parenthesis in pr_info

[Juergen Kilb]

In commit a3c76eb9 (mmc: replace printk with appropriate
display macro) the opening parenthesis of pr_info was missing.

Signed-off-by: Juergen Kilb <J.Kilb@phytec.de>
---
 drivers/mmc/host/omap_hsmmc.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
index e8ff123..101cd31 100644
--- a/drivers/mmc/host/omap_hsmmc.c
+++ b/drivers/mmc/host/omap_hsmmc.c
@@ -1270,7 +1270,7 @@ static void omap_hsmmc_protect_card(struct omap_hsmmc_host *host)
 		}
 	} else {
 		if (!host->protect_card) {
-			pr_info"%s: cover is open, "
+			pr_info("%s: cover is open, "
 					 "card is now inaccessible\n",
 					 mmc_hostname(host->mmc));
 			host->protect_card = 1;
-- 
1.7.0.4

Re: [PATCH] omap_hsmmc: fix missing parenthesis in pr_info

[Chris Ball]

Hi,

On Sat, Oct 29 2011, Juergen Kilb wrote:
> In commit a3c76eb9 (mmc: replace printk with appropriate
> display macro) the opening parenthesis of pr_info was missing.
>
> Signed-off-by: Juergen Kilb <J.Kilb@phytec.de>
> ---
>  drivers/mmc/host/omap_hsmmc.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
> index e8ff123..101cd31 100644
> --- a/drivers/mmc/host/omap_hsmmc.c
> +++ b/drivers/mmc/host/omap_hsmmc.c
> @@ -1270,7 +1270,7 @@ static void omap_hsmmc_protect_card(struct omap_hsmmc_host *host)
>  		}
>  	} else {
>  		if (!host->protect_card) {
> -			pr_info"%s: cover is open, "
> +			pr_info("%s: cover is open, "
>  					 "card is now inaccessible\n",
>  					 mmc_hostname(host->mmc));
>  			host->protect_card = 1;

Oops.  Thank you; I've pushed this to mmc-next now, and will send the
fix to Linus.  I've checked for other errors in the original patch.

- Chris.
-- 
Chris Ball   <cjb@laptop.org>   <http://printf.net/>
One Laptop Per Child

Re: [PATCH] omap_hsmmc: fix missing parenthesis in pr_info

[Kevin Hilman]

Hi Chris,

Chris Ball <cjb@laptop.org> writes:

[...]

> Oops.  Thank you; I've pushed this to mmc-next now, and will send the
> fix to Linus.  I've checked for other errors in the original patch.

Where did you push this fix?

I don't see this in the mmc-next branch of git://dev.laptop.org/users/cjb/mmc

Thanks,

Kevin

Re: [PATCH] omap_hsmmc: fix missing parenthesis in pr_info

[S, Venkatraman]

On Thu, Nov 3, 2011 at 5:29 AM, Kevin Hilman <khilman@ti.com> wrote:
> Hi Chris,
>
> Chris Ball <cjb@laptop.org> writes:
>
> [...]
>
>> Oops.  Thank you; I've pushed this to mmc-next now, and will send the
>> fix to Linus.  I've checked for other errors in the original patch.
>
> Where did you push this fix?
>
> I don't see this in the mmc-next branch of git://dev.laptop.org/users/cjb/mmc
>
MMC tree is back to kernel.org
http://git.kernel.org/?p=linux/kernel/git/cjb/mmc.git;a=summary

Re: [PATCH] omap_hsmmc: fix missing parenthesis in pr_info

[Chris Ball]

Hi,

On Thu, Nov 03 2011, S, Venkatraman wrote:
>>> Oops.  Thank you; I've pushed this to mmc-next now, and will send the
>>> fix to Linus.  I've checked for other errors in the original patch.
>>
>> Where did you push this fix?
>>
>> I don't see this in the mmc-next branch of git://dev.laptop.org/users/cjb/mmc
>>
> MMC tree is back to kernel.org
> http://git.kernel.org/?p=linux/kernel/git/cjb/mmc.git;a=summary

I've deleted the dev.laptop.org mmc tree now, so that there's a clue
that it's moved back to kernel.org.  (linux-next is already back to
pulling the kernel.org tree.)

Thanks,

- Chris.
-- 
Chris Ball   <cjb@laptop.org>   <http://printf.net/>
One Laptop Per Child
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html