From nobody Thu Apr 9 16:32:35 2026 Received: from mail-wm1-f47.google.com (mail-wm1-f47.google.com [209.85.128.47]) (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 B2F44311975 for ; Tue, 3 Mar 2026 12:23:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.47 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772540621; cv=none; b=XMLvhNbKMR2CZcd05yLq6IZLJceHHdmTOwnDMSZw2EYtW6X7W+ThVtAD40sDfJ0IcOqnso/KDwwZd0GnaZ9ruTmxNS5h0A12En//NYhPMXjLp4SUP9H71FeXt6cmfrshPJtAT4rnC7j0Jk84TVQlqUNEs/C0sr7mWSvoa2j9azQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772540621; c=relaxed/simple; bh=VaG5WWK3PEEYYRL3hvYzQ6mrxETdljwGnFaNLiRNSxs=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=gYgUSBXO+eGq1sgWuSHJ9ipILYLz18Ke/erGJ2IT6zTtqimeAHLWOFZ8OPZ6zRTMMA4yTAY71eUWhEzm1BiPtyTSb9YTKCVck+BzX1Mqib5WTejnw80TkW/g/UDlWmib3xTAUmMcuLwVa19bYH7mRNS6Abttj8zp2m6efrpkolQ= 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=nsL2O9pj; arc=none smtp.client-ip=209.85.128.47 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="nsL2O9pj" Received: by mail-wm1-f47.google.com with SMTP id 5b1f17b1804b1-4806ce0f97bso47771935e9.0 for ; Tue, 03 Mar 2026 04:23:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1772540618; x=1773145418; darn=vger.kernel.org; h=cc:to:message-id:content-transfer-encoding:mime-version:subject :date:from:from:to:cc:subject:date:message-id:reply-to; bh=xJ7JLm9HHhra60wi72CXzKV/8KnCXZClrEsgloI2Vjc=; b=nsL2O9pjKuZB/S9xv75n8eGxztBp2yfyP8qXmbWvcZTRheF9X0Nt+cT5CXX2Eoz/1r Eh/abmaCTj5eupOFoBeubkX0FSLOmmpXUJSJBJO0djpEdAzRDLhXZ1TH1VUl06cJAJeQ saK9te93nGMmLZPChn2CQYYN6G+9RZ8kOH23vsJbMzM5jq5Ca+L9LujK6ocPOa2CAWXa JtMaENTFaMmr4kf8yfH6jqN8RFqxNThD73tybHhHdo/i8V6P7Ph/dCmz3GdtvC09uuDq M4ofk16yQOYu4vQTPo8Nmd47hXu3Nt3i7zFCkZH5/LMOrelaTPeKj+zsrDnjFHBFHlxB rbyQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1772540618; x=1773145418; h=cc:to:message-id:content-transfer-encoding:mime-version:subject :date:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=xJ7JLm9HHhra60wi72CXzKV/8KnCXZClrEsgloI2Vjc=; b=WUYVKcepTRYo7pSiD6pRQHMEUT8N81vMw7tiF7BUKkPZHTXXcnBR/9msEoSWniD5Sq fSSqP/mkjnZ5LcoRaR1ecfWxSJR38sI0PCh2/YPp+ki76Y5awnvKy3VJ9CtvmgAmrp5P o0fZ851SuSMEATMkUr4SKqXsgtWuSi7Javk434pA1cwloBNQ0t+xYOmclrDNPtn75ggb cw3krxkFKaNjH4qBshwZwFKCBLdd1LONkKWPmp9+VbnOsYFNkQY/Mnnd7SZ7dPo4aH2O Ii1NU3fhHikxIuR68e8+09OxUyBCHKjr89SbltRY+Tk3rGqZxY0Fq29MZKKp+2Qdx89G PnpA== X-Forwarded-Encrypted: i=1; AJvYcCXSOp2HKQIOVZa0+Mkcu1+UlqiZQKh7b8FLtgqTEmNb3ep1/PHzN4s2E0cx6PYbUE6oK3V46U9kigG31Ic=@vger.kernel.org X-Gm-Message-State: AOJu0YxpdcdDDJTtQW/uq+Cyv5qSDlCavyffyaSdZDvKZ/V3/R7rfwMJ WhuCnGbet8sHuoJXk+Xs9a8sUzekwLGGzJv2G5VWb90FlQ9WHmEXSJi/ELnexa9l X-Gm-Gg: ATEYQzy/JsENJtM3eRyOcOJEu2X7ibCPf/w90SqkM7a3rQ0CvsMbVpYKIfrnzqDwAqc xHqQDKq9STiS8xinWS5cEAgDkjnBmKzYy9UZd7smrIvkoV/Ov4BZqmy76Zqau18n/cvPYCXLLo4 lMPCjKZ6lJhC2J7RPzHEknA15PEoCu4z4Bao4Z5A3/Vi538WCTdrBoXsyWS90iD0bDK6KqtVZ71 qR+82uiKKPNiZ8rerSoXmInps6O5tq93h3iqaFWPmTiDUqzU0thPSWTORk8DBIA6h95xTIpw8q4 GzdTjGf8GS0Fs5bt7HAuoQwbVsU59RgizQbOe/1mSqG6gFWANYGGK+2BOarj/IAxP0Cy0AP3L5l 4nL4la4ZbyDtnaHO3uqV+3ZPWvsdE8NCP8yRzSOE3XyJRpGhthVQsFtbFb6JflR2s3zK3iu+GPb WWDXz/2Vc+sdnP3H/7kH0I0TOmer2gnAjGjXKa/jQmbRl547xK3HX3fRuuIcRyth+oByUeY4IGR qw8yprWuapFdugA/ODgPkZEJSV/rw== X-Received: by 2002:a05:600c:c16e:b0:480:69b6:dfed with SMTP id 5b1f17b1804b1-483c9bf448bmr279884895e9.24.1772540617896; Tue, 03 Mar 2026 04:23:37 -0800 (PST) Received: from [127.0.1.1] (2a01cb06907588f0d6267e1b423820b8.ipv6.abo.wanadoo.fr. [2a01:cb06:9075:88f0:d626:7e1b:4238:20b8]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-439abdf5430sm23922899f8f.5.2026.03.03.04.23.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 03 Mar 2026 04:23:37 -0800 (PST) From: Amin GATTOUT Date: Tue, 03 Mar 2026 13:22:24 +0100 Subject: [PATCH] fbdev: goldfishfb: use devm_platform_ioremap_resource() Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260303-master-v1-1-85eb5ff584b0@gmail.com> X-B4-Tracking: v=1; b=H4sIAH/SpmkC/6tWKk4tykwtVrJSqFYqSi3LLM7MzwNyDHUUlJIzE vPSU3UzU4B8JSMDIzMDYwNj3dzE4pLUIl1jE8NUcwODRCMzS0sloOKCotS0zAqwQdGxtbUAK/1 wflgAAAA= To: Helge Deller Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Amin GATTOUT X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1772540616; l=1990; i=amin.gattout@gmail.com; h=from:subject:message-id; bh=VaG5WWK3PEEYYRL3hvYzQ6mrxETdljwGnFaNLiRNSxs=; b=lVG5GTj/mX7rh2hMdUCHq5iZE7MhF1bTee8t0BYfN09GJ89vBP26l4TVOoAE1jsdSg1Giabkj QwYhrCt0YB0AhEwqlZmAo/1x2BT/41wBlYN8eo3FyO+0xXO3to+ZVJc X-Developer-Key: i=amin.gattout@gmail.com; a=ed25519; pk=C9HM6Jf5d9dPl9fnP9pM/nhMwm+c280ldqYZ0ESu3a8= Replace the open-coded platform_get_resource() + ioremap() pair with devm_platform_ioremap_resource(), which requests the memory region and maps it in a single call, with automatic cleanup on device removal. Signed-off-by: Amin GATTOUT Reviewed-by: Thomas Zimmermann --- drivers/video/fbdev/goldfishfb.c | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/drivers/video/fbdev/goldfishfb.c b/drivers/video/fbdev/goldfis= hfb.c index ffe33a36b944..c9871281bc1d 100644 --- a/drivers/video/fbdev/goldfishfb.c +++ b/drivers/video/fbdev/goldfishfb.c @@ -174,7 +174,6 @@ static const struct fb_ops goldfish_fb_ops =3D { static int goldfish_fb_probe(struct platform_device *pdev) { int ret; - struct resource *r; struct goldfish_fb *fb; size_t framesize; u32 width, height; @@ -189,14 +188,9 @@ static int goldfish_fb_probe(struct platform_device *p= dev) init_waitqueue_head(&fb->wait); platform_set_drvdata(pdev, fb); =20 - r =3D platform_get_resource(pdev, IORESOURCE_MEM, 0); - if (r =3D=3D NULL) { - ret =3D -ENODEV; - goto err_no_io_base; - } - fb->reg_base =3D ioremap(r->start, PAGE_SIZE); - if (fb->reg_base =3D=3D NULL) { - ret =3D -ENOMEM; + fb->reg_base =3D devm_platform_ioremap_resource(pdev, 0); + if (IS_ERR(fb->reg_base)) { + ret =3D PTR_ERR(fb->reg_base); goto err_no_io_base; } =20 @@ -273,7 +267,6 @@ static int goldfish_fb_probe(struct platform_device *pd= ev) fb->fb.fix.smem_start); err_alloc_screen_base_failed: err_no_irq: - iounmap(fb->reg_base); err_no_io_base: kfree(fb); err_fb_alloc_failed: @@ -291,7 +284,6 @@ static void goldfish_fb_remove(struct platform_device *= pdev) =20 dma_free_coherent(&pdev->dev, framesize, (void *)fb->fb.screen_base, fb->fb.fix.smem_start); - iounmap(fb->reg_base); kfree(fb); } =20 --- base-commit: 11439c4635edd669ae435eec308f4ab8a0804808 change-id: 20260303-master-341e700a2699 Best regards, --=20 Amin GATTOUT