ASoC: rsnd: suppress rsnd_dai_call() debug message
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Tue, 13 Feb 2018 02:09:14 +0000 (02:09 +0000)
committerMark Brown <broonie@kernel.org>
Tue, 13 Feb 2018 11:58:26 +0000 (11:58 +0000)
rsnd_dai_call() is using dev_dbg(), but its message is sometimes
blocks nessesary other messages. If RSND_DEBUG_NO_DAI_CALL was
defined it will be suppressed by this patch.

Reported-by: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx@renesas.com>
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Tested-by: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx@renesas.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sh/rcar/core.c
sound/soc/sh/rcar/rsnd.h

index 64d5ecb..5aafc37 100644 (file)
  *  [mod]->fn() -> [mod]->fn() -> [mod]->fn()...
  *
  */
+
+/*
+ * you can enable below define if you don't need
+ * DAI status debug message when debugging
+ * see rsnd_dbg_dai_call()
+ *
+ * #define RSND_DEBUG_NO_DAI_CALL 1
+ */
+
 #include <linux/pm_runtime.h>
 #include "rsnd.h"
 
@@ -468,7 +477,7 @@ static int rsnd_status_update(u32 *status,
                                                __rsnd_mod_shift_##fn,  \
                                                __rsnd_mod_add_##fn,    \
                                                __rsnd_mod_call_##fn);  \
-               dev_dbg(dev, "%s[%d]\t0x%08x %s\n",                     \
+               rsnd_dbg_dai_call(dev, "%s[%d]\t0x%08x %s\n",           \
                        rsnd_mod_name(mod), rsnd_mod_id(mod), *status,  \
                        (func_call && (mod)->ops->fn) ? #fn : "");      \
                if (func_call && (mod)->ops->fn)                        \
index 5241ea7..172c8d6 100644 (file)
@@ -798,4 +798,14 @@ void rsnd_mod_make_sure(struct rsnd_mod *mod, enum rsnd_mod_type type);
        if (!IS_BUILTIN(RSND_DEBUG_NO_IRQ_STATUS))      \
                dev_dbg(dev, param)
 
+/*
+ * If you don't need rsnd_dai_call debug message,
+ * define RSND_DEBUG_NO_DAI_CALL as 1 on top of core.c
+ *
+ * #define RSND_DEBUG_NO_DAI_CALL 1
+ */
+#define rsnd_dbg_dai_call(dev, param...)               \
+       if (!IS_BUILTIN(RSND_DEBUG_NO_DAI_CALL))        \
+               dev_dbg(dev, param)
+
 #endif