Make the error reporting more useful by adding opcode to it.
Without this its almost impossible to say which command actually
timed out.

Signed-off-by: Srinivas Kandagatla <srinivas.kandaga...@linaro.org>
---
 sound/soc/qcom/qdsp6/q6asm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sound/soc/qcom/qdsp6/q6asm.c b/sound/soc/qcom/qdsp6/q6asm.c
index ae4b2cabdf2d..e0983970cba9 100644
--- a/sound/soc/qcom/qdsp6/q6asm.c
+++ b/sound/soc/qcom/qdsp6/q6asm.c
@@ -311,7 +311,7 @@ static int q6asm_apr_send_session_pkt(struct q6asm *a, 
struct audio_client *ac,
                                        5 * HZ);
 
        if (!rc) {
-               dev_err(a->dev, "CMD timeout\n");
+               dev_err(a->dev, "CMD %x timeout\n", hdr->opcode);
                rc = -ETIMEDOUT;
        } else if (ac->result.status > 0) {
                dev_err(a->dev, "DSP returned error[%x]\n",
@@ -891,7 +891,7 @@ static int q6asm_ac_send_cmd_sync(struct audio_client *ac, 
struct apr_pkt *pkt)
        rc = wait_event_timeout(ac->cmd_wait,
                                (ac->result.opcode == hdr->opcode), 5 * HZ);
        if (!rc) {
-               dev_err(ac->dev, "CMD timeout\n");
+               dev_err(ac->dev, "CMD %x timeout\n", hdr->opcode);
                rc =  -ETIMEDOUT;
                goto err;
        }
-- 
2.21.0

Reply via email to