drivers/firmware: Don't mark as busy the simple-framebuffer IO resource
authorJavier Martinez Canillas <javierm@redhat.com>
Tue, 25 Jan 2022 09:12:19 +0000 (10:12 +0100)
committerThomas Zimmermann <tzimmermann@suse.de>
Thu, 27 Jan 2022 08:20:42 +0000 (09:20 +0100)
The sysfb_create_simplefb() function requests a IO memory resource for the
simple-framebuffer platform device, but it also marks it as busy which can
lead to drivers requesting the same memory resource to fail.

Let's drop the IORESOURCE_BUSY flag and let drivers to request it as busy
instead.

Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Zack Rusin <zackr@vmware.com>
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220125091222.21457-3-tzimmermann@suse.de
drivers/firmware/sysfb_simplefb.c

index b86761904949c582cd5a3cce5d8d48f99f4f3bb5..179e9d0ef3e9eff07d4dc179155ae556949fd9b3 100644 (file)
@@ -99,7 +99,7 @@ __init int sysfb_create_simplefb(const struct screen_info *si,
 
        /* setup IORESOURCE_MEM as framebuffer memory */
        memset(&res, 0, sizeof(res));
-       res.flags = IORESOURCE_MEM | IORESOURCE_BUSY;
+       res.flags = IORESOURCE_MEM;
        res.name = simplefb_resname;
        res.start = base;
        res.end = res.start + length - 1;