From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f173.google.com (mail-pf1-f173.google.com [209.85.210.173]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 63D4E2C21EE for ; Tue, 23 Jun 2026 09:07:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.173 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782205667; cv=none; b=QzJxUGwq3rn+JpW134iItWzcyrQ6oc+mN6ICZ5GqorLneaCqYP7dQ3Oiebj1hVUhoM/ev+nuQns8CNyE/5tX2ro5W9yhEhDxtuAFTfrb/zMvEwO+rBusAGwIzOI8mbbVVbkIWLAjiDQ9TCGgz8Pp3yJAtv48kV2tQxXLTqo/oq8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782205667; c=relaxed/simple; bh=ESyqtRpi7ytPwnbKcfiQj5Bd4NGUztPgO4gK37OJIZs=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=M7e0mMNy43d6wioQ59bE5wERI+PrXw505thK5s6pjo99ZUZNM7vphiqszNx8aFYTvt4zgpFXWMDAJh5LJkpzepUdeaKt7SSewQp/aM2zyAPyHpWTKGk+wlWTsdMxX7J4JA/KlFxZ1H9jy339G6dDnEA4YJ26e0unubyDAUK9BVM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=SOWL6jI2; arc=none smtp.client-ip=209.85.210.173 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="SOWL6jI2" Received: by mail-pf1-f173.google.com with SMTP id d2e1a72fcca58-8422a92b6d6so398565b3a.1 for ; Tue, 23 Jun 2026 02:07:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1782205665; x=1782810465; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=CmzqBA91+B9bpOJKSj3goWrwy8u7sUjR1sONP+FUzew=; b=SOWL6jI21aajEgm9Iu7etCW3TU8jmNFY7QGTzjFa7cTLzR1k3AeG7DRECcSldpPDb+ IEncWaVj6pl/woWfQyazyXn/1TJpQLKOktZH1kV9acD0WLSVzBgpOALrdXX47wP973pr BGLu2Miv7IEPd4pTWUa2afd9io9CmbDuGIQgZg0ulzZFAjql39vQ4QcLDfhVjbADWw/O 2SC/S+r55KLy+RwsWIZAcCwIFbIgNt0JM+WhhkfL/u4M3wUhzX060oyRvvsb61gIwI2m N8gC+YqnspjXgaE86RSU70CF7lVSnngf+FmB6VOLB8cM/O9qApiPvxRZn3XSq5Cb7GST 56mQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1782205665; x=1782810465; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=CmzqBA91+B9bpOJKSj3goWrwy8u7sUjR1sONP+FUzew=; b=ceEXcMDGPbD44p+OvSo16M1fSAtF/OKuAAz4G5e2tDo6WrE1OaYFqUnFHZXEw4N/Eo fk0s0/qhlfMY1zkf+4bakPDQeMYqcK2X4piqfebLJSAjzODC/wmO+RpZ+0W3GLT5P2v3 ilwkN2udK8I77Fhpe+VOJxOzrQJArl5ECAZCUmbmzO0wwZRO+X+VcYxbbt/U6LAo+MU6 wVZeFNQk/GT6zzAGL7ZDGiz0DYpQgrbDXVA26Uw4sy5wFndE8xYqeDtf+Z9NWWiLWOXQ b37knJmXZV/XVrlaNVEXaDYi8C+SRbgdHEgYtI7IJuLofQ7tZsLwyzvi3DRthU5lm1Gd JCBA== X-Forwarded-Encrypted: i=1; AFNElJ+mDqn0tAs5nbwGYPJpRRIoljJ2GB/O8WjKmvc0Vs2OBY3iTlWJk1Ye73+Rk4uAE6mHU1Tvk72VZvGKcxI=@vger.kernel.org X-Gm-Message-State: AOJu0Yygsn4ktdF7ljUWx6fg4ItxE6k9RESg42TSIwhTD9/gMM+tlrLh rDJP8+X37h8o8JR00N8am8f5Bo3im1GJcPanVbmobCV2pPMtM4CIarM= X-Gm-Gg: AfdE7ck9QlHNhnAryfsnwil0JTVAkTINFR5tLcwXeLV1msaCfAu4Lv67Uohif6yiE1y JA+HIA6vD3/gBZsUk6fMj6aNooi961RDKxlqVDvQu6/zkEq4FXmI67v4dufekaLeen9jjgtvAmC 1X3csJJhtkGgt/LWjjwsgC8usCLL6EnEs5SGFrf3apl0iuIxhrHG82cBM4sA50qyOgrnxgRfla+ 1kj/oYO1xYw4qgjc8zDwVkW5VQrkL8KcqG5Sim/59vWn2Nkkuo1kUkX7jVA72CuFl6njrLtz6tQ LIQx2tlQuRbCsXGvglSqi03v25tZJua7GjNUpX0EuE9rTuqq3oR0AQDroJHcpgy5pZIG0CI+/f4 LIJMAdWH0WgyCRYvrsc2vKMFibf7513iHOOtIe46hm1EK95+35awFvAvGlbTybsY6xZXtzXrGqq EG4jki7+V677gfZQXoQ7/wNmASczzZLy4MCRNSBz+GDc9o4XhIYEc+6S1qkqj+hR8P X-Received: by 2002:a05:6a00:e0e:b0:842:1be8:f9d3 with SMTP id d2e1a72fcca58-84591cb129bmr2489589b3a.23.1782205665463; Tue, 23 Jun 2026 02:07:45 -0700 (PDT) Received: from localhost.localdomain ([14.5.152.27]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-84564ecd784sm10449409b3a.54.2026.06.23.02.07.42 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Tue, 23 Jun 2026 02:07:44 -0700 (PDT) From: Myeonghun Pak To: Michael Turquette , Stephen Boyd Cc: linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, Myeonghun Pak , Ijae Kim Subject: [PATCH] clk: palmas: unprepare clock on provider registration failure Date: Tue, 23 Jun 2026 18:07:34 +0900 Message-ID: <20260623090738.13773-1-mhun512@gmail.com> X-Mailer: git-send-email 2.47.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit palmas_clks_init_configure() prepares the clock when an external control pin is configured. If the subsequent of_clk_add_hw_provider() call fails, palmas_clks_probe() returns the error without dropping that prepare reference. Unprepare the clock on that error path when external control is active, so the probe failure path balances the successful clk_prepare(). This issue was identified during our ongoing static-analysis research while reviewing kernel code. Fixes: 942d1d674931 ("clk: Add driver for Palmas clk32kg and clk32kgaudio clocks") Co-developed-by: Ijae Kim Signed-off-by: Ijae Kim Signed-off-by: Myeonghun Pak --- drivers/clk/clk-palmas.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/clk/clk-palmas.c b/drivers/clk/clk-palmas.c index 39049f62db..ab9df2eedb 100644 --- a/drivers/clk/clk-palmas.c +++ b/drivers/clk/clk-palmas.c @@ -265,8 +265,11 @@ static int palmas_clks_probe(struct platform_device *pdev) } ret = of_clk_add_hw_provider(node, of_clk_hw_simple_get, &cinfo->hw); - if (ret < 0) + if (ret < 0) { dev_err(&pdev->dev, "Fail to add clock driver, %d\n", ret); + if (cinfo->ext_control_pin) + clk_unprepare(cinfo->hw.clk); + } return ret; } -- 2.47.1