From nobody Mon Dec 1 23:05:42 2025 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 3A3E127464F for ; Sat, 29 Nov 2025 13:18:13 +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=1764422294; cv=none; b=AV9ZWQ7n9tBnO65dwLwzzBsAiFhn1rLEGT1eIUctUWGat72CeH3m1BRw988fj+r8B8Mk3wwgkq5Ao3vb7SyrfLfAs3KjGh9T6djz1yDVRuwLzh071xLdNvC1CaJSYyPuc/vjJ+5CRGqZo88h/2lagoR0m43Au+aUfmcuN6TDyn0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764422294; c=relaxed/simple; bh=rFxrLG6iFK7DhIqNwVyLA6RFcipWa+9ielcQ0jNpcW8=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=UiuoYel2Glfl8MVzPrFF/2PvDsOT/FXQqC7LelGthMtCq3RiGfi3wY1PPrYztc7QlSgpQUza7itBIchqx7ybmxB5p2J/WIWuVTlagN2eN6j2vgPaKG1ZyADd6ajCF9EIzuHsKsaNCIxh0EH/leBwu4WC+WEL/TzeRP3ALM9h0J4= 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=ezhJxcfs; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=HT5bar6r; 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="ezhJxcfs"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="HT5bar6r" Received: from pps.filterd (m0279864.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 5AT4enlJ1267170 for ; Sat, 29 Nov 2025 13:18:12 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= WRdTU43px+9dnjQpkvdch/AXAnGzM7VVwsbwnmdMUAM=; b=ezhJxcfs8xdJTJwq 3j/p1xuLBzs4hlIijvw9/D3/7jOt8ky6h9jnRYZ1LKv0rnMZpEs7IfE6n9KYmy2D Ntc0GLOc/58gn0nX5coJyINXoXzqDyt95Tj76JTBYWmhS0wOHmegGeWxh4VByJkQ BXVIEoGjSccPBnOy+0EmYiIqHLxyi+f0Z9WodoKoChrrp38yzx4js0+edkPrgfd4 Mz6C6nDyWDDDsiM15CpcC8H90nF6LjQyA0e2vKeFYm3OhPbBgzRXqYcp1P2t2n3y QNX8aj6+ZosjiIwNHSJ5EILh8RNA7kZekVgHMamKTtRCRpkR0uP63kuY0yOFi49M vrQoIQ== Received: from mail-qk1-f199.google.com (mail-qk1-f199.google.com [209.85.222.199]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4aqt0j0kxp-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Sat, 29 Nov 2025 13:18:12 +0000 (GMT) Received: by mail-qk1-f199.google.com with SMTP id af79cd13be357-8b26bc4984bso779401485a.1 for ; Sat, 29 Nov 2025 05:18:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1764422291; x=1765027091; 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=WRdTU43px+9dnjQpkvdch/AXAnGzM7VVwsbwnmdMUAM=; b=HT5bar6raqcn31kqmKESH8dZrH9A9zl/kbygRHy0PI/WmTIC//exHghBQqWCjkenAv dEbh1d/H3DffbkR7EW/tlMGMDQYPxZ7+M6ihfYTE90n/PDp3EKOp73w+i2Pdh4/EtYNJ KdJVmxD6V9+eBcXj83PFvU0WvM7MjtpBR0mZJlzDD66UdVYut9PQVQCcOx7wxYb6U34N e7N6INcYuGR5K9uEcRJ5bR895gpiC0qVwdnBwjyj7IqQ4M1fVqhh39BzTzy7ffdgTIA5 NPzw6b+fwJCWr5BODAc53xp5yxbOVPzysE9Y5gUd95SG3SMaIloASyT6sCwB1fdwBXbv MWDw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1764422291; x=1765027091; 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=WRdTU43px+9dnjQpkvdch/AXAnGzM7VVwsbwnmdMUAM=; b=EWmTXZoJBZhFPw8qWf8jSmR+uLSLWF1ZA4Cp8XjDiEkBnx2BHceh0NDKkSmu71UVYp GkVNWE0lpBVRWaIedIUcmUKRTP4teA8APLFtrAA2CP2x4hfwmWRXvf0BUW6BzPT/ag9k YVR/GlF8JMt2CT1CcyFfSGfmQn51ylkYq84W3m6ZM9dtByDzgA95nhDbvpt42W9yA50g rm01s5Ys3eeEZkNVhQUX4BKVU9CeS0w8eSQQRxUL6RyvuNVYhI345TcIAifQgwO4Nvl6 RGcADeKM1/hes0fr1l3e/pEiPr7xrjbvVVjBW22C8lY6PUYL00C3T98DRqj6+1a6e6K4 K45A== X-Forwarded-Encrypted: i=1; AJvYcCXdlvDEwOy4vu/G3tWCFTG76/y6q0r2j47wizmfuJsfwomfFuZa9SNHQOZ2ZfciLuAVdPY/cfHQA+DOdtQ=@vger.kernel.org X-Gm-Message-State: AOJu0YzlSUfv3w+bpMmD17hU8/j3eipQ77L/c8/DjcZNswt6iJ5ZsOfP JPavViec/f3eBIGOd5wA79KHa6XQXd4XBMD04JgsPqRW2CqvAj9DNvf55QnuOqb4Q3nFOs45Nd+ Z35NrWOO9UiGFdhnwqoWlBrkGrOAivFeg2HwbagwHuEMasN0cOrzBtH/FnVNdMeTNWXA= X-Gm-Gg: ASbGnctJmO76GLJJqnjZeeTNRfc1mzf60/e9n/AnoZctDdrrLDAge+weJuQhUUbWeKA XEa9dPm0P8v8Zsp5WxnOUZ/X/DCnSo0By0fUU/AviXKmHzoug5LCbBt8HUcD3rCr47zBio9ZiRU UtKxeWalDaZz74Q+bmSYtpUa83sxP1oCyJEUJguw//i8SvSurGMXXO3E1WMF6h2dv5IA/urBQRL y2FVeorQpXaGsr5TcyFJHBp4JVNJ1MkJydx8s2E26mijDBBIaAX0mVSNSdSKnw1cReLwHVLt/gM 578LgqdFMxwCgnTmJqm1SHl61dA+6mg9fKuE1DTPT1AISEGH9MU2Ii1iRFEQsA+87TvkapniWRV CXG3c8lnoRUHXg34bnOmctmo9w1EKM3KNpw== X-Received: by 2002:a05:620a:179f:b0:8a1:b435:984a with SMTP id af79cd13be357-8b33d4ccf48mr3896154585a.69.1764422291432; Sat, 29 Nov 2025 05:18:11 -0800 (PST) X-Google-Smtp-Source: AGHT+IEN5WjeSaw/et52cC/POlt9tqBhJcGDS2ZYifyqsF2haY8nXfZ5eGR+ClgfmHNWFxVc9rF87A== X-Received: by 2002:a05:620a:179f:b0:8a1:b435:984a with SMTP id af79cd13be357-8b33d4ccf48mr3896151185a.69.1764422291023; Sat, 29 Nov 2025 05:18:11 -0800 (PST) Received: from [127.0.1.1] ([178.197.195.159]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b76f5163903sm723233866b.7.2025.11.29.05.18.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 29 Nov 2025 05:18:10 -0800 (PST) From: Krzysztof Kozlowski Date: Sat, 29 Nov 2025 14:17:56 +0100 Subject: [PATCH 4/6] ASoC: qcom: q6afe: Fix confusing cleanup.h syntax 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: <20251129-asoc-wrong-cleanup-h-can-people-stop-sending-this-without-reading-docs-v1-4-c38b06884e39@oss.qualcomm.com> References: <20251129-asoc-wrong-cleanup-h-can-people-stop-sending-this-without-reading-docs-v1-0-c38b06884e39@oss.qualcomm.com> In-Reply-To: <20251129-asoc-wrong-cleanup-h-can-people-stop-sending-this-without-reading-docs-v1-0-c38b06884e39@oss.qualcomm.com> To: Srinivas Kandagatla , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai Cc: linux-sound@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, Krzysztof Kozlowski X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=2219; i=krzysztof.kozlowski@oss.qualcomm.com; h=from:subject:message-id; bh=rFxrLG6iFK7DhIqNwVyLA6RFcipWa+9ielcQ0jNpcW8=; b=owEBbQKS/ZANAwAKAcE3ZuaGi4PXAcsmYgBpKvKIqVWoM7pPky84XR61xbxactJGJ+7aWo4UZ j8itDkFWfuJAjMEAAEKAB0WIQTd0mIoPREbIztuuKjBN2bmhouD1wUCaSryiAAKCRDBN2bmhouD 18PVEACLxagZMgDxiT/r/NQQytM3sDdCluOz35+qCRhmnWmummLyjxzohBsaFWg9FQJfwbMuJHa NnPHOPFFceF+i2iXAiI4RTiAa7EyLx+rUhbWmnp1S8MfO6ymShCxmEk9RZq9ny5QuxHghST6Ke/ 1EW97h/bPCgZ+BcmbnVIXP/Z+BV6rpI4KeVKFONUsNd4fVW+ZY3sz8k8LsmYoKuk7MoajWWYbgo WqF5z7rXvnNuoZDb2ulGNykJidQhmF/ZAUBShCzu8Lwv0IxVL3UipHfltm1HcYM5XGbFaBDE86L 15Ep5Mr96HIMJxT1MRXaDAq0peyS9Hq0383oNS/XakSJ4xKpOoY4ayJHnFJUGM4oCI2A06xJdm5 /agFjJdAx+zRDtVd5psjpJZf76YyX3JmgE6kUfx2LWKXNKS6q6yIqgYhDgpFflsBQbZIwwitgii qz5ygsItnJXma3DWX9GPcdlOUxb75VUThvGDL1vm852KqeqD9nnlm2h15RmE9bQsBt0U/Fj4ZSL 1PcIGQfcf6ChC2dMsIosRZR7lmcgNMCUF3Wd1sGT0YDQs5/HuyFjWaNWmaQTH9WAHl1xkd+wm69 3Bgt8ITddrcR9Qg+Gmc70lIzqydXy0U9eidVd0DIdkMrEp9TPeDAmEj5sNuCWaQ6uL+P0OijyoW zw3YW0T2ny3mneQ== X-Developer-Key: i=krzysztof.kozlowski@oss.qualcomm.com; a=openpgp; fpr=9BD07E0E0C51F8D59677B7541B93437D3B41629B X-Proofpoint-GUID: wEkTgtXr_b7cu6NOEUPSuHDd9fb4cfoL X-Authority-Analysis: v=2.4 cv=WpYm8Nfv c=1 sm=1 tr=0 ts=692af294 cx=c_pps a=HLyN3IcIa5EE8TELMZ618Q==:117 a=FPK7cjBCgYbqzSTDLBmM0Q==:17 a=IkcTkHD0fZMA:10 a=6UeiqGixMTsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=EUspDBNiAAAA:8 a=5W2fhO5o7LvhZa0qPfoA:9 a=QEXdDO2ut3YA:10 a=bTQJ7kPSJx9SKPbeHEYW:22 X-Proofpoint-ORIG-GUID: wEkTgtXr_b7cu6NOEUPSuHDd9fb4cfoL X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMTI5MDEwNiBTYWx0ZWRfX0h4/JW59/SQD Gst5sESBh9xohfG9+RUqGqVlajXvWvRWE1b7vuMrP+I+WwZ3tJultAQZQDYcY5zaBgK6g4kfBV4 6tgg9OOGgY597hIkp5VrymS2SvSW+6uX7DSi3wPgqn8f3bWD4ASv+LMMt9RCVkFWUqoDVVG0AiK DdKcn4hkTfsG5hvfhubxJac+8bUQwCAvEEciNXcLQwXfr2CgmrNMtMyEkJhKyms4yj4ESliczLv PZWQLMJyyU4LMDSM8PHefO8UhYJXYAtQCwGFBRnRB+op/6pcO4KE4pfb1jcCR8z2V4LE+QNOHl9 p9OppmOTx+kiHGdJwD++MxA21+gI/qPwJOTwXAgKUEWeuhejsRx+RKMtcZk6XdUvWRMIvDPa7XR Eleqw2VlQbA7XrnkQ8th8mjJVbJ86Q== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.100.49 definitions=2025-11-28_08,2025-11-27_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 adultscore=0 phishscore=0 malwarescore=0 bulkscore=0 spamscore=0 impostorscore=0 lowpriorityscore=0 suspectscore=0 clxscore=1015 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2510240001 definitions=main-2511290106 Commit 55094e55ae36 ("ASoc: qcom: q6afe: Use automatic cleanup of kfree()") did not make the code simpler but more complicated. Already simple code of allocation and free, without any error paths, got now declaration with one constructor followed by another allocation, which is in contrary to explicit coding rules guiding cleanup.h: "Given that the "__free(...) =3D NULL" pattern for variables defined at the top of the function poses this potential interdependency problem the recommendation is to always define and assign variables in one statement and not group variable definitions at the top of the function when __free() is used." Code does not have a bug, but is less readable and uses discouraged coding practice, so fix that by moving declaration to the place of assignment. Signed-off-by: Krzysztof Kozlowski --- sound/soc/qcom/qdsp6/q6afe.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/sound/soc/qcom/qdsp6/q6afe.c b/sound/soc/qcom/qdsp6/q6afe.c index 980851a12976..c9fdd53492e0 100644 --- a/sound/soc/qcom/qdsp6/q6afe.c +++ b/sound/soc/qcom/qdsp6/q6afe.c @@ -1277,7 +1277,6 @@ int q6afe_port_stop(struct q6afe_port *port) int port_id =3D port->id; int ret =3D 0; int index, pkt_size; - void *p __free(kfree) =3D NULL; =20 index =3D port->token; if (index < 0 || index >=3D AFE_PORT_MAX) { @@ -1286,7 +1285,7 @@ int q6afe_port_stop(struct q6afe_port *port) } =20 pkt_size =3D APR_HDR_SIZE + sizeof(*stop); - p =3D kzalloc(pkt_size, GFP_KERNEL); + void *p __free(kfree) =3D kzalloc(pkt_size, GFP_KERNEL); if (!p) return -ENOMEM; =20 @@ -1667,7 +1666,6 @@ int q6afe_port_start(struct q6afe_port *port) int ret, param_id =3D port->cfg_type; struct apr_pkt *pkt; int pkt_size; - void *p __free(kfree) =3D NULL; =20 ret =3D q6afe_port_set_param_v2(port, &port->port_cfg, param_id, AFE_MODULE_AUDIO_DEV_INTERFACE, @@ -1690,7 +1688,7 @@ int q6afe_port_start(struct q6afe_port *port) } =20 pkt_size =3D APR_HDR_SIZE + sizeof(*start); - p =3D kzalloc(pkt_size, GFP_KERNEL); + void *p __free(kfree) =3D kzalloc(pkt_size, GFP_KERNEL); if (!p) return -ENOMEM; =20 --=20 2.48.1