ASoC: wm_adsp: Add info print to note that the DSP has shutdown

PATCH Superseded Charles Keepax 2 participants 2 messages Jan 21, 2014

Subsystems (3)

Sound ASoC Wolfson microelectronics drivers

lore

Share & tools

lore

Charles Keepax

author

2014-01-21 16:13 UTC #0 lore raw

PATCH 1/1 ASoC: wm_adsp: Add info print to note that the DSP has shutdown

It can be useful for debugging purposes to see at what point the DSP has powered down, so add a message to inform us of this.

Signed-off-by Charles Keepax <[email protected]>

---
 sound/soc/codecs/wm_adsp.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

sound/soc/codecs/wm_adsp.c +2 -0

index 4fbcab6..a15f75b 100644
@@ -1637,6 +1637,8 @@ int wm_adsp2_event(struct snd_soc_dapm_widget *w,
 			list_del(&alg_region->list);
 			kfree(alg_region);
 		}
+
+		adsp_info(dsp, "Shutdown complete\n");
 		break;
 
 	default:

Signature · 1 line

1.7.2.5

Mark Brown

maintainer

2014-01-21 17:38 UTC #1 lore raw

sound/soc/codecs/wm_adsp.c int wm_adsp2_event(struct snd_soc_dapm_widget *w, · line 1642

Quoted 6 lines · Charles Keepax

It can be useful for debugging purposes to see at what point the DSP has powered down, so add a message to inform us of this.
sound/soc/codecs/wm_adsp.c +2 -0
+
+		adsp_info(dsp, "Shutdown complete\n");
 		break;

I'd have expected this to be _dbg() rather than info? In general stuff like enumeration or "we might crash immediately if this goes wrong" would get logged normally.