From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.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 23D79370D69 for ; Mon, 22 Jun 2026 09:15:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782119734; cv=none; b=bdDdZhKq6s7ixObPZwukS0+FXRCfU2uAfvetX9ckmR87nCsw6LonRCi3GX2k8G54Pz2wcWKiDnajUUukCumpbbsqv34TlnfMyloZedvYulMvBSFfcEu1LLmW0EIMjGrU+BSw4Gpw4Mh4uv3iH0aY8xFh46MT0YszkfjQ6N3qxQ8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782119734; c=relaxed/simple; bh=Tlo+7Wj26o/tczDevrBsrDRZR65r9QEfQFDFfnmJgx4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=Z2kudz6kquCcAa65NMVTEDXWcPunX6cOwfLbGgeOHpR5HWW93QpjxuiBLVXpPgrvjxja9rn9XECN3Bj7i+WpVpwNlEQhCfheAB61hK9GWBFjJ8Ho3wxB8PS1GGrpD881rGYN1kCY8ctwmTlLM98yaVNdXKJ6C3OqkmZQsdq9zgA= 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=P9fMRmyz; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=WZjJTVvD; arc=none smtp.client-ip=205.220.180.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="P9fMRmyz"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="WZjJTVvD" Received: from pps.filterd (m0279872.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 65M6GfsK095272 for ; Mon, 22 Jun 2026 09:15:27 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= vzWtviEx2OSdZ4B5CRuGR9Mw6mjrODSpp1ouLLCU9Ok=; b=P9fMRmyzJsnMwAkb WBAiN5P3g7xoDAa0PmPTkxN7puluy91zVjJTqyklSHgMTLhm3pcdaDrSq4pdE7ew aX2h3DX+1/BYMaVKSOwChObcmLUXVtxHl+SPKw3CHYL/elgb+i0sBW6CtHa3kCqc g87zGUUyRtqzhaimLPcHoEW40n53KapLJSUsLTf2tI9LN+rdBmT+651SVGUDPTt6 yZlRC9yGDVWN2V9hLf4cV31sgdFwkSak1n9kKO4Bf9/Lr27y9MBoe2NFTuYqg/Tn 7QtZ7FH51iq7fXU2EGBQ5AhXgUE3NS4VsxsDgvGG0KXqobY6AK++LWTZu3TE+pR6 jqoFBA== Received: from mail-qt1-f197.google.com (mail-qt1-f197.google.com [209.85.160.197]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4exyn1rstv-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Mon, 22 Jun 2026 09:15:27 +0000 (GMT) Received: by mail-qt1-f197.google.com with SMTP id d75a77b69052e-517647fbff1so13382441cf.2 for ; Mon, 22 Jun 2026 02:15:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1782119727; x=1782724527; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=vzWtviEx2OSdZ4B5CRuGR9Mw6mjrODSpp1ouLLCU9Ok=; b=WZjJTVvDzJlgOZSWId4u6TdLQpI/d0jhvtKq/+nnq5JtF5yCi15PZPaHQzdVFTlEvV fQ2qytXL6Te1Z4BWb/mmQ71z4QEFxbApkN6354Abn8KcnmCfrxIo8vCztGAtQrXcL40c TTLkDL+EB8FoC5UuCcQ0UB7tISzS1g6eqHVRYlLbh80o8j0FplmEltFmR5N7l/DNsmkC wdmYFXRfc0ebLrbX+JHzuANYkMxCkIZvz/jxclGZthjQilSpvA72yoFZPDSNn+nymXIx djoCbjd7fqdXe1olPlbBJXx/t2WDo4zgWQTBxco+rMt+ZOGNlbSdpEeY2Um7MWazVUvK ecFA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1782119727; x=1782724527; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=vzWtviEx2OSdZ4B5CRuGR9Mw6mjrODSpp1ouLLCU9Ok=; b=i8fNWVi16gx6jj/KT8iV7luSOejXAC0Ezay95rBJIFR+5IXOKTOi366HVaul2l43WR ReIYFztDZlPHD608AGVT3Uox7ProQM2cd3M7s4AUnO9SH+LwVOAngJYH2XzLWTCj4mq5 aQU7igD3sigXfXmlfDL4uKDebhNskNlg2yT/OtpyfuBJkE9SgUnfF/6ygLx9QNA4h1cu 3C0KbxPPM6gvnnEzUvpVEwjdKrBawIIwauBHQD1TY52ai53tAUEFm/mm6NROL1yUlLJl 1LJ028jdAdQIE7ZzcTp5QfF5sMku8keygCdjl+44SPi4D8rGXc8h1RtpupQkLgpTSQJN bUuA== X-Forwarded-Encrypted: i=1; AFNElJ8+5Zz48EsMeW2xulnKqHaTw5JXDeORR/rQxqHrBD1ClTUZddTaQSRu9ADDwR7z7eUp5Iw0Wwn0NHg09+0=@vger.kernel.org X-Gm-Message-State: AOJu0YyLw9CY9C6PDhNF2XsiQrXSp7+iVNPTOrzWg6+9mGulvsqafJM4 jtNKwULVJ5opCinwjqusghFLBrndUZyA4WGYRr9UtuvZTSAAf+2uOi3O/LUv8y26QV3MJZnAYl8 NeZC+tB1bRIF8l4DIEOrMQt5/ualiP3T80M4UO/eTGBUTxjBiLX8mCvxkVeFzRe5hamdggZkJXR o= X-Gm-Gg: AfdE7clOakGmkREmuU4jCGm5gR4oACIsfjXm7tCXwg78MWRI0kzMxmtJvU3x2wDc8qX sqSLG9epD8xmmI8qEifzFME2KG1OiJYOx/op8zFcxXLdq6qOI9yDI3qnEMTJbCv3slFmeXqRxCo WWMPSBDpEEK2ZPFg7dnh26KNGArOVdZvnasIxNzWbBM3pgDeN8S8lxRgbgdf6yRdIWsivRI7+oX tkZkr8F1G/Eb6SLUExZU21B4EaVLNkGJGWyCnWZY2sssXR6JQ4trMRgx/3HvcMALd/1WCE0Hh0u kPbG40bRl7whpipNNatG08lKA0Qpz5xLHdqGtFl1f/O3fAXOH6uq5mmyg+sPjksutXZkmjMwh2p /pdQIboUu9pVAnnWjpE6BOx+BTXe1zOPURNU= X-Received: by 2002:a05:620a:6082:b0:913:b4b9:5ec8 with SMTP id af79cd13be357-9208d0f9107mr1295813185a.6.1782119726873; Mon, 22 Jun 2026 02:15:26 -0700 (PDT) X-Received: by 2002:a05:620a:6082:b0:913:b4b9:5ec8 with SMTP id af79cd13be357-9208d0f9107mr1295811185a.6.1782119726516; Mon, 22 Jun 2026 02:15:26 -0700 (PDT) Received: from [192.168.120.170] ([178.235.128.140]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c0c5e9994d5sm323364966b.15.2026.06.22.02.15.24 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 22 Jun 2026 02:15:25 -0700 (PDT) Message-ID: Date: Mon, 22 Jun 2026 11:15:23 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] bus: qcom-ebi2: Fix clock leak on probe failure To: Ruoyu Wang , Bjorn Andersson , Konrad Dybcio Cc: Linus Walleij , linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260620080406.1970447-1-ruoyuw560@gmail.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <20260620080406.1970447-1-ruoyuw560@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-GUID: xL79ztqaoE1IAky_9MESAvpouAnHxGdw X-Authority-Analysis: v=2.4 cv=EOU2FVZC c=1 sm=1 tr=0 ts=6a38fd2f cx=c_pps a=EVbN6Ke/fEF3bsl7X48z0g==:117 a=PRfkaYvzSr8QmIIGAkY2Sg==:17 a=IkcTkHD0fZMA:10 a=FelO9ux0wxsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=yx91gb_oNiZeI1HMLzn7:22 a=IA0wh2spAAAA:8 a=VwQbUJbxAAAA:8 a=EUspDBNiAAAA:8 a=pGLkceISAAAA:8 a=K0X2F306fXUgid1TeWIA:9 a=QEXdDO2ut3YA:10 a=a_PwQJl-kcHnX1M80qC6:22 X-Proofpoint-Spam-Info: AW1haW4tMjYwNjIyMDA5MCBTYWx0ZWRfX/jvvC2/O+q0Q r74LH8m5+dTsQE1agitZahWEuh5g394BVxbFywEQZZEbS+ojnTVvDUO1lpmEkh+HHG62RsPVfzq Qiz22z0pJA1M5tM1tSFwcNWv84jGNtw= X-Proofpoint-ORIG-GUID: xL79ztqaoE1IAky_9MESAvpouAnHxGdw X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNjIyMDA5MCBTYWx0ZWRfXzc7iSk8jSqsR 2UphH73b2yxAwrQUsU/aMOD7+S2M3JJZE+LEMkCC9VGiGYA6ODDZeLhzze0aVmBKfqO6d3qWkT5 j9mRveYToMqz+V36rnAuJlrxXWlCFAsOnN9+oGsuRMCnUbopKdGj9URTuLrfL1VkHFUaUP+2BgX 895qwJJh5ERATtz6lpexPW6NXlXtOIB2bfp9j1C/Oib8WVsiDeu+McdBZInwFFfCzKXp93vUWz6 xjJReeeQPDWAQqHkmJV1pDSJWRlRmC2HTrOUxrGu5LKSzpy14fqtkqgN5sJS1O7jJVcHwSolbqn x+S2sGH6hcC+1QAc3oKpqQEUHtjbrAH9AnaHIblDEHBv4b2WP4KhuK1h46QZsH4cqEF8qyZvNav 4bGk55OHg435qCeW0tahEXaeBvHUmNFbyCKrcrsEJgK0xyZcpzXkSDbRJ2PqC0TZkA4ChzdMd6k fpwP+XVuHggA6YvDQFQ== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.125,FMLib:17.12.100.49 definitions=2026-06-22_01,2026-06-18_03,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 bulkscore=0 priorityscore=1501 malwarescore=0 adultscore=0 lowpriorityscore=0 phishscore=0 clxscore=1015 spamscore=0 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2606220090 On 6/20/26 10:04 AM, Ruoyu Wang wrote: > qcom_ebi2_probe() enables the EBI2X and EBI2 clocks before it walks > child nodes and populates child devices. If reading a child node's reg > property fails, or if of_platform_default_populate() fails, probe returns > without disabling either clock. > > Route those failure paths through the existing clock cleanup labels so a > failed probe does not leave the clocks prepared and enabled. > > Fixes: 335a12754808 ("bus: qcom: add EBI2 driver") > Signed-off-by: Ruoyu Wang > --- > This is a minimal error-path fix for a previously reported leak. A broader > devm_clk_get_enabled() conversion was posted in 2023 and reviewed, and a > similar cleanup was posted in 2024, but neither appears in current upstream: > https://lkml.iu.edu/hypermail/linux/kernel/2305.1/00605.html > https://lkml.iu.edu/2408.2/09536.html Someone raised an identical patch last week https://lore.kernel.org/linux-arm-msm/d1882207-4756-4e65-ba30-b47ccc821fd4@oss.qualcomm.com/ Konrad