linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/11] treewide: Remove pr_<level> uses with KERN_<level>
@ 2010-09-12  5:10 Joe Perches
  2010-09-12  5:10 ` [PATCH 05/11] arch/powerpc: Remove pr_<level> uses of KERN_<level> Joe Perches
  2010-09-12  5:10 ` [PATCH 11/11] sound: " Joe Perches
  0 siblings, 2 replies; 6+ messages in thread
From: Joe Perches @ 2010-09-12  5:10 UTC (permalink / raw)
  To: Jiri Kosina
  Cc: linux-mips, alsa-devel, Marcelo Tosatti, kvm, Takashi Iwai,
	Steve Wise, Tom Tucker, Jaroslav Kysela, Paul Mackerras,
	H. Peter Anvin, Sean Hefty, devel, David Brownell, Russell King,
	linux-rdma, x86, Alexander Graf, Ingo Molnar, Avi Kivity,
	Hal Rosenstock, cbe-oss-dev, Mark Brown, microblaze-uclinux,
	kvm-ppc, Thomas Gleixner, linux-arm-kernel, Liam Girdwood,
	Michal Simek, Eric Miao, Haavard Skinnemoen, Geoff Levand, netdev,
	Greg Kroah-Hartman, linux-kernel, Ralf Baechle, linux-usb,
	linuxppc-dev

