[PATCH v2025.09.y 05/58] video: simplefb-client: switch to dev_get_resource

Ahmad Fatoum a.fatoum at pengutronix.de
Fri Mar 13 06:24:49 PDT 2026


From: Ivaylo Ivanov <ivo.ivanov.ivanov1 at gmail.com>

If the framebuffer memory resource resides in ram, it has already
been requested and mapped, so only get the resource to avoid
requesting a busy resource. This is also the approach for linux.

While at it, use IOMEM for mem->start and drop an unnecessary newline.

(cherry picked from commit c31d855177ff9390876378885d7f8352411cb40b)

Signed-off-by: Ivaylo Ivanov <ivo.ivanov.ivanov1 at gmail.com>
Reviewed-by: Ahmad Fatoum <a.fatoum at pengutronix.de>
Link: https://lore.barebox.org/20250902140006.1410408-2-ivo.ivanov.ivanov1@gmail.com
Signed-off-by: Sascha Hauer <s.hauer at pengutronix.de>
---
 drivers/video/simplefb-client.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/video/simplefb-client.c b/drivers/video/simplefb-client.c
index 1c0ee3e7d426..d4e36df61298 100644
--- a/drivers/video/simplefb-client.c
+++ b/drivers/video/simplefb-client.c
@@ -97,7 +97,7 @@ static int simplefb_probe(struct device *dev)
 	if (ret)
 		return ret;
 
-	mem = dev_request_mem_resource(dev, 0);
+	mem = dev_get_resource(dev, IORESOURCE_MEM, 0);
 	if (IS_ERR(mem)) {
 		dev_err(dev, "No memory resource\n");
 		return PTR_ERR(mem);
@@ -117,7 +117,7 @@ static int simplefb_probe(struct device *dev)
 	info->blue = params.format->blue;
 	info->transp = params.format->transp;
 
-	info->screen_base = (void *)mem->start;
+	info->screen_base = IOMEM(mem->start);
 	info->screen_size = resource_size(mem);
 
 	/*
-- 
2.47.3




More information about the barebox mailing list