From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f49.google.com (mail-pj1-f49.google.com [209.85.216.49]) (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 21016413791 for ; Mon, 3 Aug 2026 14:17:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785766671; cv=none; b=kfKPH/5t+YStK0ExlDmvBEzXbzIT5ReaIiPFbJfGSQoaco93a3DTkmYuu59YCkMXpEW2EMV64672+I9DM7EoclFbr53pAmt1IkT+foXoQHhXfp3cPcex+rzOEMSrYP6BIXkuom7x1U2qhNffVqfW0mkOdjiwj7kT/b7R2n+ciVM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785766671; c=relaxed/simple; bh=q1Uz+Gm71OPt/zJsTxyArW2pEOjZRyhgf2I4biF/dNg=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=GYFgk+VjU+rNc5EMHW5zFUeal/w0Q/bAiffDoLuXuRRORReyPjqdn857M0rcj2N6/liAuee1sJbPpTbxtnVl9cwXgfJASpp6kKflg4mq0dIHfRvplzMKOzKcRXPGpzsTgEB1wo07WH7EXRTT6pfaSl9ghI1uNuJuoPjnu5oPLUc= 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=mGG62n+g; arc=none smtp.client-ip=209.85.216.49 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="mGG62n+g" Received: by mail-pj1-f49.google.com with SMTP id 98e67ed59e1d1-38e041ea211so3564048a91.0 for ; Mon, 03 Aug 2026 07:17:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1785766669; x=1786371469; 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=iWp6MzQC1qDygJL4OCtM1qnllOHL0djYegVbWv/i/KI=; b=mGG62n+gxE57o+8qRAEcvglTLaHHxWqsRO2gbcsY3lopfDIO7ynotl6PvKeHun06CS xC8fGyYSlpPAceQ7AL3qvtDlAGz1dBj5JVgSfaXpyIvFAvakseHsxtYS+2LUHGmmwpLd dl1iqbYxwpPkLt665JTbDu76QziLMQy+HR7I9lBNPTAeiwboxCdKISVzC80kGgP8Qj5h bf81nGvYBRIjyWlACQwsHrbmmAes2Cyo2vFOD9OhiSvfFmDI3l75M0iFy4DJjAB49Sdp o2AY/ke65dcrhEBFzsupDdSYh9m4NOfnpAqeoy3yvCqi39HNu/11DsOTaFzRQ9MNiB2b pn7A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785766669; x=1786371469; 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=iWp6MzQC1qDygJL4OCtM1qnllOHL0djYegVbWv/i/KI=; b=kr1C/VySdnDG22GSjpRjRO7GLVFjnxKPcFLmUIPS74jEFeti7AhWcwwy39KN+zxJ0a MoNcbgUAiGQx4nNI1xafc134j1YKRNneObB8D8cJY0Fc/NFIwEb+GOF6Pez+eM3cTj53 RblHAowP8LPUqbbLPsxpK6TdFvdE9XPgVXqkbexEDgRpP0twk0sd2Wch2RXKZl9ojygb y5m1gtjfDahVws9vO4GvOkG3XBwwj0zfIzL/rlYpYK09qZxU4z6b9LI/KVno+Ih1zRIK U7TpxYEtE1qyPLvg/0pepo4b+FRNwze5A0GXnDC+atNzhvi7B7PSyzqs0SkGZVJ6WtXC jNZg== X-Forwarded-Encrypted: i=1; AHgh+Rop3lj+4HiGAM0LD+D2cacp4eVEKydT3+adAO4ITHlqGCjp/bysGkxDkwB+Z+Kks2wiDOXnk/Cax52aEyo=@vger.kernel.org X-Gm-Message-State: AOJu0YyQ7jalBd5RM4SRhldNyPTCRsi0q1E3gYqLmep172mHTw9PIb/q F1M6bi0RQh6oaKY+IYEyVGS6zcGG/VpbECXZ45zZp2poL81KebVUQIBo X-Gm-Gg: AR+sD13Sr78BmziINVyJkXdfa45ztTeRGUfYZhHp9fLiPiH+2q2Z63GphDCtDgpSTmZ AZYamJR/N5TSR2XIRiMdvItqeN1/xZsXSkRvQ7hB1+hzu7Kj30/+LAU5JoHyYfmfvWzoAtV54VT tb2K+WUMsrVq80ueZbC9+RbltMxKCLcjOt8LpQ+JN0+4s+yeOKr771uRP+fC/z2TNPGE4vrW2SZ 7FdTC6qeQS+LEuR5V6zTjJLFWJxFXnEiaef3jY/YI31SolWHOZPuc3NcwE22MB3XRIpBa7CkavG TyFBX1FqOfaucof4rBqFCjbF6YvxZadnDBGa87dMwPUVbiUOa5uN8QUOqWvnbIAvOcR6V45KmuU ulRTQ1mYTklzcDC9pecv3TgrkPnHx3EizyJJW6vI0ZKSEBV3L+AAaWEj6euxgR9OKccCXpB6T/+ qVjW49o2BqFRLNVktwDRxTlIEv1jDFGmOf+ZnNxKN7olCsppMoRH7ON3YTKy0LHDKcuoO2XNetw 7gEFozugGHt70mzsBuJ X-Received: by 2002:a17:90b:528c:b0:37f:9ce1:7364 with SMTP id 98e67ed59e1d1-38fbc4d5e09mr9059524a91.26.1785766669331; Mon, 03 Aug 2026 07:17:49 -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.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 03 Aug 2026 07:17:49 -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 05/32] clk: mediatek: mt8173: fix memory leak on module removal Date: Mon, 3 Aug 2026 23:16:32 +0900 Message-ID: <20260803141659.559129-6-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-mt8173-apmixedsys.c and clk-mt8173-infracfg.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: 4c02c9af3cb9 ("clk: mediatek: mt8173: Break down clock drivers and allow module build") Cc: stable@vger.kernel.org Signed-off-by: Akari Tsuyukusa --- drivers/clk/mediatek/clk-mt8173-apmixedsys.c | 2 ++ drivers/clk/mediatek/clk-mt8173-infracfg.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/drivers/clk/mediatek/clk-mt8173-apmixedsys.c b/drivers/clk/mediatek/clk-mt8173-apmixedsys.c index d7d416172ab3..65dc4489a09c 100644 --- a/drivers/clk/mediatek/clk-mt8173-apmixedsys.c +++ b/drivers/clk/mediatek/clk-mt8173-apmixedsys.c @@ -179,6 +179,8 @@ static int clk_mt8173_apmixed_probe(struct platform_device *pdev) if (r) goto unregister_ref2usb; + platform_set_drvdata(pdev, clk_data); + return 0; unregister_ref2usb: diff --git a/drivers/clk/mediatek/clk-mt8173-infracfg.c b/drivers/clk/mediatek/clk-mt8173-infracfg.c index fa2d1d557e04..9b8c35897789 100644 --- a/drivers/clk/mediatek/clk-mt8173-infracfg.c +++ b/drivers/clk/mediatek/clk-mt8173-infracfg.c @@ -128,6 +128,8 @@ static int clk_mt8173_infracfg_probe(struct platform_device *pdev) if (r) goto unregister_clk_hw; + platform_set_drvdata(pdev, infra_clk_data); + return 0; unregister_clk_hw: -- 2.55.0