Most of these are pr_debug(KERN_<foo>.
pr_<level>(KERN_<level> is unnecessary.

Joe Perches (11):
  arch/arm: Remove pr_<level> uses of KERN_<level>
  arch/avr32: Remove pr_<level> uses of KERN_<level>
  arch/microblaze: Remove pr_<level> uses of KERN_<level>
  arch/mips: Remove pr_<level> uses of KERN_<level>
  arch/powerpc: Remove pr_<level> uses of KERN_<level>
  arch/x86: Remove pr_<level> uses of KERN_<level>
  drivers/infiniband: Remove pr_<level> uses of KERN_<level>
  drivers/net/skfp: Remove pr_<level> uses of KERN_<level>
  drivers/staging/msm: Remove pr_<level> uses of KERN_<level>
  drivers/usb/gadget: Remove pr_<level> uses of KERN_<level>
  sound: Remove pr_<level> uses of KERN_<level>

 arch/arm/mach-pxa/cpufreq-pxa2xx.c       |    3 +-
 arch/avr32/mach-at32ap/at32ap700x.c      |    4 +-
 arch/microblaze/kernel/exceptions.c      |   16 ++----
 arch/mips/kernel/irq-gic.c               |    2 +-
 arch/mips/pci/pci-rc32434.c              |    2 +-
 arch/powerpc/kvm/emulate.c               |    4 +-
 arch/powerpc/sysdev/pmi.c                |    2 +-
 arch/x86/kernel/apb_timer.c              |    4 +-
 drivers/infiniband/hw/amso1100/c2_intr.c |    4 +-
 drivers/net/skfp/skfddi.c                |   84 +++++++++++++++---------------
 drivers/staging/msm/staging-devices.c    |    2 +-
 drivers/usb/gadget/r8a66597-udc.c        |    2 +-
 sound/ppc/snd_ps3.c                      |    2 +-
 sound/soc/s3c24xx/s3c-dma.c              |    3 +-
 14 files changed, 64 insertions(+), 70 deletions(-)

-- 
1.7.3.rc1

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 05/11] arch/powerpc: Remove pr_<level> uses of KERN_<level>
  2010-09-12  5:10 [PATCH 00/11] treewide: Remove pr_<level> uses with KERN_<level> Joe Perches
@ 2010-09-12  5:10 ` Joe Perches
  2010-09-12  5:10 ` [PATCH 11/11] sound: " Joe Perches
  1 sibling, 0 replies; 6+ messages in thread
From: Joe Perches @ 2010-09-12  5:10 UTC (permalink / raw)
  To: Jiri Kosina
  Cc: kvm, Marcelo Tosatti, Alexander Graf, kvm-ppc, linux-kernel,
	Paul Mackerras, Avi Kivity, linuxppc-dev

Signed-off-by: Joe Perches <joe@perches.com>
---
 arch/powerpc/kvm/emulate.c |    4 ++--
 arch/powerpc/sysdev/pmi.c  |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/powerpc/kvm/emulate.c b/arch/powerpc/kvm/emulate.c
index 4568ec3..b83ba58 100644
--- a/arch/powerpc/kvm/emulate.c
+++ b/arch/powerpc/kvm/emulate.c
@@ -145,7 +145,7 @@ int kvmppc_emulate_instruction(struct kvm_run *run, struct kvm_vcpu *vcpu)
 	/* this default type might be overwritten by subcategories */
 	kvmppc_set_exit_type(vcpu, EMULATED_INST_EXITS);
 
-	pr_debug(KERN_INFO "Emulating opcode %d / %d\n", get_op(inst), get_xop(inst));
+	pr_debug("Emulating opcode %d / %d\n", get_op(inst), get_xop(inst));
 
 	switch (get_op(inst)) {
 	case OP_TRAP:
@@ -275,7 +275,7 @@ int kvmppc_emulate_instruction(struct kvm_run *run, struct kvm_vcpu *vcpu)
 			{
 				u64 jd = get_tb() - vcpu->arch.dec_jiffies;
 				kvmppc_set_gpr(vcpu, rt, vcpu->arch.dec - jd);
-				pr_debug(KERN_INFO "mfDEC: %x - %llx = %lx\n",
+				pr_debug("mfDEC: %x - %llx = %lx\n",
 					 vcpu->arch.dec, jd,
 					 kvmppc_get_gpr(vcpu, rt));
 				break;
diff --git a/arch/powerpc/sysdev/pmi.c b/arch/powerpc/sysdev/pmi.c
index 24a0bb9..4260f36 100644
--- a/arch/powerpc/sysdev/pmi.c
+++ b/arch/powerpc/sysdev/pmi.c
@@ -114,7 +114,7 @@ static void pmi_notify_handlers(struct work_struct *work)
 
 	spin_lock(&data->handler_spinlock);
 	list_for_each_entry(handler, &data->handler, node) {
-		pr_debug(KERN_INFO "pmi: notifying handler %p\n", handler);
+		pr_debug("pmi: notifying handler %p\n", handler);
 		if (handler->type == data->msg.type)
 			handler->handle_pmi_message(data->msg);
 	}
-- 
1.7.3.rc1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 11/11] sound: Remove pr_<level> uses of KERN_<level>
  2010-09-12  5:10 [PATCH 00/11] treewide: Remove pr_<level> uses with KERN_<level> Joe Perches
  2010-09-12  5:10 ` [PATCH 05/11] arch/powerpc: Remove pr_<level> uses of KERN_<level> Joe Perches
@ 2010-09-12  5:10 ` Joe Perches
  2010-09-13  8:25   ` Mark Brown
                     ` (2 more replies)
  1 sibling, 3 replies; 6+ messages in thread
From: Joe Perches @ 2010-09-12  5:10 UTC (permalink / raw)
  To: Jiri Kosina
  Cc: cbe-oss-dev, alsa-devel, Geoff Levand, Mark Brown, linux-kernel,
	Jaroslav Kysela, Takashi Iwai, linuxppc-dev, Liam Girdwood

Signed-off-by: Joe Perches <joe@perches.com>
---
 sound/ppc/snd_ps3.c         |    2 +-
 sound/soc/s3c24xx/s3c-dma.c |    3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/sound/ppc/snd_ps3.c b/sound/ppc/snd_ps3.c
index 2f12da4..581a670 100644
--- a/sound/ppc/snd_ps3.c
+++ b/sound/ppc/snd_ps3.c
@@ -579,7 +579,7 @@ static int snd_ps3_delay_to_bytes(struct snd_pcm_substream *substream,
 				  rate * delay_ms / 1000)
 		* substream->runtime->channels;
 
-	pr_debug(KERN_ERR "%s: time=%d rate=%d bytes=%ld, frames=%d, ret=%d\n",
+	pr_debug("%s: time=%d rate=%d bytes=%ld, frames=%d, ret=%d\n",
 		 __func__,
 		 delay_ms,
 		 rate,
diff --git a/sound/soc/s3c24xx/s3c-dma.c b/sound/soc/s3c24xx/s3c-dma.c
index 1b61c23..f1b1bc4 100644
--- a/sound/soc/s3c24xx/s3c-dma.c
+++ b/sound/soc/s3c24xx/s3c-dma.c
@@ -94,8 +94,7 @@ static void s3c_dma_enqueue(struct snd_pcm_substream *substream)
 
 		if ((pos + len) > prtd->dma_end) {
 			len  = prtd->dma_end - pos;
-			pr_debug(KERN_DEBUG "%s: corrected dma len %ld\n",
-			       __func__, len);
+			pr_debug("%s: corrected dma len %ld\n", __func__, len);
 		}
 
 		ret = s3c2410_dma_enqueue(prtd->params->channel,
-- 
1.7.3.rc1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 11/11] sound: Remove pr_<level> uses of KERN_<level>
  2010-09-12  5:10 ` [PATCH 11/11] sound: " Joe Perches
@ 2010-09-13  8:25   ` Mark Brown
  2010-09-13 17:55   ` Geoff Levand
  2010-09-13 21:41   ` Takashi Iwai
  2 siblings, 0 replies; 6+ messages in thread
From: Mark Brown @ 2010-09-13  8:25 UTC (permalink / raw)
  To: Joe Perches
  Cc: cbe-oss-dev, alsa-devel, Jiri Kosina, Geoff Levand, linux-kernel,
	Jaroslav Kysela, Takashi Iwai, linuxppc-dev, Liam Girdwood

On Sat, Sep 11, 2010 at 10:10:59PM -0700, Joe Perches wrote:
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
>  sound/ppc/snd_ps3.c         |    2 +-
>  sound/soc/s3c24xx/s3c-dma.c |    3 +--
>  2 files changed, 2 insertions(+), 3 deletions(-)

Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 11/11] sound: Remove pr_<level> uses of KERN_<level>
  2010-09-12  5:10 ` [PATCH 11/11] sound: " Joe Perches
  2010-09-13  8:25   ` Mark Brown
@ 2010-09-13 17:55   ` Geoff Levand
  2010-09-13 21:41   ` Takashi Iwai
  2 siblings, 0 replies; 6+ messages in thread
From: Geoff Levand @ 2010-09-13 17:55 UTC (permalink / raw)
  To: Joe Perches
  Cc: cbe-oss-dev, alsa-devel, Jiri Kosina, Takashi Iwai, Mark Brown,
	linux-kernel, Jaroslav Kysela, linuxppc-dev, Liam Girdwood

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

Hi Joe,

On 09/11/2010 10:10 PM, Joe Perches wrote:
>  sound/ppc/snd_ps3.c         |    2 +-
> 
> diff --git a/sound/ppc/snd_ps3.c b/sound/ppc/snd_ps3.c
> index 2f12da4..581a670 100644
> --- a/sound/ppc/snd_ps3.c
> +++ b/sound/ppc/snd_ps3.c
> @@ -579,7 +579,7 @@ static int snd_ps3_delay_to_bytes(struct snd_pcm_substream *substream,
>  				  rate * delay_ms / 1000)
>  		* substream->runtime->channels;
>  
> -	pr_debug(KERN_ERR "%s: time=%d rate=%d bytes=%ld, frames=%d, ret=%d\n",
> +	pr_debug("%s: time=%d rate=%d bytes=%ld, frames=%d, ret=%d\n",
>  		 __func__,
>  		 delay_ms,
>  		 rate,

snd_ps3.c looks OK.

Acked-by: Geoff Levand <geoff@infradead.org>

[-- Attachment #2: 0x2E141B16.asc --]
[-- Type: application/pgp-keys, Size: 9418 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 11/11] sound: Remove pr_<level> uses of KERN_<level>
  2010-09-12  5:10 ` [PATCH 11/11] sound: " Joe Perches
  2010-09-13  8:25   ` Mark Brown
  2010-09-13 17:55   ` Geoff Levand
@ 2010-09-13 21:41   ` Takashi Iwai
  2 siblings, 0 replies; 6+ messages in thread
From: Takashi Iwai @ 2010-09-13 21:41 UTC (permalink / raw)
  To: Joe Perches
  Cc: cbe-oss-dev, alsa-devel, Jiri Kosina, Geoff Levand, Mark Brown,
	linux-kernel, Jaroslav Kysela, linuxppc-dev, Liam Girdwood

At Sat, 11 Sep 2010 22:10:59 -0700,
Joe Perches wrote:
> 
> Signed-off-by: Joe Perches <joe@perches.com>

Applied to sound git tree now.  Thanks.


Takashi

> ---
>  sound/ppc/snd_ps3.c         |    2 +-
>  sound/soc/s3c24xx/s3c-dma.c |    3 +--
>  2 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/sound/ppc/snd_ps3.c b/sound/ppc/snd_ps3.c
> index 2f12da4..581a670 100644
> --- a/sound/ppc/snd_ps3.c
> +++ b/sound/ppc/snd_ps3.c
> @@ -579,7 +579,7 @@ static int snd_ps3_delay_to_bytes(struct snd_pcm_substream *substream,
>  				  rate * delay_ms / 1000)
>  		* substream->runtime->channels;
>  
> -	pr_debug(KERN_ERR "%s: time=%d rate=%d bytes=%ld, frames=%d, ret=%d\n",
> +	pr_debug("%s: time=%d rate=%d bytes=%ld, frames=%d, ret=%d\n",
>  		 __func__,
>  		 delay_ms,
>  		 rate,
> diff --git a/sound/soc/s3c24xx/s3c-dma.c b/sound/soc/s3c24xx/s3c-dma.c
> index 1b61c23..f1b1bc4 100644
> --- a/sound/soc/s3c24xx/s3c-dma.c
> +++ b/sound/soc/s3c24xx/s3c-dma.c
> @@ -94,8 +94,7 @@ static void s3c_dma_enqueue(struct snd_pcm_substream *substream)
>  
>  		if ((pos + len) > prtd->dma_end) {
>  			len  = prtd->dma_end - pos;
> -			pr_debug(KERN_DEBUG "%s: corrected dma len %ld\n",
> -			       __func__, len);
> +			pr_debug("%s: corrected dma len %ld\n", __func__, len);
>  		}
>  
>  		ret = s3c2410_dma_enqueue(prtd->params->channel,
> -- 
> 1.7.3.rc1
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2010-09-13 21:41 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-12  5:10 [PATCH 00/11] treewide: Remove pr_<level> uses with KERN_<level> Joe Perches
2010-09-12  5:10 ` [PATCH 05/11] arch/powerpc: Remove pr_<level> uses of KERN_<level> Joe Perches
2010-09-12  5:10 ` [PATCH 11/11] sound: " Joe Perches
2010-09-13  8:25   ` Mark Brown
2010-09-13 17:55   ` Geoff Levand
2010-09-13 21:41   ` Takashi Iwai

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).