From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f46.google.com (mail-pj1-f46.google.com [209.85.216.46]) (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 5414540A940 for ; Mon, 3 Aug 2026 14:17:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785766658; cv=none; b=uvakhtBhTedJwPelQivU3LK/c52kxGcEne4d9k2zpG3nWVhP3Op74LyhaKPTlJ6xtoUSCCo3MLWnnb3BF+aOql/yihVB/cUER5z4fun8btSS8DKdWnBnw3LH0CF6M3COjaedy9pNIA5mTax9JJQerinM9qJGZWvkMQx5Sx8L+pg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785766658; c=relaxed/simple; bh=jXMNEPGSmgzb4Ni0tFwqVaC5hDYyBsmFzVyIZbl8eok=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=HlNL/Ir2+SM3OVEViXGxkWx/i2T+yGZ+9aM4hUkDhBahs2aLMwQdt2wO9DWZYgf9k4A/bgQ6GlIYKD1nXiU+D6TlocrUE+G+BhagDVDmOukCleVg896lj3FBRLVDIizshwoVs34oZuKiDAvleBGjIrLO6hir77nUpP1gRwKOwU8= 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=LrEGY7NH; arc=none smtp.client-ip=209.85.216.46 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="LrEGY7NH" Received: by mail-pj1-f46.google.com with SMTP id 98e67ed59e1d1-383b4a3755fso3309508a91.3 for ; Mon, 03 Aug 2026 07:17:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1785766656; x=1786371456; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=YIV+HU50AEtRbtVbE/kWRl5gYNOX4lr3tpB0iIeyarg=; b=LrEGY7NHH8ULc+ge2eBS13hi56QQRdd/tb+/Z6J++ZCphX5LcEnmEC6zlygG4F7v5I 8Gsj3rkPOaXrdWt710nYv8yzfMcqQ17zQgmjix10HOPNRcYVZDlwniYGTWtZqH48D76O utxhy11jJo1Xlrf3UKIPgXtULASeI1er9So6bfFk+ovguHBHQU3CADigYdSqquMNJJlF VU11fBHTmcfKOJ2F3s+0oozGHGGVzqf2Gtpo2LnbyhUlx19zMK7rGruEIfVatp7kBRFg UcMZAC8dZyU3jgMTrq9XatJTZ84jrfRNfeF54mPFsGpUJWEuj3CJwyFzU9qRLCs722hz aKKw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785766656; x=1786371456; h=content-transfer-encoding:mime-version:references:in-reply-to :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=YIV+HU50AEtRbtVbE/kWRl5gYNOX4lr3tpB0iIeyarg=; b=cqwyKJ+OSzy3x+/leAgI6mmJBfyUF7xSQgGi7Bf4c7jgloflsTSA8Fd0wzPAeyFfPm eyMCW5//VDMhutxc/z6tk4x/Azv6dbjWFn2T1miZM4uodxu7nQhSoRfSTmpHARNn3Qct MZRyOAEN4BbD2Qwr/Rimndb9N8BvcKS4QDVAinapxY4JoJzENedbE1qM/AvEIEyACTmU y5XUBWawzwVTy9XRo3uqEezH+ops+yGvxk/12kgYCjOyF32XxIYlaRXypEfSp4H1CQk0 hkewWzMuvy/NFcWBDjWyy3Bx+U0+M8Jxpd4U1JDVC90v6FVggYpzt3OdcHZiQQrG3N9m KyXA== X-Forwarded-Encrypted: i=1; AHgh+RrgYcdxuMscHHv30+nrWM4nf4SnM1n2Egpe1RkmM2WwUpRSUgthRMMHTledIApElMian3XN8Se9npnI6hU=@vger.kernel.org X-Gm-Message-State: AOJu0YywgJ47zIJkTAe40Wi7CkvpQ3VCPaxs8h52oyCDok/yi0ghDXZo J5htsKaIYeVNhdUgnSYQuV4Q6tmTtHciurvBiCwSWkWtej6VuZuQyFJY X-Gm-Gg: AR+sD1021d+2AnMnJBLhGIMFxjquapmG2OXZLOZ35jRYIoDU95YPPUpuPOYTtOkPfse yXHNbzaaRQnr2jmtyRvaEzOuXohY4oZ4oE8j1VqhGj0X9OOmiSmspfiZzsWFG1O4GRj9M9wdvzh deoWg4TjfF1ol5z26druiN3GB+kEuYlawUNwWR+YhaZ/Zo9TS4NNOXQInfdOZPxELSKyeciyHwC VQ00ZtvH46IOuywpXCkiUic1J/6mIwrBx5oyWxiLqjscc//03e7WLvU0eDulKQVDXdBEYYbuzYZ yKp3gLRIIxADVa8bmxgMl23sLh6TQ8hDyvR26Ye4WSQRfEtBo3ja9r1itwDPCRuZTi2MZbMdtQu UupvW4f5oXEGVPc4RdiF7YtX513t+9AhkA+uUVw+I4Dq66yJygCcWB0egk32Q9S8Vgy6YZXBrpS RbtSGNNwQTHg9dnAlReY+QELCTDoK/36dDag0q1YG2RhUNxSQ7lNELWNGANxCu0aF7Cc3fVtwbk gQOAhQuUtuLtQ5kzm2V X-Received: by 2002:a17:90b:5690:b0:38d:a8be:a597 with SMTP id 98e67ed59e1d1-38fbc56732bmr9286831a91.37.1785766655662; Mon, 03 Aug 2026 07:17:35 -0700 (PDT) Received: from buffalo-ssd (M014013071096.v4.enabler.ne.jp. [14.13.71.96]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-38fb30dff2esm4078462a91.13.2026.08.03.07.17.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 03 Aug 2026 07:17:35 -0700 (PDT) From: Akari Tsuyukusa To: Michael Turquette , Stephen Boyd , Brian Masney , Matthias Brugger , AngeloGioacchino Del Regno , Yassine Oudjana , Richard Cochran Cc: Laura Nao , =?UTF-8?q?N=C3=ADcolas=20F=2E=20R=2E=20A=2E=20Prado?= , Chen-Yu Tsai , Akari Tsuyukusa , Bartosz Golaszewski , Danilo Krummrich , Bjorn Helgaas , =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig=20=28The=20Capable=20Hub=29?= , Nicolas Frattaroli , Takashi Sakamoto , Haotian Zhang , Kees Cook , Miles Chen , Daniel Golle , Sam Shih , linux-clk@vger.kernel.org (open list:COMMON CLK FRAMEWORK), linux-kernel@vger.kernel.org (open list), linux-arm-kernel@lists.infradead.org (moderated list:ARM/Mediatek SoC support), linux-mediatek@lists.infradead.org (moderated list:ARM/Mediatek SoC support), netdev@vger.kernel.org (open list:PTP HARDWARE CLOCK SUPPORT:Keyword:(?:\b|_)ptp(?:\b|_)), stable@vger.kernel.org Subject: [PATCH 02/32] clk: mediatek: mt6795: fix memory leak on module removal Date: Mon, 3 Aug 2026 23:16:29 +0900 Message-ID: <20260803141659.559129-3-akkun11.open@gmail.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260803141659.559129-1-akkun11.open@gmail.com> References: <20260803141659.559129-1-akkun11.open@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit clk-mt6795-apmixedsys.c, clk-mt6795-infracfg.c and clk-mt6795-pericfg.c do not call platform_set_drvdata() during their driver probe callback, but their remove callback calls platform_get_drvdata(). This results in platform_get_drvdata() returning NULL, which leads to calling kfree(NULL) in mtk_free_clk_data(NULL). This leaves clk_data unreleased, causing a memory leak. Fix this by calling platform_set_drvdata() during probe. Fixes: 0d363282bb0c ("clk: mediatek: Add MediaTek Helio X10 MT6795 clock drivers") Cc: stable@vger.kernel.org Signed-off-by: Akari Tsuyukusa Reviewed-by: Brian Masney --- drivers/clk/mediatek/clk-mt6795-apmixedsys.c | 2 ++ drivers/clk/mediatek/clk-mt6795-infracfg.c | 2 ++ drivers/clk/mediatek/clk-mt6795-pericfg.c | 2 ++ 3 files changed, 6 insertions(+) diff --git a/drivers/clk/mediatek/clk-mt6795-apmixedsys.c b/drivers/clk/mediatek/clk-mt6795-apmixedsys.c index 123d5d7fea85..239a3f5e1760 100644 --- a/drivers/clk/mediatek/clk-mt6795-apmixedsys.c +++ b/drivers/clk/mediatek/clk-mt6795-apmixedsys.c @@ -175,6 +175,8 @@ static int clk_mt6795_apmixed_probe(struct platform_device *pdev) dev_dbg(dev, "Performing initial setup for MD1\n"); clk_mt6795_apmixed_setup_md1(base); + platform_set_drvdata(pdev, clk_data); + return 0; unregister_ref2usb: diff --git a/drivers/clk/mediatek/clk-mt6795-infracfg.c b/drivers/clk/mediatek/clk-mt6795-infracfg.c index e4559569f5b0..914bb069aa85 100644 --- a/drivers/clk/mediatek/clk-mt6795-infracfg.c +++ b/drivers/clk/mediatek/clk-mt6795-infracfg.c @@ -116,6 +116,8 @@ static int clk_mt6795_infracfg_probe(struct platform_device *pdev) if (ret) goto unregister_cpumuxes; + platform_set_drvdata(pdev, clk_data); + return 0; unregister_cpumuxes: diff --git a/drivers/clk/mediatek/clk-mt6795-pericfg.c b/drivers/clk/mediatek/clk-mt6795-pericfg.c index d48240eb2a67..27d0ef7d7b2f 100644 --- a/drivers/clk/mediatek/clk-mt6795-pericfg.c +++ b/drivers/clk/mediatek/clk-mt6795-pericfg.c @@ -125,6 +125,8 @@ static int clk_mt6795_pericfg_probe(struct platform_device *pdev) if (ret) goto unregister_composites; + platform_set_drvdata(pdev, clk_data); + return 0; unregister_composites: -- 2.55.0