From nobody Fri Sep 25 02:43:46 2026 Received: from cstnet.cn (smtp25.cstnet.cn [159.226.251.25]) (using TLSv1.2 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4A93F4BC00B; Thu, 17 Sep 2026 10:28:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=159.226.251.25 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789640900; cv=none; b=E90azIXAOt+Jgtf2ct1CsUClHLFOAqQAqpNKYP9xlBtviI9xWLpgmRZjQeL9+QR+sR014WZWUofeZVTUXeRDNcNL58PIklFiTErBTsFkexvMdzKN0ufxTWzdi/iGtB3B1pOxfQprWKezJJ3KuBDsgFetq0UnCXN4TcVJgUCGKFs= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789640900; c=relaxed/simple; bh=fLAwyxJU/avsaVgGvmVbChT34DCHO2PGP7Xf6y4z8w0=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=m+cKYJYoorgH/vpNrxolhGPs0yY/cv6tOIJQ3m5X2M5/N/fH0GSx/vE8psjnTkILdKpx6EqDB0yuyxqpx5TIDj/d803Ytv1UBY86/uEGGk9KcXJrZPiCze45BEa4VNnBiRm8f7l2zIwhG7rlXs25762TZBbV4xk12kH1uEGmQHY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=iscas.ac.cn; spf=pass smtp.mailfrom=iscas.ac.cn; arc=none smtp.client-ip=159.226.251.25 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=iscas.ac.cn Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=iscas.ac.cn Received: from dfae2b116770.home.arpa (unknown [36.110.52.2]) by APP-05 (Coremail) with SMTP id zQCowAAXgDq0wKtqR5x+CA--.63922S2; Thu, 17 Sep 2026 18:28:04 +0800 (CST) From: Wentao Liang To: alexandre.belloni@bootlin.com Cc: bbrezillon@kernel.org, claudiu.beznea@tuxon.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org, miquel.raynal@bootlin.com, nicolas.ferre@microchip.com, richard@nod.at, vigneshr@ti.com, Wentao Liang , stable@vger.kernel.org Subject: [PATCH] mtd: rawnand: atmel: Fix HSMC clock leak in legacy controller init Date: Thu, 17 Sep 2026 10:28:04 +0000 Message-Id: <20260917102804.2146887-1-vulab@iscas.ac.cn> X-Mailer: git-send-email 2.34.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-CM-TRANSID: zQCowAAXgDq0wKtqR5x+CA--.63922S2 X-Coremail-Antispam: 1UD129KBjvJXoWxJF47Aw1UXryxKFyfJFyrJFb_yoW5uFyxpF W7XFWxGw18XayfZFZrKa1UWF1FqFs7KFyDCwsxW34fZw4aqryqyF1rZFyFqFW5GFWfXw43 Za17t3W8CFn5A3JanT9S1TB71UUUUU7qnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDU0xBIdaVrnRJUUUPvb7Iv0xC_Kw4lb4IE77IF4wAFc2x0x2IEx4CE42xK8VAvwI8I cIk0rVWrJVCq3wAFIxvE14AKwVWUJVWUGwA2ocxC64kIII0Yj41l84x0c7CEw4AK67xGY2 AK021l84ACjcxK6xIIjxv20xvE14v26ryj6F1UM28EF7xvwVC0I7IYx2IY6xkF7I0E14v2 6F4j6r4UJwA2z4x0Y4vEx4A2jsIE14v26rxl6s0DM28EF7xvwVC2z280aVCY1x0267AKxV W0oVCq3wAS0I0E0xvYzxvE52x082IY62kv0487Mc02F40EFcxC0VAKzVAqx4xG6I80ewAv 7VC0I7IYx2IY67AKxVWUAVWUtwAv7VC2z280aVAFwI0_Gr1j6F4UJwAm72CE4IkC6x0Yz7 v_Jr0_Gr1lF7xvr2IYc2Ij64vIr41lF7I21c0E0s8F02x267IIx4CEVc8vx2IErcIFxwAC I402YVCY1x02628vn2kIc2xKxwCY1x0262kKe7AKxVWUtVW8ZwCF04k20xvY0x0EwIxGrw CF04k20xvE74AGY7Cv6cx26r48ZrWUJr1UMxC20s026xCaFVCjc4AY6r1j6r4UMI8I3I0E 5I8CrVAFwI0_Jr0_Jr4lx2IqxVCjr7xvwVAFwI0_JrI_JrWlx4CE17CEb7AF67AKxVWUtV W8ZwCIc40Y0x0EwIxGrwCI42IY6xIIjxv20xvE14v26r1I6r4UMIIF0xvE2Ix0cI8IcVCY 1x0267AKxVWxJVW8Jr1lIxAIcVCF04k26cxKx2IYs7xG6r1j6r1xMIIF0xvEx4A2jsIE14 v26r4UJVWxJr1lIxAIcVC2z280aVCY1x0267AKxVW0oVCq3bIYCTnIWIevJa73UjIFyTuY vjxUqdHjUUUUU X-CM-SenderInfo: pyxotu46lvutnvoduhdfq/1tbiBggNA2qrq3ZOXQABsO Content-Type: text/plain; charset="utf-8" atmel_hsmc_nand_controller_legacy_init() takes a reference to the HSMC clock with of_clk_get() and enables it, but the error paths that follow only release the device node, so both the reference and the enable are leaked whenever the controller cannot be fully initialized. The probe fails in that case and never reaches atmel_hsmc_nand_controller_remove(), which is where the clock is normally disabled and put. Add an err_disable_clk path that disables and releases the clock before jumping to the existing out path. Fixes: f88fc122cc34 ("mtd: nand: Cleanup/rework the atmel_nand driver") Cc: stable@vger.kernel.org Signed-off-by: Wentao Liang --- drivers/mtd/nand/raw/atmel/nand-controller.c | 25 +++++++++++++------- 1 file changed, 16 insertions(+), 9 deletions(-) diff --git a/drivers/mtd/nand/raw/atmel/nand-controller.c b/drivers/mtd/nan= d/raw/atmel/nand-controller.c index e7fdf532c5fe..87d9227f6ecc 100644 --- a/drivers/mtd/nand/raw/atmel/nand-controller.c +++ b/drivers/mtd/nand/raw/atmel/nand-controller.c @@ -2184,6 +2184,7 @@ atmel_hsmc_nand_controller_legacy_init(struct atmel_h= smc_nand_controller *nc) if (ret) { dev_err(dev, "Failed to enable the HSMC clock (err =3D %d)\n", ret); + clk_put(nc->clk); goto out; } =20 @@ -2193,20 +2194,20 @@ atmel_hsmc_nand_controller_legacy_init(struct atmel= _hsmc_nand_controller *nc) if (ret !=3D -EPROBE_DEFER) dev_err(dev, "Failed to get IRQ number (err =3D %d)\n", ret); - goto out; + goto err_disable_clk; } =20 ret =3D of_address_to_resource(nfc_np, 0, &res); if (ret) { dev_err(dev, "Invalid or missing NFC IO resource (err =3D %d)\n", ret); - goto out; + goto err_disable_clk; } =20 iomem =3D devm_ioremap_resource(dev, &res); if (IS_ERR(iomem)) { ret =3D PTR_ERR(iomem); - goto out; + goto err_disable_clk; } =20 regmap_conf.name =3D "nfc-io"; @@ -2216,20 +2217,20 @@ atmel_hsmc_nand_controller_legacy_init(struct atmel= _hsmc_nand_controller *nc) ret =3D PTR_ERR(nc->io); dev_err(dev, "Could not create NFC IO regmap (err =3D %d)\n", ret); - goto out; + goto err_disable_clk; } =20 ret =3D of_address_to_resource(nfc_np, 1, &res); if (ret) { dev_err(dev, "Invalid or missing HSMC resource (err =3D %d)\n", ret); - goto out; + goto err_disable_clk; } =20 iomem =3D devm_ioremap_resource(dev, &res); if (IS_ERR(iomem)) { ret =3D PTR_ERR(iomem); - goto out; + goto err_disable_clk; } =20 regmap_conf.name =3D "smc"; @@ -2239,20 +2240,20 @@ atmel_hsmc_nand_controller_legacy_init(struct atmel= _hsmc_nand_controller *nc) ret =3D PTR_ERR(nc->base.smc); dev_err(dev, "Could not create NFC IO regmap (err =3D %d)\n", ret); - goto out; + goto err_disable_clk; } =20 ret =3D of_address_to_resource(nfc_np, 2, &res); if (ret) { dev_err(dev, "Invalid or missing SRAM resource (err =3D %d)\n", ret); - goto out; + goto err_disable_clk; } =20 nc->sram.virt =3D devm_ioremap_resource(dev, &res); if (IS_ERR(nc->sram.virt)) { ret =3D PTR_ERR(nc->sram.virt); - goto out; + goto err_disable_clk; } =20 nc->sram.dma =3D res.start; @@ -2261,6 +2262,12 @@ atmel_hsmc_nand_controller_legacy_init(struct atmel_= hsmc_nand_controller *nc) of_node_put(nfc_np); =20 return ret; + +err_disable_clk: + clk_disable_unprepare(nc->clk); + clk_put(nc->clk); + + goto out; } =20 static int --=20 2.34.1