From nobody Sun Apr 5 13:12:13 2026 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7D10F3793D2 for ; Wed, 1 Apr 2026 03:22:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775013773; cv=none; b=AljGoJj+mTYHDhniBuebKPdOrXQwLjFwCC21z+OIaEl0oGp6ZckRsiEG8wfims7GLek8+/ZvbtHblFjgvOQlO7PJ5KjGyZs4W4r42imnmZattCB5171+gUFEvwHY4HhvvaSHSeJaRGyNLFKlOA5d/OJs5mFGH8WrtIATMiTgXBc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775013773; c=relaxed/simple; bh=kbO2y45sZLvpmthZIJJ25uuZUuk4g1h1zkNH/OFxXNI=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=CYvnCWAsuXGg1Vn6jEldOfE0av2p68Yk6/E2OfUgaVbQtCWMbaZ3q7wAzk+zKv0gc/YJPOnPrVwQK7CSLBxZZIuBDdmH775tw7yfJo1w0NZFPgAXGKu20ivBOaDHMT30IblqkVkL3BegCZFZOfFuvbyAXj0duammTn/7sBnnjAw= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=UgpKi5hj; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=P7pYNvLA; arc=none smtp.client-ip=205.220.168.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="UgpKi5hj"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="P7pYNvLA" Received: from pps.filterd (m0279863.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 6311WU2B3103964 for ; Wed, 1 Apr 2026 03:22:52 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= tHpnVx2Z9Jz5XG9zwKBPRDeKmDPN8MLUXTLxA+f8ELo=; b=UgpKi5hj5XnpTIwW 09tN4NN9tBeRUYtN7psQp2SmQ2w3WbWryMo3KSXeHqc9tFEDmAIAkS5wC6f9KodA 23/KoNCTvK1Bj/L9gV0Hgs8GGQraqVz8GhLP1sN1QaNwojcnLBHmRU0FTa24kE3l bkPUxiZN/qbdw7rh15lOH8+dAf/RF4W/PUiswKPNBnp6mFbdsjDZdvJ+BGUqcxwO NlLusYrwhPePa1AGcn7m19+sISz0bqnkNgX1ef9Ntmqc6tKfrmBQ6ZpWUa1EuoYs pzrU0V9y21ziqY/FmptbW0WQ8E/Dq5pBL/LAOwWPoQvoEWAc++LLmXJ0OESQOp3G XeBtgQ== Received: from mail-ot1-f69.google.com (mail-ot1-f69.google.com [209.85.210.69]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4d8js226nv-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Wed, 01 Apr 2026 03:22:51 +0000 (GMT) Received: by mail-ot1-f69.google.com with SMTP id 46e09a7af769-7d7efc5a904so23820896a34.3 for ; Tue, 31 Mar 2026 20:22:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1775013771; x=1775618571; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=tHpnVx2Z9Jz5XG9zwKBPRDeKmDPN8MLUXTLxA+f8ELo=; b=P7pYNvLAtHft1LV3mPGqodqsfB5xnlsY44jqSKVI2l8MmL3qyokYdOuIjvZAlgrIzL EStqdENLVDsKBu9vQSnDXc7PXDBCg+ZMEoOweOyKcdJ/XxrKOIj8O+6S18AFrwUgXkPS ZSh6d+jWqUy3L9kuPeFYRL3AwPnwVe0IalhRReKSJpGkNd2stoh40FtjzYadGKe02y8f yxTuqJJuWn6GEX80ycGNETbWaXpFtTx/liNGrmNSf6ixQeLh9nmt7Y66zDTHVveihWhT oHFC6kRmRbBBcEaEewrIsweMQQGo/X56zLxa28C91lYOnrYTIihPjv1ns74jsK1DPizd K9sQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775013771; x=1775618571; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=tHpnVx2Z9Jz5XG9zwKBPRDeKmDPN8MLUXTLxA+f8ELo=; b=CcgD0knTfHDIlVU/BthaDZVjEOg2gDIPnxmBDsfGK2rVCsBy7QUASBNeoyqHa2Ym9O 3hz2SlPKKXf6IBNEDxobYp5Hgz7mQA69F7xq9VUlUYQ4R3Dwpk9sxh73IPKEud9Lh38P pG17opxJRuxIqOy1lNbZEJ7Wb4MP5atnL010vsocY0VJd4sgkFUSK47MKMquOM/jHDVu 5+hZut0DLjlpiAMP6TblJZ+9IVlIhI2WA9eERMn3DWIxcLukqcuzt44qGuZO0XhmLazK scnDu4FoBMU2gkVlQA+5d2B5Z4dMfETI92Fv/ZB2dzv70SGNmunFDZIKve4bMx+kC3pQ He2w== X-Forwarded-Encrypted: i=1; AJvYcCWwxexnnO1Fg7Vs2PNq9Nnvn3Dy/hBznuyNQ+K4ek+7rNkbtkb9x/qFjTcCuBHuGoEnwsXO+iyqYrMYPno=@vger.kernel.org X-Gm-Message-State: AOJu0Yz/dtCZ4gI1C7Bc6KFaYa/e47rSaLVVjIoHDx/KxPCLyMwgYRR4 tk1H4E9af9QnPrt3HO1H9t7/wwz3Ns8sY9m5cMmPA9whE1F8dgMvvSslPNUmlGiZ3VbP+IX6BbP 51At/zAdgKUopVF03BD+FhEvQX154sb2wtYK0NGJbobyzRfweioR6RT4X7KkThIf4Fy8= X-Gm-Gg: ATEYQzySlcDaC2jfQiaVLZerkiGmWzQp3pJWPdIYV8yl29qUC/SPfvAfOIVkIE8aJtm MQFMcp3vtvPEt4ZkLADElT0fEIAkrwgELoPVddgunYvinKV1Uh4wWBXSQVKFpr0Ee1YL/S8DJHL uY4Lz7fb8CM3TkeFJWzMHEnGRPmps5+eAlrgkBdHXRvccuSzTmceYRzYk6pf7VWc4nEDotH4gjX WQga+QfTYfwJihQiUBknlRiYGYyGayvGlz9ri7H9Qqxrtr637zAuOmR4TCz4iQKH3rBglK9iHNX dhfg3pYtmy0Of8Td4VkVXwxNEhtOlBRK71YfRHGFTs9obGzrVrksOhDCMNJdFHim9t2MzSS0WEy W1Ue2rYHW6+DDI5mdsvSRnLnCgcuwMfFpyf1gIE47uNw= X-Received: by 2002:a05:6830:7007:b0:7d9:c69e:ea27 with SMTP id 46e09a7af769-7db99410b8cmr1603166a34.32.1775013771020; Tue, 31 Mar 2026 20:22:51 -0700 (PDT) X-Received: by 2002:a05:6830:7007:b0:7d9:c69e:ea27 with SMTP id 46e09a7af769-7db99410b8cmr1603147a34.32.1775013770650; Tue, 31 Mar 2026 20:22:50 -0700 (PDT) Received: from [192.168.86.59] ([104.57.184.186]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-7da0a336d73sm9589357a34.5.2026.03.31.20.22.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 31 Mar 2026 20:22:50 -0700 (PDT) From: Bjorn Andersson Date: Tue, 31 Mar 2026 22:22:44 -0500 Subject: [PATCH v2 2/7] slimbus: qcom-ngd-ctrl: Fix probe error path ordering Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260331-slim-ngd-dev-v2-2-9441e9c8420e@oss.qualcomm.com> References: <20260331-slim-ngd-dev-v2-0-9441e9c8420e@oss.qualcomm.com> In-Reply-To: <20260331-slim-ngd-dev-v2-0-9441e9c8420e@oss.qualcomm.com> To: Srinivas Kandagatla , Greg Kroah-Hartman , Vinod Koul , Krzysztof Kozlowski Cc: linux-arm-msm@vger.kernel.org, linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, Bjorn Andersson , stable@vger.kernel.org, Dmitry Baryshkov , Mukesh Ojha X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=1682; i=bjorn.andersson@oss.qualcomm.com; h=from:subject:message-id; bh=kbO2y45sZLvpmthZIJJ25uuZUuk4g1h1zkNH/OFxXNI=; b=owEBgwJ8/ZANAwAKAQsfOT8Nma3FAcsmYgBpzI+HXQ17eY/AysPOZgJT0P1HYQKYt0DlWxLJF OVWJFMx3dWJAkkEAAEKADMWIQQF3gPMXzXqTwlm1SULHzk/DZmtxQUCacyPhxUcYW5kZXJzc29u QGtlcm5lbC5vcmcACgkQCx85Pw2ZrcUs+A//TvwvmBOkN51ZzQcz1KmKpKI7R2PMtPeD8TcF1yK svc7GZTWr9qMoEtwHY+24dxJwBiAW/WQ3UYDnVbgF27vDT9GRAMaokU2dtbHcWcpY7So00PmaeN 3q61YNXYk0CuDz9OjWmmVrWeP4gJRxW/+YAe06xjXYIExfAG7gCZ0y0XySyIQ3MCPL2RFm85pK+ TXHT1lDQ3kUSCfkTkx/z6RFhThRBCEQ5n1DoJL0hVddEXucZcRCxGMx2Do7y3/RUU5DqfkEijWm 9LBacsd+VDQ040VfpIvZJbYGlU7gpjYi7u92WdeQ6dpjyb17FwYcXYSLVTI0eedhjRaq0qCFlyP SMpC6DBAzBfXiIwpYg7sHhaCO6o/ekGrCqVoz8/U8B9ihq3z8otjdEXBhQTssyAK0UJsNildjqq 3qegM8z6HdldAq4PmI2k/MtJRehEY8I1iUjFsFJA8hbp3fUoX0qYBvMp6dGbwAfkr5zhdXUGY/h nsXoJ11lqDwv5yDfFUStj04TlwT8IumK1dGw0L3aJpJeNkTfeQDNd1pR9dhZEX1X0pJ9EMHZjWN Mf8DNok4zikczGd2RvqIpfAhSN/uBq2Vw7004ez9Fgu5Xx0uMCCsGn+HIVZt97lPRB610XBMz9J Allu+ItNT1hBe6JLGmMII+r/JZ/pVNTsfJBQ+HvHEQK4= X-Developer-Key: i=bjorn.andersson@oss.qualcomm.com; a=openpgp; fpr=05DE03CC5F35EA4F0966D5250B1F393F0D99ADC5 X-Proofpoint-ORIG-GUID: 8k-WetnNGnyLAlRcEpNikW7lnkYva_RS X-Authority-Analysis: v=2.4 cv=XfqEDY55 c=1 sm=1 tr=0 ts=69cc8f8b cx=c_pps a=z9lCQkyTxNhZyzAvolXo/A==:117 a=DaeiM5VmU20ml6RIjrOvYw==:17 a=IkcTkHD0fZMA:10 a=A5OVakUREuEA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=yOCtJkima9RkubShWh1s:22 a=VwQbUJbxAAAA:8 a=EUspDBNiAAAA:8 a=gyDl647GgXGSOFt2m_oA:9 a=QEXdDO2ut3YA:10 a=EyFUmsFV_t8cxB2kMr4A:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNDAxMDAyNCBTYWx0ZWRfX4e6HkWONlK8C Yd3nexhk1ceL1XdTYSSP7rSSbllAVRjEmowaqI2WY0rjI5VF1WLsSUn1/IZ5KTE3cy7TXRPYUlP kjn/uX4S6hGYUartnenBXVgsir6mViMcAdjQgx7Q3auCr0YMxP8y1lPkc/26qD157ZyYKfnuKEt gWBLe/ST+tLZHPb24/yVkVonT0jrK4lRiaOJPLb86LDGCK8+GHCKl0Oj3hINZvSQ77hbRkSHZmE 6v1TVkbbELspSJhzm4tcYlsiZZ1D/NAQgblaiHKZ9ffX+yCeHXYZSKbhH53/qp+rPoGf2j941Ac yWMUA+707+duAokkv6NL8R0nuJPqyPbSOJk/u7w4Hb9399gwAbo08LQawvznZxJ15n2mNNkz3CR ljLPLc/7ZndRxrMzO2tWVwloafPJZj0klydmPZUZF2IQfWJXsNmYoSngin7ydyX6GyKtT90Cuiw 8Sr0cymLaGcrZ+Cn0WA== X-Proofpoint-GUID: 8k-WetnNGnyLAlRcEpNikW7lnkYva_RS X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-04-01_01,2026-03-31_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 adultscore=0 priorityscore=1501 impostorscore=0 lowpriorityscore=0 phishscore=0 suspectscore=0 clxscore=1015 bulkscore=0 malwarescore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2603050001 definitions=main-2604010024 qcom_slim_ngd_ctrl_probe() first registers the SSR callback then allocates the PDR context, as such the error path needs to come in opposite order to allow us to unroll each step. Fixes: 16f14551d0df ("slimbus: qcom-ngd: cleanup in probe error path") Cc: stable@vger.kernel.org Reviewed-by: Dmitry Baryshkov Reviewed-by: Mukesh Ojha Signed-off-by: Bjorn Andersson --- drivers/slimbus/qcom-ngd-ctrl.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/slimbus/qcom-ngd-ctrl.c b/drivers/slimbus/qcom-ngd-ctr= l.c index b603b9337905438b6c9f5dbe800e560c864d946d..f26fe54b2ffb4bbfe6da6b71725= 7313536abf60f 100644 --- a/drivers/slimbus/qcom-ngd-ctrl.c +++ b/drivers/slimbus/qcom-ngd-ctrl.c @@ -1660,22 +1660,21 @@ static int qcom_slim_ngd_ctrl_probe(struct platform= _device *pdev) if (IS_ERR(ctrl->pdr)) { ret =3D dev_err_probe(dev, PTR_ERR(ctrl->pdr), "Failed to init PDR handle\n"); - goto err_pdr_alloc; + goto err_unregister_ssr; } =20 pds =3D pdr_add_lookup(ctrl->pdr, "avs/audio", "msm/adsp/audio_pd"); if (IS_ERR(pds) && PTR_ERR(pds) !=3D -EALREADY) { ret =3D dev_err_probe(dev, PTR_ERR(pds), "pdr add lookup failed\n"); - goto err_pdr_lookup; + goto err_pdr_release; } =20 return of_qcom_slim_ngd_register(dev, ctrl); =20 -err_pdr_alloc: - qcom_unregister_ssr_notifier(ctrl->notifier, &ctrl->nb); - -err_pdr_lookup: +err_pdr_release: pdr_handle_release(ctrl->pdr); +err_unregister_ssr: + qcom_unregister_ssr_notifier(ctrl->notifier, &ctrl->nb); =20 return ret; } --=20 2.51.0