All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/3] ASoC: Reduce audio related kernel spew.
@ 2016-04-12 22:08 vedang.patel
  2016-04-12 22:08 ` [PATCH 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
                   ` (2 more replies)
  0 siblings, 3 replies; 27+ messages in thread
From: vedang.patel @ 2016-04-12 22:08 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

There have been complaints regarding audio related kernel messages flooding
dmesg. This series tries to address those complaints.

Also, I have created 3 different patches for this because they touch different
drivers. But, I can combine those to reduce the number of patches.

Vedang Patel (3):
  ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  ASoC: Intel: common: increase the loglevel of debug message.
  ASoC: Intel: Skylake: Increase loglevel of debug message.

 sound/soc/codecs/hdac_hdmi.c           | 2 ++
 sound/soc/intel/common/sst-dsp.c       | 2 +-
 sound/soc/intel/skylake/skl-messages.c | 2 ++
 sound/soc/intel/skylake/skl-sst-ipc.c  | 7 +++++--
 4 files changed, 10 insertions(+), 3 deletions(-)

-- 
2.1.2

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

* [PATCH 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  2016-04-12 22:08 [PATCH 0/3] ASoC: Reduce audio related kernel spew vedang.patel
@ 2016-04-12 22:08 ` vedang.patel
  2016-04-13  1:09   ` Takashi Sakamoto
  2016-04-12 22:08 ` [PATCH 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
  2016-04-12 22:08 ` [PATCH 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
  2 siblings, 1 reply; 27+ messages in thread
From: vedang.patel @ 2016-04-12 22:08 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

The hdac_hdmi codec driver prints the ELD information everytime an
external monitor is connected. Make it so that the information is only
printed when someone trying to debug the driver explicitly enables it.

print_hex_dump uses printk(KERN_DEBUG,... which is different from
dev_dbg used elsewhere in the driver: it's always enabled at
compile-time. Add #ifdef DEBUG condition for logging consistency.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/codecs/hdac_hdmi.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/sound/soc/codecs/hdac_hdmi.c b/sound/soc/codecs/hdac_hdmi.c
index aaa038f..653fd9e 100644
--- a/sound/soc/codecs/hdac_hdmi.c
+++ b/sound/soc/codecs/hdac_hdmi.c
@@ -1066,8 +1066,10 @@ static void hdac_hdmi_present_sense(struct hdac_hdmi_pin *pin, int repoll)
 				snd_jack_report(pcm->jack, SND_JACK_AVOUT);
 			}
 
+#ifdef DEBUG
 			print_hex_dump_bytes("ELD: ", DUMP_PREFIX_OFFSET,
 					pin->eld.eld_buffer, pin->eld.eld_size);
+#endif
 		} else {
 			pin->eld.monitor_present = false;
 			pin->eld.eld_valid = false;
-- 
2.1.2

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

* [PATCH 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status".
  2016-04-12 22:08 [PATCH 0/3] ASoC: Reduce audio related kernel spew vedang.patel
  2016-04-12 22:08 ` [PATCH 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
@ 2016-04-12 22:08 ` vedang.patel
  2016-06-27 16:40   ` Applied "ASoC: Intel: common: increase the loglevel of "FW Poll Status"." to the asoc tree Mark Brown
  2016-04-12 22:08 ` [PATCH 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
  2 siblings, 1 reply; 27+ messages in thread
From: vedang.patel @ 2016-04-12 22:08 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

For consistency with other log statements, change dev_info to dev_dbg
for a kernel print which is frequently printed by the driver.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/intel/common/sst-dsp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/intel/common/sst-dsp.c b/sound/soc/intel/common/sst-dsp.c
index b5bbdf4..ff2196e 100644
--- a/sound/soc/intel/common/sst-dsp.c
+++ b/sound/soc/intel/common/sst-dsp.c
@@ -285,7 +285,7 @@ int sst_dsp_register_poll(struct sst_dsp *ctx, u32 offset, u32 mask,
 	}
 
 	reg = sst_dsp_shim_read_unlocked(ctx, offset);
-	dev_info(ctx->dev, "FW Poll Status: reg=%#x %s %s\n", reg, operation,
+	dev_dbg(ctx->dev, "FW Poll Status: reg=%#x %s %s\n", reg, operation,
 			(time < timeout) ? "successful" : "timedout");
 	ret = time < timeout ? 0 : -ETIME;
 
-- 
2.1.2

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

* [PATCH 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages.
  2016-04-12 22:08 [PATCH 0/3] ASoC: Reduce audio related kernel spew vedang.patel
  2016-04-12 22:08 ` [PATCH 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
  2016-04-12 22:08 ` [PATCH 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
@ 2016-04-12 22:08 ` vedang.patel
  2016-04-26 23:03   ` [PATCH v2 0/3] ASoC: Reduce audio related kernel spew vedang.patel
  2 siblings, 1 reply; 27+ messages in thread
From: vedang.patel @ 2016-04-12 22:08 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

There is log spam while doing playback, record or reloading the
audio firmware.

print_hex_dump uses printk(KERN_DEBUG,... which is different from
dev_dbg used elsewhere in the driver: it's always enabled at
compile-time. Add #ifdef DEBUG condition for logging consistency.

For consistency with other log statements, change dev_info to dev_dbg
for a kernel print which is frequently printed by the driver.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/intel/skylake/skl-messages.c | 2 ++
 sound/soc/intel/skylake/skl-sst-ipc.c  | 7 +++++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/sound/soc/intel/skylake/skl-messages.c b/sound/soc/intel/skylake/skl-messages.c
index e3d149c..da2dbb2 100644
--- a/sound/soc/intel/skylake/skl-messages.c
+++ b/sound/soc/intel/skylake/skl-messages.c
@@ -730,8 +730,10 @@ static int skl_set_module_format(struct skl_sst *ctx,
 
 	dev_dbg(ctx->dev, "Module type=%d config size: %d bytes\n",
 			module_config->id.module_id, param_size);
+#ifdef DEBUG
 	print_hex_dump(KERN_DEBUG, "Module params:", DUMP_PREFIX_OFFSET, 8, 4,
 			*param_data, param_size, false);
+#endif
 	return 0;
 }
 
diff --git a/sound/soc/intel/skylake/skl-sst-ipc.c b/sound/soc/intel/skylake/skl-sst-ipc.c
index 5434602..5ee1420 100644
--- a/sound/soc/intel/skylake/skl-sst-ipc.c
+++ b/sound/soc/intel/skylake/skl-sst-ipc.c
@@ -363,7 +363,7 @@ static void skl_ipc_process_reply(struct sst_generic_ipc *ipc,
 	/* first process the header */
 	switch (reply) {
 	case IPC_GLB_REPLY_SUCCESS:
-		dev_info(ipc->dev, "ipc FW reply %x: success\n", header.primary);
+		dev_dbg(ipc->dev, "ipc FW reply %x: success\n", header.primary);
 		/* copy the rx data from the mailbox */
 		sst_dsp_inbox_read(ipc->dsp, msg->rx_data, msg->rx_size);
 		break;
@@ -688,12 +688,15 @@ int skl_ipc_init_instance(struct sst_generic_ipc *ipc,
 	struct skl_ipc_header header = {0};
 	u64 *ipc_header = (u64 *)(&header);
 	int ret;
-	u32 *buffer = (u32 *)param_data;
 	 /* param_block_size must be in dwords */
 	u16 param_block_size = msg->param_data_size / sizeof(u32);
 
+#ifdef DEBUG
+	u32 *buffer = (u32 *)param_data;
+
 	print_hex_dump(KERN_DEBUG, NULL, DUMP_PREFIX_NONE,
 		16, 4, buffer, param_block_size, false);
+#endif
 
 	header.primary = IPC_MSG_TARGET(IPC_MOD_MSG);
 	header.primary |= IPC_MSG_DIR(IPC_MSG_REQUEST);
-- 
2.1.2

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

* Re: [PATCH 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  2016-04-12 22:08 ` [PATCH 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
@ 2016-04-13  1:09   ` Takashi Sakamoto
  2016-04-14 15:15     ` Vedang Patel
  0 siblings, 1 reply; 27+ messages in thread
From: Takashi Sakamoto @ 2016-04-13  1:09 UTC (permalink / raw)
  To: vedang.patel, alsa-devel
  Cc: vinod.koul, jeeja.kp, marc.herbert, subhransu.s.prusty, liam.r.girdwood

Hi,

On Apr 13 2016 07:08, vedang.patel@intel.com wrote:
> From: Vedang Patel <vedang.patel@intel.com>
>
> The hdac_hdmi codec driver prints the ELD information everytime an
> external monitor is connected. Make it so that the information is only
> printed when someone trying to debug the driver explicitly enables it.

For this purpose, I think it better to use Linux tracing framework, 
instead of such an ancient fashion. The type of '__dynamic_array' is 
suitable for your aim.

As a quick glance of ASOC, there're several usage of the framework (see 
include/trace/events/asoc.h). So overall, I believe it's OK to use the 
framework.

I also have the same advice for your patch 3/3.

> print_hex_dump uses printk(KERN_DEBUG,... which is different from
> dev_dbg used elsewhere in the driver: it's always enabled at
> compile-time. Add #ifdef DEBUG condition for logging consistency.
>
> Signed-off-by: Vedang Patel <vedang.patel@intel.com>
> ---
>   sound/soc/codecs/hdac_hdmi.c | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/sound/soc/codecs/hdac_hdmi.c b/sound/soc/codecs/hdac_hdmi.c
> index aaa038f..653fd9e 100644
> --- a/sound/soc/codecs/hdac_hdmi.c
> +++ b/sound/soc/codecs/hdac_hdmi.c
> @@ -1066,8 +1066,10 @@ static void hdac_hdmi_present_sense(struct hdac_hdmi_pin *pin, int repoll)
>   				snd_jack_report(pcm->jack, SND_JACK_AVOUT);
>   			}
>
> +#ifdef DEBUG
>   			print_hex_dump_bytes("ELD: ", DUMP_PREFIX_OFFSET,
>   					pin->eld.eld_buffer, pin->eld.eld_size);
> +#endif
>   		} else {
>   			pin->eld.monitor_present = false;
>   			pin->eld.eld_valid = false;


Regards

Takashi Sakamoto

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

* Re: [PATCH 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  2016-04-13  1:09   ` Takashi Sakamoto
@ 2016-04-14 15:15     ` Vedang Patel
  2016-04-15 19:13       ` Vedang Patel
  0 siblings, 1 reply; 27+ messages in thread
From: Vedang Patel @ 2016-04-14 15:15 UTC (permalink / raw)
  To: Takashi Sakamoto, alsa-devel
  Cc: vinod.koul, jeeja.kp, marc.herbert, subhransu.s.prusty, liam.r.girdwood

On Wed, 2016-04-13 at 10:09 +0900, Takashi Sakamoto wrote:
> Hi,
> 
> On Apr 13 2016 07:08, vedang.patel@intel.com wrote:
> > From: Vedang Patel <vedang.patel@intel.com>
> > 
> > The hdac_hdmi codec driver prints the ELD information everytime an
> > external monitor is connected. Make it so that the information is
> > only
> > printed when someone trying to debug the driver explicitly enables
> > it.
> 
> For this purpose, I think it better to use Linux tracing framework, 
> instead of such an ancient fashion. The type of '__dynamic_array' is 
> suitable for your aim.
> 
> As a quick glance of ASOC, there're several usage of the framework
> (see 
> include/trace/events/asoc.h). So overall, I believe it's OK to use
> the 
> framework.
> 
> I also have the same advice for your patch 3/3.
> 

Thanks Takashi for the input. I will work on it and send out the new
patches soon.

-Vedang

> > print_hex_dump uses printk(KERN_DEBUG,... which is different from
> > dev_dbg used elsewhere in the driver: it's always enabled at
> > compile-time. Add #ifdef DEBUG condition for logging consistency.
> > 
> > Signed-off-by: Vedang Patel <vedang.patel@intel.com>
> > ---
> >   sound/soc/codecs/hdac_hdmi.c | 2 ++
> >   1 file changed, 2 insertions(+)
> > 
> > diff --git a/sound/soc/codecs/hdac_hdmi.c
> > b/sound/soc/codecs/hdac_hdmi.c
> > index aaa038f..653fd9e 100644
> > --- a/sound/soc/codecs/hdac_hdmi.c
> > +++ b/sound/soc/codecs/hdac_hdmi.c
> > @@ -1066,8 +1066,10 @@ static void hdac_hdmi_present_sense(struct
> > hdac_hdmi_pin *pin, int repoll)
> >   				snd_jack_report(pcm->jack,
> > SND_JACK_AVOUT);
> >   			}
> > 
> > +#ifdef DEBUG
> >   			print_hex_dump_bytes("ELD: ",
> > DUMP_PREFIX_OFFSET,
> >   					pin->eld.eld_buffer, pin
> > ->eld.eld_size);
> > +#endif
> >   		} else {
> >   			pin->eld.monitor_present = false;
> >   			pin->eld.eld_valid = false;
> 
> 
> Regards
> 
> Takashi Sakamoto

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

* Re: [PATCH 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  2016-04-14 15:15     ` Vedang Patel
@ 2016-04-15 19:13       ` Vedang Patel
  0 siblings, 0 replies; 27+ messages in thread
From: Vedang Patel @ 2016-04-15 19:13 UTC (permalink / raw)
  To: Takashi Sakamoto, alsa-devel
  Cc: vinod.koul, jeeja.kp, marc.herbert, subhransu.s.prusty, liam.r.girdwood

On Thu, 2016-04-14 at 08:15 -0700, Vedang Patel wrote:
> On Wed, 2016-04-13 at 10:09 +0900, Takashi Sakamoto wrote:
> > Hi,
> > 
> > On Apr 13 2016 07:08, vedang.patel@intel.com wrote:
> > > From: Vedang Patel <vedang.patel@intel.com>
> > > 
> > > The hdac_hdmi codec driver prints the ELD information everytime
> > > an
> > > external monitor is connected. Make it so that the information is
> > > only
> > > printed when someone trying to debug the driver explicitly
> > > enables
> > > it.
> > 
> > For this purpose, I think it better to use Linux tracing framework,
> > instead of such an ancient fashion. The type of '__dynamic_array'
> > is 
> > suitable for your aim.
> > 
> > As a quick glance of ASOC, there're several usage of the framework
> > (see 
> > include/trace/events/asoc.h). So overall, I believe it's OK to use
> > the 
> > framework.
> > 
> > I also have the same advice for your patch 3/3.
> > 
> 
> Thanks Takashi for the input. I will work on it and send out the new
> patches soon.
> 
> -Vedang
> 
Hi Takashi, 

I modified the dynamic tracing framework for this. But, i could not
find a way to present the debug messages. 

Following is how it was printed using print_hex_dump: 

Module params:00000000: 000186a0 000000c0 00000180 00000000
Module params:00000010: 0000bb80 00000010 ffffff10 00000001
Module params:00000020: 00000000 00001002 0000bb80 00000020
Module params:00000030: ffffff10 00000001 00000000 00002002
Module params:00000040: 00000000 00000000 00000300 00000000
Module params:00000050: 00000000

>From the tracing framework, I used __dynamic_array along with
__print_hex as follows: 

TP_printk("%s: %s",
	__get_str(prefix),
	__print_array(__get_dynamic_array(hex_data), 
			__entry->length, 4))

and I got: 

cras-1410  [002] ...1   640.673815: skl_adsp_module_params_dump: Module
params: a0 86 01 00 80 01 00 00 80 01 00 00 00 00 00 00 80 bb 00 00 20
00 00 00 10 ff ff ff 01 00 00 00 00 00 00 00 02 18 00 00 80 bb 00 00 20
00 00 00 10 ff ff ff 01 00 00 00 00 00 00 00 02 18 00 00 00 00 00 00 02
0c 00 00 00 03 00 00 15 00 00 00 00 00 00 00 10 ff ff ff 32 ff ff ff 10
32 ff ff 10 32 ff ff 10 32 ff ff 10 32 ff ff 10 32 ff ff 10 32 ff ff 37
09 d0 81 00 00 70 c0 00 00 00 00 00 00 99 02 03 00 00 00 03 00 00 00 02
40 00 00 00 00 00 00 00 0f 07 07 20 00 00 00 01 00 00 00 ff 0f 00 00 00
00 00 00

Ignoring the endianness problem , I think that the previous format is
more readable compared to the one in tracing framework. Also, I just
found out that there is a print_hex_dump_debug function which can also
be enabled dynamically. Do you think using print_hex_dump_debug is a
good idea in this scenario?

Thanks,
Vedang

> > > print_hex_dump uses printk(KERN_DEBUG,... which is different from
> > > dev_db
> > > g used elsewhere in the driver: it's always enabled at
> > > compile-time. Add
> > > #ifdef DEBUG condition for logging consistency.
> > > 
> > > Signed-off-by: Vedang Patel <vedang.patel@intel.com>
> > > ---
> > >   sound/soc/codecs/hdac_hdmi.c | 2 ++
> > >   1 file changed, 2 insertions(+)
> > > 
> > > diff --git a/sound/soc/codecs/hdac_hdmi.c
> > > b/sound/soc/codecs/hdac_hdmi.c
> > > index aaa038f..653fd9e 100644
> > > --- a/sound/soc/codecs/hdac_hdmi.c
> > > +++ b/sound/soc/codecs/hdac_hdmi.c
> > > @@ -1066,8 +1066,10 @@ static void hdac_hdmi_present_sense(struct
> > > hdac_hdmi_pin *pin, int repoll)
> > >   > > > 	> > > 	> > > 	> > > 	> > > snd_jack_report(pcm->jack,
> > > SND_JACK_AVOUT);
> > >   > > > 	> > > 	> > > 	> > > }
> > > 
> > > +#ifdef DEBUG
> > >   > > > 	> > > 	> > > 	> > > print_hex_dump_bytes("ELD: ",
> > > DUMP_PREFIX_OFFSET,
> > >   > > > 	> > > 	> > > 	> > > 	> > > 	> > > pin->eld.eld_buffer, pin
> > > ->eld.eld_size);
> > > +#endif
> > >   > > > 	> > > 	> > > } else {
> > >   > > > 	> > > 	> > > 	> > > pin->eld.monitor_present = false;
> > >   > > > 	> > > 	> > > 	> > > pin->eld.eld_valid = false;
> > 
> > 
> > Regards
> > 
> > Takashi Sakamoto
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

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

* [PATCH v2 0/3] ASoC: Reduce audio related kernel spew.
  2016-04-12 22:08 ` [PATCH 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
@ 2016-04-26 23:03   ` vedang.patel
  2016-04-26 23:03     ` [PATCH v2 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
                       ` (2 more replies)
  0 siblings, 3 replies; 27+ messages in thread
From: vedang.patel @ 2016-04-26 23:03 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

There have been complaints regarding audio related kernel messages flooding
dmesg. This series tries to address those complaints.

Also, I have created 3 different patches for this because they touch different
drivers. But, I can combine those to reduce the number of patches.

v2 Changes:
- use print_hex_dump_debug instead of #ifdef DEBUG

Vedang Patel (3):
  ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  ASoC: Intel: common: increase the loglevel of "FW Poll Status".
  ASoC: Intel: Skylake: Increase loglevel of debug messages.

 sound/soc/codecs/hdac_hdmi.c           | 5 +++--
 sound/soc/intel/common/sst-dsp.c       | 2 +-
 sound/soc/intel/skylake/skl-messages.c | 2 +-
 sound/soc/intel/skylake/skl-sst-ipc.c  | 4 ++--
 4 files changed, 7 insertions(+), 6 deletions(-)

-- 
2.1.2

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

* [PATCH v2 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  2016-04-26 23:03   ` [PATCH v2 0/3] ASoC: Reduce audio related kernel spew vedang.patel
@ 2016-04-26 23:03     ` vedang.patel
  2016-04-26 23:03     ` [PATCH v2 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
  2016-04-26 23:03     ` [PATCH v2 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
  2 siblings, 0 replies; 27+ messages in thread
From: vedang.patel @ 2016-04-26 23:03 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

The hdac_hdmi codec driver prints the ELD information everytime an
external monitor is connected. Make it so that the information is only
printed when someone trying to debug the driver explicitly enables it.

print_hex_dump_bytes (which just calls print_hex_dump) uses
printk(KERN_DEBUG,... which is different from dev_dbg used elsewhere in
the driver: it's always enabled at compile-time. Change it to
print_hex_dump_debug for logging consistency.

Change-Id: I6a7ac1d40a73369ea5dc58b7afde0288b8e86558
Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/codecs/hdac_hdmi.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/sound/soc/codecs/hdac_hdmi.c b/sound/soc/codecs/hdac_hdmi.c
index aaa038f..def565d 100644
--- a/sound/soc/codecs/hdac_hdmi.c
+++ b/sound/soc/codecs/hdac_hdmi.c
@@ -1066,8 +1066,9 @@ static void hdac_hdmi_present_sense(struct hdac_hdmi_pin *pin, int repoll)
 				snd_jack_report(pcm->jack, SND_JACK_AVOUT);
 			}
 
-			print_hex_dump_bytes("ELD: ", DUMP_PREFIX_OFFSET,
-					pin->eld.eld_buffer, pin->eld.eld_size);
+			print_hex_dump_debug("ELD: ",
+					DUMP_PREFIX_OFFSET, 16, 1,
+					pin->eld.eld_buffer, pin->eld.eld_size, true);
 		} else {
 			pin->eld.monitor_present = false;
 			pin->eld.eld_valid = false;
-- 
2.1.2

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

* [PATCH v2 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status".
  2016-04-26 23:03   ` [PATCH v2 0/3] ASoC: Reduce audio related kernel spew vedang.patel
  2016-04-26 23:03     ` [PATCH v2 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
@ 2016-04-26 23:03     ` vedang.patel
  2016-04-26 23:03     ` [PATCH v2 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
  2 siblings, 0 replies; 27+ messages in thread
From: vedang.patel @ 2016-04-26 23:03 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

For consistency with other log statements, change dev_info to dev_dbg
for a kernel print which is frequently printed by the driver.

Change-Id: I26fbd7bb66b68f4319d08b6bd4e604b519df1a9e
Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/intel/common/sst-dsp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/intel/common/sst-dsp.c b/sound/soc/intel/common/sst-dsp.c
index b5bbdf4..ff2196e 100644
--- a/sound/soc/intel/common/sst-dsp.c
+++ b/sound/soc/intel/common/sst-dsp.c
@@ -285,7 +285,7 @@ int sst_dsp_register_poll(struct sst_dsp *ctx, u32 offset, u32 mask,
 	}
 
 	reg = sst_dsp_shim_read_unlocked(ctx, offset);
-	dev_info(ctx->dev, "FW Poll Status: reg=%#x %s %s\n", reg, operation,
+	dev_dbg(ctx->dev, "FW Poll Status: reg=%#x %s %s\n", reg, operation,
 			(time < timeout) ? "successful" : "timedout");
 	ret = time < timeout ? 0 : -ETIME;
 
-- 
2.1.2

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

* [PATCH v2 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages.
  2016-04-26 23:03   ` [PATCH v2 0/3] ASoC: Reduce audio related kernel spew vedang.patel
  2016-04-26 23:03     ` [PATCH v2 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
  2016-04-26 23:03     ` [PATCH v2 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
@ 2016-04-26 23:03     ` vedang.patel
  2016-06-01  1:14       ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew vedang.patel
                         ` (2 more replies)
  2 siblings, 3 replies; 27+ messages in thread
From: vedang.patel @ 2016-04-26 23:03 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

There is log spam while doing playback, record or reloading the
audio firmware.

print_hex_dump uses printk(KERN_DEBUG,... which is different from
dev_dbg used elsewhere in the driver: it's always enabled at
compile-time. Change it to print_hex_dump_debug for logging consistency.

For consistency with other log statements, change dev_info to dev_dbg
for a kernel print which is frequently printed by the driver.

Change-Id: Ib3f29a02e79eabbd7a4452279841514277a471c4
Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/intel/skylake/skl-messages.c | 2 +-
 sound/soc/intel/skylake/skl-sst-ipc.c  | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/sound/soc/intel/skylake/skl-messages.c b/sound/soc/intel/skylake/skl-messages.c
index e3d149c..b208f56 100644
--- a/sound/soc/intel/skylake/skl-messages.c
+++ b/sound/soc/intel/skylake/skl-messages.c
@@ -730,7 +730,7 @@ static int skl_set_module_format(struct skl_sst *ctx,
 
 	dev_dbg(ctx->dev, "Module type=%d config size: %d bytes\n",
 			module_config->id.module_id, param_size);
-	print_hex_dump(KERN_DEBUG, "Module params:", DUMP_PREFIX_OFFSET, 8, 4,
+	print_hex_dump_debug("Module params:", DUMP_PREFIX_OFFSET, 8, 4,
 			*param_data, param_size, false);
 	return 0;
 }
diff --git a/sound/soc/intel/skylake/skl-sst-ipc.c b/sound/soc/intel/skylake/skl-sst-ipc.c
index 5434602..c141f24 100644
--- a/sound/soc/intel/skylake/skl-sst-ipc.c
+++ b/sound/soc/intel/skylake/skl-sst-ipc.c
@@ -363,7 +363,7 @@ static void skl_ipc_process_reply(struct sst_generic_ipc *ipc,
 	/* first process the header */
 	switch (reply) {
 	case IPC_GLB_REPLY_SUCCESS:
-		dev_info(ipc->dev, "ipc FW reply %x: success\n", header.primary);
+		dev_dbg(ipc->dev, "ipc FW reply %x: success\n", header.primary);
 		/* copy the rx data from the mailbox */
 		sst_dsp_inbox_read(ipc->dsp, msg->rx_data, msg->rx_size);
 		break;
@@ -692,7 +692,7 @@ int skl_ipc_init_instance(struct sst_generic_ipc *ipc,
 	 /* param_block_size must be in dwords */
 	u16 param_block_size = msg->param_data_size / sizeof(u32);
 
-	print_hex_dump(KERN_DEBUG, NULL, DUMP_PREFIX_NONE,
+	print_hex_dump_debug(NULL, DUMP_PREFIX_NONE,
 		16, 4, buffer, param_block_size, false);
 
 	header.primary = IPC_MSG_TARGET(IPC_MOD_MSG);
-- 
2.1.2

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

* [PATCH v3 0/3] ASoC: Reduce audio related kernel spew.
  2016-04-26 23:03     ` [PATCH v2 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
@ 2016-06-01  1:14       ` vedang.patel
  2016-06-01  1:14         ` [PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
                           ` (3 more replies)
  2016-06-25  0:37       ` [REPOST REPOST PATCH " vedang.patel
  2016-06-27 16:40       ` Applied "ASoC: Intel: Skylake: Increase loglevel of debug messages." to the asoc tree Mark Brown
  2 siblings, 4 replies; 27+ messages in thread
From: vedang.patel @ 2016-06-01  1:14 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

There have been complaints regarding audio related kernel messages flooding
dmesg. This series tries to address those complaints.

Also, I have created 3 different patches for this because they touch different
drivers. But, I can combine those to reduce the number of patches.

v2 Changes:
- Use print_hex_dump_debug instead of #ifdef DEBUG

v3 Changes:
- Remove Change-Id from all the patches.

Vedang Patel (3):
  ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  ASoC: Intel: common: increase the loglevel of "FW Poll Status".
  ASoC: Intel: Skylake: Increase loglevel of debug messages.

 sound/soc/codecs/hdac_hdmi.c           |    6 ++++--
 sound/soc/intel/common/sst-dsp.c       |    2 +-
 sound/soc/intel/skylake/skl-messages.c |    2 +-
 sound/soc/intel/skylake/skl-sst-ipc.c  |    4 ++--
 4 files changed, 8 insertions(+), 6 deletions(-)

-- 
1.7.9.5

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

* [PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  2016-06-01  1:14       ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew vedang.patel
@ 2016-06-01  1:14         ` vedang.patel
  2016-06-27 16:40           ` Applied "ASoC: hdac_hdmi: Increase loglevel of hex dump printed" to the asoc tree Mark Brown
  2016-06-01  1:15         ` [PATCH v3 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
                           ` (2 subsequent siblings)
  3 siblings, 1 reply; 27+ messages in thread
From: vedang.patel @ 2016-06-01  1:14 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

The hdac_hdmi codec driver prints the ELD information everytime an
external monitor is connected. Make it so that the information is only
printed when someone trying to debug the driver explicitly enables it.

print_hex_dump_bytes (which just calls print_hex_dump) uses
printk(KERN_DEBUG,... which is different from dev_dbg used elsewhere in
the driver: it's always enabled at compile-time. Change it to
print_hex_dump_debug for logging consistency.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/codecs/hdac_hdmi.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/sound/soc/codecs/hdac_hdmi.c b/sound/soc/codecs/hdac_hdmi.c
index 181cd3b..62d2181 100644
--- a/sound/soc/codecs/hdac_hdmi.c
+++ b/sound/soc/codecs/hdac_hdmi.c
@@ -1124,8 +1124,10 @@ static void hdac_hdmi_present_sense(struct hdac_hdmi_pin *pin, int repoll)
 			}
 			hdac_hdmi_parse_eld(edev, pin);
 
-			print_hex_dump_bytes("ELD: ", DUMP_PREFIX_OFFSET,
-					pin->eld.eld_buffer, pin->eld.eld_size);
+			print_hex_dump_debug("ELD: ",
+					DUMP_PREFIX_OFFSET, 16, 1,
+					pin->eld.eld_buffer, pin->eld.eld_size,
+					true);
 		} else {
 			pin->eld.monitor_present = false;
 			pin->eld.eld_valid = false;
-- 
1.7.9.5

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

* [PATCH v3 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status".
  2016-06-01  1:14       ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew vedang.patel
  2016-06-01  1:14         ` [PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
@ 2016-06-01  1:15         ` vedang.patel
  2016-06-01  1:15         ` [PATCH v3 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
  2016-06-01  5:56         ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew Vinod Koul
  3 siblings, 0 replies; 27+ messages in thread
From: vedang.patel @ 2016-06-01  1:15 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

For consistency with other log statements, change dev_info to dev_dbg
for a kernel print which is frequently printed by the driver.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/intel/common/sst-dsp.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/intel/common/sst-dsp.c b/sound/soc/intel/common/sst-dsp.c
index b5bbdf4..ff2196e 100644
--- a/sound/soc/intel/common/sst-dsp.c
+++ b/sound/soc/intel/common/sst-dsp.c
@@ -285,7 +285,7 @@ int sst_dsp_register_poll(struct sst_dsp *ctx, u32 offset, u32 mask,
 	}
 
 	reg = sst_dsp_shim_read_unlocked(ctx, offset);
-	dev_info(ctx->dev, "FW Poll Status: reg=%#x %s %s\n", reg, operation,
+	dev_dbg(ctx->dev, "FW Poll Status: reg=%#x %s %s\n", reg, operation,
 			(time < timeout) ? "successful" : "timedout");
 	ret = time < timeout ? 0 : -ETIME;
 
-- 
1.7.9.5

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

* [PATCH v3 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages.
  2016-06-01  1:14       ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew vedang.patel
  2016-06-01  1:14         ` [PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
  2016-06-01  1:15         ` [PATCH v3 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
@ 2016-06-01  1:15         ` vedang.patel
  2016-06-01  5:56         ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew Vinod Koul
  3 siblings, 0 replies; 27+ messages in thread
From: vedang.patel @ 2016-06-01  1:15 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

There is log spam while doing playback, record or reloading the
audio firmware.

print_hex_dump uses printk(KERN_DEBUG,... which is different from
dev_dbg used elsewhere in the driver: it's always enabled at
compile-time. Change it to print_hex_dump_debug for logging consistency.

For consistency with other log statements, change dev_info to dev_dbg
for a kernel print which is frequently printed by the driver.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/intel/skylake/skl-messages.c |    2 +-
 sound/soc/intel/skylake/skl-sst-ipc.c  |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/sound/soc/intel/skylake/skl-messages.c b/sound/soc/intel/skylake/skl-messages.c
index 226db84..81224de 100644
--- a/sound/soc/intel/skylake/skl-messages.c
+++ b/sound/soc/intel/skylake/skl-messages.c
@@ -730,7 +730,7 @@ static int skl_set_module_format(struct skl_sst *ctx,
 
 	dev_dbg(ctx->dev, "Module type=%d config size: %d bytes\n",
 			module_config->id.module_id, param_size);
-	print_hex_dump(KERN_DEBUG, "Module params:", DUMP_PREFIX_OFFSET, 8, 4,
+	print_hex_dump_debug("Module params:", DUMP_PREFIX_OFFSET, 8, 4,
 			*param_data, param_size, false);
 	return 0;
 }
diff --git a/sound/soc/intel/skylake/skl-sst-ipc.c b/sound/soc/intel/skylake/skl-sst-ipc.c
index 5434602..c141f24 100644
--- a/sound/soc/intel/skylake/skl-sst-ipc.c
+++ b/sound/soc/intel/skylake/skl-sst-ipc.c
@@ -363,7 +363,7 @@ static void skl_ipc_process_reply(struct sst_generic_ipc *ipc,
 	/* first process the header */
 	switch (reply) {
 	case IPC_GLB_REPLY_SUCCESS:
-		dev_info(ipc->dev, "ipc FW reply %x: success\n", header.primary);
+		dev_dbg(ipc->dev, "ipc FW reply %x: success\n", header.primary);
 		/* copy the rx data from the mailbox */
 		sst_dsp_inbox_read(ipc->dsp, msg->rx_data, msg->rx_size);
 		break;
@@ -692,7 +692,7 @@ int skl_ipc_init_instance(struct sst_generic_ipc *ipc,
 	 /* param_block_size must be in dwords */
 	u16 param_block_size = msg->param_data_size / sizeof(u32);
 
-	print_hex_dump(KERN_DEBUG, NULL, DUMP_PREFIX_NONE,
+	print_hex_dump_debug(NULL, DUMP_PREFIX_NONE,
 		16, 4, buffer, param_block_size, false);
 
 	header.primary = IPC_MSG_TARGET(IPC_MOD_MSG);
-- 
1.7.9.5

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

* Re: [PATCH v3 0/3] ASoC: Reduce audio related kernel spew.
  2016-06-01  1:14       ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew vedang.patel
                           ` (2 preceding siblings ...)
  2016-06-01  1:15         ` [PATCH v3 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
@ 2016-06-01  5:56         ` Vinod Koul
  3 siblings, 0 replies; 27+ messages in thread
From: Vinod Koul @ 2016-06-01  5:56 UTC (permalink / raw)
  To: vedang.patel
  Cc: jeeja.kp, alsa-devel, subhransu.s.prusty, marc.herbert, liam.r.girdwood

On Tue, May 31, 2016 at 06:14:58PM -0700, vedang.patel@intel.com wrote:
> From: Vedang Patel <vedang.patel@intel.com>
> 
> There have been complaints regarding audio related kernel messages flooding
> dmesg. This series tries to address those complaints.
> 
> Also, I have created 3 different patches for this because they touch different
> drivers. But, I can combine those to reduce the number of patches.
Split sound fine to me

Acked-by: Vinod Koul <vinod.koul@intel.com>

-- 
~Vinod

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

* [REPOST REPOST PATCH v3 0/3] ASoC: Reduce audio related kernel spew.
  2016-04-26 23:03     ` [PATCH v2 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
  2016-06-01  1:14       ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew vedang.patel
@ 2016-06-25  0:37       ` vedang.patel
  2016-06-25  0:37         ` [REPOST REPOST PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
                           ` (3 more replies)
  2016-06-27 16:40       ` Applied "ASoC: Intel: Skylake: Increase loglevel of debug messages." to the asoc tree Mark Brown
  2 siblings, 4 replies; 27+ messages in thread
From: vedang.patel @ 2016-06-25  0:37 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, broonie, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

There have been complaints regarding audio related kernel messages flooding
dmesg. This series tries to address those complaints.

Also, I have created 3 different patches for this because they touch different
drivers. But, I can combine those to reduce the number of patches.

v2 Changes:
- Use print_hex_dump_debug instead of #ifdef DEBUG

v3 Changes:
- Remove Change-Id from all the patches.

Vedang Patel (3):
  ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  ASoC: Intel: common: increase the loglevel of "FW Poll Status".
  ASoC: Intel: Skylake: Increase loglevel of debug messages.

 sound/soc/codecs/hdac_hdmi.c           |    6 ++++--
 sound/soc/intel/common/sst-dsp.c       |    2 +-
 sound/soc/intel/skylake/skl-messages.c |    2 +-
 sound/soc/intel/skylake/skl-sst-ipc.c  |    4 ++--
 4 files changed, 8 insertions(+), 6 deletions(-)

-- 
1.7.9.5

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

* [REPOST REPOST PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed
  2016-06-25  0:37       ` [REPOST REPOST PATCH " vedang.patel
@ 2016-06-25  0:37         ` vedang.patel
  2016-06-25  0:37         ` [REPOST REPOST PATCH v3 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
                           ` (2 subsequent siblings)
  3 siblings, 0 replies; 27+ messages in thread
From: vedang.patel @ 2016-06-25  0:37 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, broonie, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

The hdac_hdmi codec driver prints the ELD information everytime an
external monitor is connected. Make it so that the information is only
printed when someone trying to debug the driver explicitly enables it.

print_hex_dump_bytes (which just calls print_hex_dump) uses
printk(KERN_DEBUG,... which is different from dev_dbg used elsewhere in
the driver: it's always enabled at compile-time. Change it to
print_hex_dump_debug for logging consistency.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/codecs/hdac_hdmi.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/sound/soc/codecs/hdac_hdmi.c b/sound/soc/codecs/hdac_hdmi.c
index 181cd3b..62d2181 100644
--- a/sound/soc/codecs/hdac_hdmi.c
+++ b/sound/soc/codecs/hdac_hdmi.c
@@ -1124,8 +1124,10 @@ static void hdac_hdmi_present_sense(struct hdac_hdmi_pin *pin, int repoll)
 			}
 			hdac_hdmi_parse_eld(edev, pin);
 
-			print_hex_dump_bytes("ELD: ", DUMP_PREFIX_OFFSET,
-					pin->eld.eld_buffer, pin->eld.eld_size);
+			print_hex_dump_debug("ELD: ",
+					DUMP_PREFIX_OFFSET, 16, 1,
+					pin->eld.eld_buffer, pin->eld.eld_size,
+					true);
 		} else {
 			pin->eld.monitor_present = false;
 			pin->eld.eld_valid = false;
-- 
1.7.9.5

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

* [REPOST REPOST PATCH v3 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status".
  2016-06-25  0:37       ` [REPOST REPOST PATCH " vedang.patel
  2016-06-25  0:37         ` [REPOST REPOST PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
@ 2016-06-25  0:37         ` vedang.patel
  2016-06-25  0:37         ` [REPOST REPOST PATCH v3 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
  2016-06-27 14:50         ` [REPOST REPOST PATCH v3 0/3] ASoC: Reduce audio related kernel spew Mark Brown
  3 siblings, 0 replies; 27+ messages in thread
From: vedang.patel @ 2016-06-25  0:37 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, broonie, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

For consistency with other log statements, change dev_info to dev_dbg
for a kernel print which is frequently printed by the driver.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/intel/common/sst-dsp.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/intel/common/sst-dsp.c b/sound/soc/intel/common/sst-dsp.c
index b5bbdf4..ff2196e 100644
--- a/sound/soc/intel/common/sst-dsp.c
+++ b/sound/soc/intel/common/sst-dsp.c
@@ -285,7 +285,7 @@ int sst_dsp_register_poll(struct sst_dsp *ctx, u32 offset, u32 mask,
 	}
 
 	reg = sst_dsp_shim_read_unlocked(ctx, offset);
-	dev_info(ctx->dev, "FW Poll Status: reg=%#x %s %s\n", reg, operation,
+	dev_dbg(ctx->dev, "FW Poll Status: reg=%#x %s %s\n", reg, operation,
 			(time < timeout) ? "successful" : "timedout");
 	ret = time < timeout ? 0 : -ETIME;
 
-- 
1.7.9.5

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

* [REPOST REPOST PATCH v3 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages.
  2016-06-25  0:37       ` [REPOST REPOST PATCH " vedang.patel
  2016-06-25  0:37         ` [REPOST REPOST PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
  2016-06-25  0:37         ` [REPOST REPOST PATCH v3 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
@ 2016-06-25  0:37         ` vedang.patel
  2016-06-27 14:50         ` [REPOST REPOST PATCH v3 0/3] ASoC: Reduce audio related kernel spew Mark Brown
  3 siblings, 0 replies; 27+ messages in thread
From: vedang.patel @ 2016-06-25  0:37 UTC (permalink / raw)
  To: alsa-devel
  Cc: vinod.koul, marc.herbert, broonie, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty, Vedang Patel

From: Vedang Patel <vedang.patel@intel.com>

There is log spam while doing playback, record or reloading the
audio firmware.

print_hex_dump uses printk(KERN_DEBUG,... which is different from
dev_dbg used elsewhere in the driver: it's always enabled at
compile-time. Change it to print_hex_dump_debug for logging consistency.

For consistency with other log statements, change dev_info to dev_dbg
for a kernel print which is frequently printed by the driver.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
---
 sound/soc/intel/skylake/skl-messages.c |    2 +-
 sound/soc/intel/skylake/skl-sst-ipc.c  |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/sound/soc/intel/skylake/skl-messages.c b/sound/soc/intel/skylake/skl-messages.c
index 226db84..81224de 100644
--- a/sound/soc/intel/skylake/skl-messages.c
+++ b/sound/soc/intel/skylake/skl-messages.c
@@ -730,7 +730,7 @@ static int skl_set_module_format(struct skl_sst *ctx,
 
 	dev_dbg(ctx->dev, "Module type=%d config size: %d bytes\n",
 			module_config->id.module_id, param_size);
-	print_hex_dump(KERN_DEBUG, "Module params:", DUMP_PREFIX_OFFSET, 8, 4,
+	print_hex_dump_debug("Module params:", DUMP_PREFIX_OFFSET, 8, 4,
 			*param_data, param_size, false);
 	return 0;
 }
diff --git a/sound/soc/intel/skylake/skl-sst-ipc.c b/sound/soc/intel/skylake/skl-sst-ipc.c
index 5434602..c141f24 100644
--- a/sound/soc/intel/skylake/skl-sst-ipc.c
+++ b/sound/soc/intel/skylake/skl-sst-ipc.c
@@ -363,7 +363,7 @@ static void skl_ipc_process_reply(struct sst_generic_ipc *ipc,
 	/* first process the header */
 	switch (reply) {
 	case IPC_GLB_REPLY_SUCCESS:
-		dev_info(ipc->dev, "ipc FW reply %x: success\n", header.primary);
+		dev_dbg(ipc->dev, "ipc FW reply %x: success\n", header.primary);
 		/* copy the rx data from the mailbox */
 		sst_dsp_inbox_read(ipc->dsp, msg->rx_data, msg->rx_size);
 		break;
@@ -692,7 +692,7 @@ int skl_ipc_init_instance(struct sst_generic_ipc *ipc,
 	 /* param_block_size must be in dwords */
 	u16 param_block_size = msg->param_data_size / sizeof(u32);
 
-	print_hex_dump(KERN_DEBUG, NULL, DUMP_PREFIX_NONE,
+	print_hex_dump_debug(NULL, DUMP_PREFIX_NONE,
 		16, 4, buffer, param_block_size, false);
 
 	header.primary = IPC_MSG_TARGET(IPC_MOD_MSG);
-- 
1.7.9.5

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

* Re: [REPOST REPOST PATCH v3 0/3] ASoC: Reduce audio related kernel spew.
  2016-06-25  0:37       ` [REPOST REPOST PATCH " vedang.patel
                           ` (2 preceding siblings ...)
  2016-06-25  0:37         ` [REPOST REPOST PATCH v3 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
@ 2016-06-27 14:50         ` Mark Brown
  2016-06-29 23:39           ` Patel, Vedang
  3 siblings, 1 reply; 27+ messages in thread
From: Mark Brown @ 2016-06-27 14:50 UTC (permalink / raw)
  To: vedang.patel
  Cc: alsa-devel, vinod.koul, marc.herbert, liam.r.girdwood, jeeja.kp,
	subhransu.s.prusty


[-- Attachment #1.1: Type: text/plain, Size: 541 bytes --]

On Fri, Jun 24, 2016 at 05:37:08PM -0700, vedang.patel@intel.com wrote:
> From: Vedang Patel <vedang.patel@intel.com>
> 
> There have been complaints regarding audio related kernel messages flooding
> dmesg. This series tries to address those complaints.

Please don't include noise like REPOST in the subject line, especially
not at the start of the subject line where it makes it look less like a
patch e-mail and so harder to spot in an inbox.  I also note that this
appears to be the first time you've sent me these patches...

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



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

* Applied "ASoC: Intel: Skylake: Increase loglevel of debug messages." to the asoc tree
  2016-04-26 23:03     ` [PATCH v2 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
  2016-06-01  1:14       ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew vedang.patel
  2016-06-25  0:37       ` [REPOST REPOST PATCH " vedang.patel
@ 2016-06-27 16:40       ` Mark Brown
  2 siblings, 0 replies; 27+ messages in thread
From: Mark Brown @ 2016-06-27 16:40 UTC (permalink / raw)
  To: Vedang Patel
  Cc: alsa-devel, vinod.koul, marc.herbert, Mark Brown,
	liam.r.girdwood, jeeja.kp, subhransu.s.prusty

The patch

   ASoC: Intel: Skylake: Increase loglevel of debug messages.

has been applied to the asoc tree at

   git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git 

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.  

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

>From 91c1832579700891747820862633f9a8d0d81fa4 Mon Sep 17 00:00:00 2001
From: Vedang Patel <vedang.patel@intel.com>
Date: Fri, 24 Jun 2016 17:37:11 -0700
Subject: [PATCH] ASoC: Intel: Skylake: Increase loglevel of debug messages.

There is log spam while doing playback, record or reloading the
audio firmware.

print_hex_dump uses printk(KERN_DEBUG,... which is different from
dev_dbg used elsewhere in the driver: it's always enabled at
compile-time. Change it to print_hex_dump_debug for logging consistency.

For consistency with other log statements, change dev_info to dev_dbg
for a kernel print which is frequently printed by the driver.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/intel/skylake/skl-messages.c | 2 +-
 sound/soc/intel/skylake/skl-sst-ipc.c  | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/sound/soc/intel/skylake/skl-messages.c b/sound/soc/intel/skylake/skl-messages.c
index 804091aa6e64..6902020df946 100644
--- a/sound/soc/intel/skylake/skl-messages.c
+++ b/sound/soc/intel/skylake/skl-messages.c
@@ -730,7 +730,7 @@ static int skl_set_module_format(struct skl_sst *ctx,
 
 	dev_dbg(ctx->dev, "Module type=%d config size: %d bytes\n",
 			module_config->id.module_id, param_size);
-	print_hex_dump(KERN_DEBUG, "Module params:", DUMP_PREFIX_OFFSET, 8, 4,
+	print_hex_dump_debug("Module params:", DUMP_PREFIX_OFFSET, 8, 4,
 			*param_data, param_size, false);
 	return 0;
 }
diff --git a/sound/soc/intel/skylake/skl-sst-ipc.c b/sound/soc/intel/skylake/skl-sst-ipc.c
index 543460293b00..c141f24cae05 100644
--- a/sound/soc/intel/skylake/skl-sst-ipc.c
+++ b/sound/soc/intel/skylake/skl-sst-ipc.c
@@ -363,7 +363,7 @@ static void skl_ipc_process_reply(struct sst_generic_ipc *ipc,
 	/* first process the header */
 	switch (reply) {
 	case IPC_GLB_REPLY_SUCCESS:
-		dev_info(ipc->dev, "ipc FW reply %x: success\n", header.primary);
+		dev_dbg(ipc->dev, "ipc FW reply %x: success\n", header.primary);
 		/* copy the rx data from the mailbox */
 		sst_dsp_inbox_read(ipc->dsp, msg->rx_data, msg->rx_size);
 		break;
@@ -692,7 +692,7 @@ int skl_ipc_init_instance(struct sst_generic_ipc *ipc,
 	 /* param_block_size must be in dwords */
 	u16 param_block_size = msg->param_data_size / sizeof(u32);
 
-	print_hex_dump(KERN_DEBUG, NULL, DUMP_PREFIX_NONE,
+	print_hex_dump_debug(NULL, DUMP_PREFIX_NONE,
 		16, 4, buffer, param_block_size, false);
 
 	header.primary = IPC_MSG_TARGET(IPC_MOD_MSG);
-- 
2.8.1

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

* Applied "ASoC: Intel: common: increase the loglevel of "FW Poll Status"." to the asoc tree
  2016-04-12 22:08 ` [PATCH 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
@ 2016-06-27 16:40   ` Mark Brown
  0 siblings, 0 replies; 27+ messages in thread
From: Mark Brown @ 2016-06-27 16:40 UTC (permalink / raw)
  To: Vedang Patel
  Cc: alsa-devel, vinod.koul, marc.herbert, Mark Brown,
	liam.r.girdwood, jeeja.kp, subhransu.s.prusty

The patch

   ASoC: Intel: common: increase the loglevel of "FW Poll Status".

has been applied to the asoc tree at

   git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git 

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.  

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

>From ef06b6f3912f4438f9275922dae17c11360ceefc Mon Sep 17 00:00:00 2001
From: Vedang Patel <vedang.patel@intel.com>
Date: Fri, 24 Jun 2016 17:37:10 -0700
Subject: [PATCH] ASoC: Intel: common: increase the loglevel of "FW Poll
 Status".

For consistency with other log statements, change dev_info to dev_dbg
for a kernel print which is frequently printed by the driver.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/intel/common/sst-dsp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/intel/common/sst-dsp.c b/sound/soc/intel/common/sst-dsp.c
index b5bbdf4fe93a..ff2196ef359f 100644
--- a/sound/soc/intel/common/sst-dsp.c
+++ b/sound/soc/intel/common/sst-dsp.c
@@ -285,7 +285,7 @@ int sst_dsp_register_poll(struct sst_dsp *ctx, u32 offset, u32 mask,
 	}
 
 	reg = sst_dsp_shim_read_unlocked(ctx, offset);
-	dev_info(ctx->dev, "FW Poll Status: reg=%#x %s %s\n", reg, operation,
+	dev_dbg(ctx->dev, "FW Poll Status: reg=%#x %s %s\n", reg, operation,
 			(time < timeout) ? "successful" : "timedout");
 	ret = time < timeout ? 0 : -ETIME;
 
-- 
2.8.1

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

* Applied "ASoC: hdac_hdmi: Increase loglevel of hex dump printed" to the asoc tree
  2016-06-01  1:14         ` [PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
@ 2016-06-27 16:40           ` Mark Brown
  0 siblings, 0 replies; 27+ messages in thread
From: Mark Brown @ 2016-06-27 16:40 UTC (permalink / raw)
  To: Vedang Patel
  Cc: alsa-devel, vinod.koul, marc.herbert, Mark Brown,
	liam.r.girdwood, jeeja.kp, subhransu.s.prusty

The patch

   ASoC: hdac_hdmi: Increase loglevel of hex dump printed

has been applied to the asoc tree at

   git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git 

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.  

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

>From 3cb7cec14415ff8544ae702f396f913cd9008e7e Mon Sep 17 00:00:00 2001
From: Vedang Patel <vedang.patel@intel.com>
Date: Fri, 24 Jun 2016 17:37:09 -0700
Subject: [PATCH] ASoC: hdac_hdmi: Increase loglevel of hex dump printed

The hdac_hdmi codec driver prints the ELD information everytime an
external monitor is connected. Make it so that the information is only
printed when someone trying to debug the driver explicitly enables it.

print_hex_dump_bytes (which just calls print_hex_dump) uses
printk(KERN_DEBUG,... which is different from dev_dbg used elsewhere in
the driver: it's always enabled at compile-time. Change it to
print_hex_dump_debug for logging consistency.

Signed-off-by: Vedang Patel <vedang.patel@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/hdac_hdmi.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/sound/soc/codecs/hdac_hdmi.c b/sound/soc/codecs/hdac_hdmi.c
index 181cd3bf0b92..62d21812b9b8 100644
--- a/sound/soc/codecs/hdac_hdmi.c
+++ b/sound/soc/codecs/hdac_hdmi.c
@@ -1124,8 +1124,10 @@ static void hdac_hdmi_present_sense(struct hdac_hdmi_pin *pin, int repoll)
 			}
 			hdac_hdmi_parse_eld(edev, pin);
 
-			print_hex_dump_bytes("ELD: ", DUMP_PREFIX_OFFSET,
-					pin->eld.eld_buffer, pin->eld.eld_size);
+			print_hex_dump_debug("ELD: ",
+					DUMP_PREFIX_OFFSET, 16, 1,
+					pin->eld.eld_buffer, pin->eld.eld_size,
+					true);
 		} else {
 			pin->eld.monitor_present = false;
 			pin->eld.eld_valid = false;
-- 
2.8.1

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

* Re: [REPOST REPOST PATCH v3 0/3] ASoC: Reduce audio related kernel spew.
  2016-06-27 14:50         ` [REPOST REPOST PATCH v3 0/3] ASoC: Reduce audio related kernel spew Mark Brown
@ 2016-06-29 23:39           ` Patel, Vedang
  2016-06-30  2:27             ` Vinod Koul
  0 siblings, 1 reply; 27+ messages in thread
From: Patel, Vedang @ 2016-06-29 23:39 UTC (permalink / raw)
  To: broonie
  Cc: alsa-devel, Koul, Vinod, Herbert, Marc, Girdwood, Liam R, Kp,
	Jeeja, Prusty, Subhransu S

Thanks Mark for merging the patches. I apologize for any unwanted noise
in your inbox.

I did post the patches a few weeks back (http://www.spinics.net/lists/a
lsa-devel/msg50782.html). The only difference this time was I cc'ed you
in the patches. Moving forward, what is the preferred method for
submitting patches for the first time or reposting them so that you are
less likely to miss them?

Thanks,
Vedang

On Mon, 2016-06-27 at 15:50 +0100, Mark Brown wrote:
> On Fri, Jun 24, 2016 at 05:37:08PM -0700, vedang.patel@intel.com
> wrote:
> > 
> > From: Vedang Patel <vedang.patel@intel.com>
> > 
> > There have been complaints regarding audio related kernel messages
> > flooding
> > dmesg. This series tries to address those complaints.
> Please don't include noise like REPOST in the subject line,
> especially
> not at the start of the subject line where it makes it look less like
> a
> patch e-mail and so harder to spot in an inbox.  I also note that
> this
> appears to be the first time you've sent me these patches...
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

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

* Re: [REPOST REPOST PATCH v3 0/3] ASoC: Reduce audio related kernel spew.
  2016-06-29 23:39           ` Patel, Vedang
@ 2016-06-30  2:27             ` Vinod Koul
  2016-07-01  7:50               ` Mark Brown
  0 siblings, 1 reply; 27+ messages in thread
From: Vinod Koul @ 2016-06-30  2:27 UTC (permalink / raw)
  To: Patel, Vedang
  Cc: alsa-devel, Herbert, Marc, broonie, Girdwood, Liam R, Kp, Jeeja,
	Prusty, Subhransu S

On Thu, Jun 30, 2016 at 05:09:09AM +0530, Patel, Vedang wrote:
> Thanks Mark for merging the patches. I apologize for any unwanted noise
> in your inbox.

Please never *top post*

> 
> I did post the patches a few weeks back (http://www.spinics.net/lists/a
> lsa-devel/msg50782.html). The only difference this time was I cc'ed you
> in the patches. Moving forward, what is the preferred method for
> submitting patches for the first time or reposting them so that you are
> less likely to miss them?

Ah not CCing maintainers is not correct. That is why checkpatch gives you
names to cc.

Your patch got ignored for this reason then!

> 
> Thanks,
> Vedang
> 
> On Mon, 2016-06-27 at 15:50 +0100, Mark Brown wrote:
> > On Fri, Jun 24, 2016 at 05:37:08PM -0700, vedang.patel@intel.com
> > wrote:
> > > 
> > > From: Vedang Patel <vedang.patel@intel.com>
> > > 
> > > There have been complaints regarding audio related kernel messages
> > > flooding
> > > dmesg. This series tries to address those complaints.
> > Please don't include noise like REPOST in the subject line,
> > especially
> > not at the start of the subject line where it makes it look less like
> > a
> > patch e-mail and so harder to spot in an inbox.  I also note that
> > this
> > appears to be the first time you've sent me these patches...

-- 
~Vinod

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

* Re: [REPOST REPOST PATCH v3 0/3] ASoC: Reduce audio related kernel spew.
  2016-06-30  2:27             ` Vinod Koul
@ 2016-07-01  7:50               ` Mark Brown
  0 siblings, 0 replies; 27+ messages in thread
From: Mark Brown @ 2016-07-01  7:50 UTC (permalink / raw)
  To: Vinod Koul
  Cc: alsa-devel, Herbert, Marc, Girdwood, Liam R, Kp, Jeeja, Prusty,
	Subhransu S, Patel, Vedang


[-- Attachment #1.1: Type: text/plain, Size: 675 bytes --]

On Thu, Jun 30, 2016 at 07:57:47AM +0530, Vinod Koul wrote:
> On Thu, Jun 30, 2016 at 05:09:09AM +0530, Patel, Vedang wrote:

> > I did post the patches a few weeks back (http://www.spinics.net/lists/a
> > lsa-devel/msg50782.html). The only difference this time was I cc'ed you
> > in the patches. Moving forward, what is the preferred method for
> > submitting patches for the first time or reposting them so that you are
> > less likely to miss them?

> Ah not CCing maintainers is not correct. That is why checkpatch gives you
> names to cc.

> Your patch got ignored for this reason then!

The process for submitting patches is covered in
Documentation/SubmittingPatches

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



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

end of thread, other threads:[~2016-07-01  7:51 UTC | newest]

Thread overview: 27+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-12 22:08 [PATCH 0/3] ASoC: Reduce audio related kernel spew vedang.patel
2016-04-12 22:08 ` [PATCH 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
2016-04-13  1:09   ` Takashi Sakamoto
2016-04-14 15:15     ` Vedang Patel
2016-04-15 19:13       ` Vedang Patel
2016-04-12 22:08 ` [PATCH 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
2016-06-27 16:40   ` Applied "ASoC: Intel: common: increase the loglevel of "FW Poll Status"." to the asoc tree Mark Brown
2016-04-12 22:08 ` [PATCH 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
2016-04-26 23:03   ` [PATCH v2 0/3] ASoC: Reduce audio related kernel spew vedang.patel
2016-04-26 23:03     ` [PATCH v2 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
2016-04-26 23:03     ` [PATCH v2 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
2016-04-26 23:03     ` [PATCH v2 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
2016-06-01  1:14       ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew vedang.patel
2016-06-01  1:14         ` [PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
2016-06-27 16:40           ` Applied "ASoC: hdac_hdmi: Increase loglevel of hex dump printed" to the asoc tree Mark Brown
2016-06-01  1:15         ` [PATCH v3 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
2016-06-01  1:15         ` [PATCH v3 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
2016-06-01  5:56         ` [PATCH v3 0/3] ASoC: Reduce audio related kernel spew Vinod Koul
2016-06-25  0:37       ` [REPOST REPOST PATCH " vedang.patel
2016-06-25  0:37         ` [REPOST REPOST PATCH v3 1/3] ASoC: hdac_hdmi: Increase loglevel of hex dump printed vedang.patel
2016-06-25  0:37         ` [REPOST REPOST PATCH v3 2/3] ASoC: Intel: common: increase the loglevel of "FW Poll Status" vedang.patel
2016-06-25  0:37         ` [REPOST REPOST PATCH v3 3/3] ASoC: Intel: Skylake: Increase loglevel of debug messages vedang.patel
2016-06-27 14:50         ` [REPOST REPOST PATCH v3 0/3] ASoC: Reduce audio related kernel spew Mark Brown
2016-06-29 23:39           ` Patel, Vedang
2016-06-30  2:27             ` Vinod Koul
2016-07-01  7:50               ` Mark Brown
2016-06-27 16:40       ` Applied "ASoC: Intel: Skylake: Increase loglevel of debug messages." to the asoc tree Mark Brown

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.