From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj2-f7.google.com (mail-pj2-f7.google.com [74.125.227.135]) (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 1A126318BB5 for ; Sat, 19 Sep 2026 17:42:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.227.135 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789839756; cv=none; b=M931mw193wexSj9B8Fn6W8ECTWZj+OOToEU7Sa6UyM8guPeSL6JOaTzI0CQgnmawFmmV0+sFYpcAgCxRgLNRlsT7DsTCO2vry9MOj3DUUmBkBm667lnaz7gzko/8n5QK+HlerCh00lSr/+ri/AEd8oY24OBlm4PzdABarSdbsgE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789839756; c=relaxed/simple; bh=ERVYNPfubS1lL7XJj8hGOUw4uQFg9cuUwbGKdJsaNUQ=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=MaTVLgApAvBfDaPokuKxjZeQRTFJJNoJoVTPuVmuNyWbsmUL/MJvfliJTEj+wxaZ+fg2iGuJLI2oZXZ63Jge8hkG4PP4aMQhZ2o204lWkeyYvcaB79zFNapF1TST/IR4Xy630ZtIcNoBuwZexM1Gc8vRyqNb06D0km7pZZOjh2U= 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=Yr89mYWe; arc=none smtp.client-ip=74.125.227.135 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="Yr89mYWe" Received: by mail-pj2-f7.google.com with SMTP id 98e67ed59e1d1-39e501442e2so1224387a91.0 for ; Sat, 19 Sep 2026 10:42:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1789839754; x=1790444554; 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:content-type; bh=wqtGKu4ehtoV9DVHcDQz0i0Wul1TLpZRrMxi7IKUvp4=; b=Yr89mYWeUB8aXyJmU5tGNiLWRx6H40u++exYmhiCcYrJwHhbyNYK1HyulcZkr/KT4E 9gfnJ9tDMZsxgAe68BIBmvqGCR8FyAC5+SKSY9so48H+1Xrl6qOizQGLdF//bv5Wlqzz PWVjlIRWZgGGtJL6OUAe9cAHFiBjeUwgMyiKKObogYt6spPf73A8VKwuKaueEYMUKCae LwlTBftCVR4CgQewFA1qK7zRnqkZmHbHqnE/rkakhKujIQmUVsOUz44kzyXujxDxt1US hnFHOgCJaTwjbx4bD98Mo+lh7qVw39ncO3uiUBH90GtC2XdW51cX4ClFzEtPT91lgxBO P+pA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1789839754; x=1790444554; 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:content-type; bh=wqtGKu4ehtoV9DVHcDQz0i0Wul1TLpZRrMxi7IKUvp4=; b=xIYshooHiBF0JGWsTvjn4U0yoXsVD9WHcVu9Eet+w1asFIjh3qGqrKPiGxWvFjWs/B Ou/fXlmdBAG6NPOAViW9uVGa53tux3z7H0vVBLPBycgACgVTlDcJer67WUrauB8kn6Mj eIoCdp8uu9noHnS0P0oj0gGSheZGe4GT29pw+fEsBk1CoXqTpbTFiq4T3ylGBde3zXsO E5EjpTbY3SHBJM0siYYQBLV3EfkvHnjDBnzuT1ImvAoWVy4Qr2Bzad5Vi8ng6S+RBVXT GvZ9SHyuHGhr00JC9s3tFE9oRwgk9Y3BP20g5qqy3APbrnqXbti4lK12pRQyXJK46kIQ MWrg== X-Forwarded-Encrypted: i=1; AKwUvBxAVGAegvu1u6t7iKbHlk2sVQKbPxPcuMeHT9zzv3/iFadVpxTwR6+apoCZ/i/6HKk6BFGuqXEjJrEtMSE=@vger.kernel.org X-Gm-Message-State: AFuF++lu2FySSPuJGOYcCgIxLhrQQOXGsTuTOhksQpXNTW1JyhzGyIup o1or97w0s73tl7GCcrOyu/YKt2zOa1OiOYXYDbsvEGEnEHQYt/fW8m7q X-Gm-Gg: AYBFou19gAyq5DYxE6fkdT//C5uOt3b3Ss88CArZ+Gl7GofxBL6oTadTbDkrxm5yhNC 3U3Q3YvKCuFhA8PQPahn8EecNT9F3jZVbIqPjwddvV9bYXbjT8dbaHh+1hIU7zaUuZNYAT+uvDh ZtD822jENvcsN4TUfSLoLL51QO8f9LK1ttZGOUdlcoZ4+KeXKDkMj2zR8siNslWmn1GiAQe5w3B mKhtoEcOxBa6o9TN8QcO+gonEO6TTCSi3w0Lgd60J3lxxwxliuZEgJXyKyJpQ8CfT05VYXmlWXa kxbjpi6SjtWyV48nhZudRWzc+nPiNpePRZzNcKXGEPp00mqBMpLBVU4/22f1F41OjBxKw1hhF9H LcPhMoqY4MkwK1Bty9zWDplgy9pKQFivQLg2ONwYnR/Arx2PAHhVoEIYj4JiUFqSyQBCJdUzMix vIyRWu7fyuNKq6KOGZHkbMzwMSRAC8LiKoS1WwAfnEIBZjtld1qkBOKKmzstdRZE85cG5Pz7U1c PXGeze/ X-Received: by 2002:a17:90b:384d:b0:39e:6c6a:656d with SMTP id 98e67ed59e1d1-39e6c6a6757mr4257438a91.48.1789839754436; Sat, 19 Sep 2026 10:42:34 -0700 (PDT) Received: from server.lan ([168.138.199.194]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-39e6f1835adsm5144018a91.6.2026.09.19.10.42.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 19 Sep 2026 10:42:32 -0700 (PDT) From: Coia Prant To: Heiko Stuebner , Maxime Chevallier , Andrew Lunn , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni Cc: Sebastian Reichel , netdev@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Coia Prant Subject: [PATCH net] net: ethernet: stmmac: dwmac-rk: fix bulk clock leak when the PHY clock fails Date: Sun, 20 Sep 2026 01:42:23 +0800 Message-ID: <20260919174223.2356964-1-coiaprant@gmail.com> X-Mailer: git-send-email 2.47.3 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit gmac_clk_enable() enables the bulk clocks first and then the optional PHY clock. If clk_prepare_enable() on the PHY clock fails, the function returns without rolling back the bulk clocks, and bsp_priv->clk_enabled stays false, so the later gmac_clk_enable(bsp_priv, false) becomes a no-op and the bulk clock references are leaked. Add the missing clk_bulk_disable_unprepare() on that failure path. Fixes: ea449f7fa0bf ("net: ethernet: stmmac: dwmac-rk: rework optional clock handling") Signed-off-by: Coia Prant --- drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c index 8d7042e689261..f3a98bd9d6ead 100644 --- a/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c @@ -1163,7 +1163,10 @@ static int gmac_clk_enable(struct rk_priv_data *bsp_priv, bool enable) ret = clk_prepare_enable(bsp_priv->clk_phy); if (ret) + clk_bulk_disable_unprepare(bsp_priv->num_clks, + bsp_priv->clks); return ret; + } rk_configure_io_clksel(bsp_priv); rk_ungate_rmii_clock(bsp_priv); -- 2.47.3