From: Daniel Walker <dwalker@codeaurora.org>
To: Joe Perches <joe@perches.com>
Cc: Jiri Kosina <trivial@kernel.org>,
David Brown <davidb@codeaurora.org>,
Bryan Huntsman <bryanh@codeaurora.org>,
linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 09/10] drivers/video/msm/mddi.c: Remove multiple KERN_<level> uses
Date: Thu, 28 Oct 2010 10:08:36 -0700 [thread overview]
Message-ID: <1288285716.2683.12.camel@c-dwalke-linux.qualcomm.com> (raw)
In-Reply-To: <f1dbb5d550e2146508541335999374cf8910ff59.1288059486.git.joe@perches.com>
On Mon, 2010-10-25 at 19:44 -0700, Joe Perches wrote:
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/video/msm/mddi.c | 5 ++---
> 1 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/video/msm/mddi.c b/drivers/video/msm/mddi.c
> index 7c316c3..b66d86a 100644
> --- a/drivers/video/msm/mddi.c
> +++ b/drivers/video/msm/mddi.c
> @@ -318,7 +318,7 @@ static long mddi_wait_interrupt_timeout(struct mddi_info *mddi,
> static void mddi_wait_interrupt(struct mddi_info *mddi, uint32_t intmask)
> {
> if (mddi_wait_interrupt_timeout(mddi, intmask, HZ/10) == 0)
> - printk(KERN_INFO KERN_ERR "mddi_wait_interrupt %d, timeout "
> + printk(KERN_INFO "mddi_wait_interrupt %d, timeout "
> "waiting for %x, INT = %x, STAT = %x gotint = %x\n",
> current->pid, intmask, mddi_readl(INT), mddi_readl(STAT),
> mddi->got_int);
> @@ -465,8 +465,7 @@ static int __init mddi_get_client_caps(struct mddi_info *mddi)
>
> if (mddi->flags & FLAG_HAVE_CAPS)
> break;
> - printk(KERN_INFO KERN_ERR "mddi_init, timeout waiting for "
> - "caps\n");
> + printk(KERN_INFO "mddi_init, timeout waiting for caps\n");
> }
> return mddi->flags & FLAG_HAVE_CAPS;
> }
Applied to my tree, thanks.
Daniel
--
Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
prev parent reply other threads:[~2010-10-28 17:09 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-26 2:44 [PATCH 00/10] Remove multiple uses of KERN_<level> Joe Perches
2010-10-26 2:44 ` [PATCH 09/10] drivers/video/msm/mddi.c: Remove multiple KERN_<level> uses Joe Perches
2010-10-28 17:08 ` Daniel Walker [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1288285716.2683.12.camel@c-dwalke-linux.qualcomm.com \
--to=dwalker@codeaurora.org \
--cc=bryanh@codeaurora.org \
--cc=davidb@codeaurora.org \
--cc=joe@perches.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=trivial@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).