From patchwork Wed Jul 5 13:18:42 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Srinivas Kandagatla X-Patchwork-Id: 116131 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:9f45:0:b0:3ea:f831:8777 with SMTP id v5csp1870083vqx; Wed, 5 Jul 2023 06:29:31 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ7XnFy+y1+Bhp6B7wcReFeyjUVaVb2B73D4zHfligz6qG8JiLKzkvH3GamVGZOQz9ThiFab X-Received: by 2002:a05:6808:1824:b0:39e:c660:a5fa with SMTP id bh36-20020a056808182400b0039ec660a5famr20128648oib.10.1688563771634; Wed, 05 Jul 2023 06:29:31 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1688563771; cv=none; d=google.com; s=arc-20160816; b=Vq6+Q6kgZl37sPyDygUV8eJfGTL16+8gxnlftWiMYFMoCitmCrzLGqzEOgdJYgtoks pONyzW4d5zFp8kvCPeH2D8gv2BvQcRN4UlrWtwgffE6xnu9vdyCceDUWfIixb0RslODE 9TczoPcLPi9zXpWOSMnRTJdQxoY2+C9O6zOK1Rqe+A+99R91Ki9kpSYSeXMZVp5ePvFs +ea7B/HWjyB5gdtoYYtTIXXAkH4lri3xo3FoV7R4JrZIWhomCcI5X48LUw5l6CXXQ9Yf 7GRc69HvhqdiRdLhqkOOsMEYSlcFfS7z0eQDeEX8sI/o11dkHTpYcSlE1UXs8ZYdYiSd BSjw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=NqzFoVlqZuDwdBeJJ7onmWROcWP+Ot6gtGdl2eC6RO8=; fh=1ichuXQsdAWvc0025PAJlr5lV8fYZBZOjrshOwswhjI=; b=e6mmMyUtO0O7GEgU6paOaJlanDPEQ5d0atsCj+1VGaOL2UJOR0lenyUoXQGdX2ikMa EU6Srse1RIwmBqlKzZdfgwI/4zKW63gI5knCK7NHMxC+fr7SNoRurqj6ae7XTf5AbH00 86xUCHzc8R3R8jVA6NFbCFkkOrtizVwG18v9/L1SSrQsvre62fZZ9W8J/CIexud1/lBn LsYJjNE820yCcLBQ9eVn7InrgOzPuJrlKrS/xUb3K7pT5Q9CP4LUP+NHLB9cKnuhMRs8 Kv7KAq/JeJgtaBbys9cM9EVE3deYspuxtVxGB/KBx8XMkjZyaJEoOP8f6vD7Ha+JwepD p4BQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b="gUabt4/v"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id h12-20020a17090a710c00b002507cbb009bsi1596849pjk.112.2023.07.05.06.29.15; Wed, 05 Jul 2023 06:29:31 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b="gUabt4/v"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231623AbjGENSu (ORCPT + 99 others); Wed, 5 Jul 2023 09:18:50 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35544 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231344AbjGENSt (ORCPT ); Wed, 5 Jul 2023 09:18:49 -0400 Received: from mail-wm1-x32b.google.com (mail-wm1-x32b.google.com [IPv6:2a00:1450:4864:20::32b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 070771712 for ; Wed, 5 Jul 2023 06:18:48 -0700 (PDT) Received: by mail-wm1-x32b.google.com with SMTP id 5b1f17b1804b1-3fbd33a57dcso48325165e9.0 for ; Wed, 05 Jul 2023 06:18:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1688563126; x=1691155126; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=NqzFoVlqZuDwdBeJJ7onmWROcWP+Ot6gtGdl2eC6RO8=; b=gUabt4/ve2Donhp0sXBcgmdz/tHeUFmWmL0Lom+e1nOg5zE4ae1XOMOe0QBwxBO5wI 1dIMgLF9R77E5ea6KSF5orZpSJOWqmxWemJTRd/bSY7hryxOMsE9QTYddZhrWPBo87D7 Qkdpcf+FxgdEWTGteSrjaRWK78i8ZViWAzFhdmhmgxc8K90Lhl4YjBHHxsPN8/wIOecn gMTsB5Tr2nj9NI0vY/udOBI2380OJuGaWMPfTj+Rx6Y/OSFxx/KxzwRDS/qN/qq3btg7 8i40il+M5P8Vbwxk8eds3tIaj3H86465/ckbZlOdD1b2nFUbmxK4YCpfak8UfIYySvg2 oN+g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1688563126; x=1691155126; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=NqzFoVlqZuDwdBeJJ7onmWROcWP+Ot6gtGdl2eC6RO8=; b=QWlyiEHCM65XFD2Y7efFdeX5PS0Tu7ZT5YWH45eh9QBUZQy4dwN5a7dpd4Yxxf8qiy 0dkB5iSVhybtgLUBzphSjkB9ckI7EEFy7ziBK8nxFrmn1EDhj+6Cw9RXhh74YYOOnCpn +YyackiqrMdgs1Ew92Q89kwN85Fc3lqw4toa8N9fne8DwR57QlsHE1TvfeHdm5GrThJW 5KpN02B/4PxQZqsRUIob9PgtyAoSC2Qa01NFPbA21yUYfEuWArM01wVlWgsP3OjTHkhb QNjW29Q+H/4rJzwdHPL8LTtaMlISV8P/gInxODqF85EbfPxGjsGJ1BFrcu1n9dFGmPqi w2kA== X-Gm-Message-State: ABy/qLY67Q000bBMXFdczicf/zK9bXNLiWvu2xuSETyPHrERKA+d/u8v X90iPPQ/p8wUTBRvJ6+rps5PRg== X-Received: by 2002:adf:f282:0:b0:313:eb29:4436 with SMTP id k2-20020adff282000000b00313eb294436mr13789647wro.67.1688563126524; Wed, 05 Jul 2023 06:18:46 -0700 (PDT) Received: from localhost.localdomain ([5.133.47.210]) by smtp.gmail.com with ESMTPSA id cx16-20020a056000093000b00301a351a8d6sm31230039wrb.84.2023.07.05.06.18.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 05 Jul 2023 06:18:45 -0700 (PDT) From: Srinivas Kandagatla To: broonie@kernel.org Cc: johan+linaro@kernel.org, perex@perex.cz, tiwai@suse.com, lgirdwood@gmail.com, ckeepax@opensource.cirrus.com, kuninori.morimoto.gx@renesas.com, linux-kernel@vger.kernel.org, pierre-louis.bossart@linux.intel.com, alsa-devel@alsa-project.org, Srinivas Kandagatla Subject: [PATCH] ASoC: qcom: q6apm: do not close GPR port before closing graph Date: Wed, 5 Jul 2023 14:18:42 +0100 Message-Id: <20230705131842.41584-1-srinivas.kandagatla@linaro.org> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1770587445515291651?= X-GMAIL-MSGID: =?utf-8?q?1770587445515291651?= Closing GPR port before graph close can result in un handled notifications from DSP, this results in spam of errors from GPR driver as there is no one to handle these notification at that point in time. Fix this by closing GPR port after graph close is finished. Fixes: 5477518b8a0e ("ASoC: qdsp6: audioreach: add q6apm support") Signed-off-by: Srinivas Kandagatla --- sound/soc/qcom/qdsp6/q6apm.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/sound/soc/qcom/qdsp6/q6apm.c b/sound/soc/qcom/qdsp6/q6apm.c index 7bfac9492ab5..5d44d07acd69 100644 --- a/sound/soc/qcom/qdsp6/q6apm.c +++ b/sound/soc/qcom/qdsp6/q6apm.c @@ -511,6 +511,8 @@ static int graph_callback(struct gpr_resp_pkt *data, void *priv, int op) switch (hdr->opcode) { case DATA_CMD_RSP_WR_SH_MEM_EP_DATA_BUFFER_DONE_V2: + if (!graph->ar_graph) + break; client_event = APM_CLIENT_EVENT_DATA_WRITE_DONE; mutex_lock(&graph->lock); token = hdr->token & APM_WRITE_TOKEN_MASK; @@ -544,6 +546,8 @@ static int graph_callback(struct gpr_resp_pkt *data, void *priv, int op) wake_up(&graph->cmd_wait); break; case DATA_CMD_RSP_RD_SH_MEM_EP_DATA_BUFFER_V2: + if (!graph->ar_graph) + break; client_event = APM_CLIENT_EVENT_DATA_READ_DONE; mutex_lock(&graph->lock); rd_done = data->payload; @@ -649,8 +653,9 @@ int q6apm_graph_close(struct q6apm_graph *graph) { struct audioreach_graph *ar_graph = graph->ar_graph; - gpr_free_port(graph->port); + graph->ar_graph = NULL; kref_put(&ar_graph->refcount, q6apm_put_audioreach_graph); + gpr_free_port(graph->port); kfree(graph); return 0;