drm: Remove flag FBINFO_DEFAULT from fbdev emulation
authorThomas Zimmermann <tzimmermann@suse.de>
Sat, 15 Jul 2023 18:51:43 +0000 (20:51 +0200)
committerThomas Zimmermann <tzimmermann@suse.de>
Mon, 24 Jul 2023 14:50:38 +0000 (16:50 +0200)
The flag FBINFO_DEFAULT is 0 and has no effect, as struct fbinfo.flags
has been allocated to zero by framebuffer_alloc(). So do not set it.

Flags should signal differences from the default values. After cleaning
up all occurrences of FBINFO_DEFAULT, the token will be removed.

v2:
* fix commit message (Miguel)

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: "Christian König" <christian.koenig@amd.com>
Cc: "Pan, Xinhui" <Xinhui.Pan@amd.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230715185343.7193-2-tzimmermann@suse.de
drivers/gpu/drm/drm_fbdev_dma.c
drivers/gpu/drm/drm_fbdev_generic.c
drivers/gpu/drm/gma500/fbdev.c
drivers/gpu/drm/radeon/radeon_fbdev.c

index 73b8e99..af44162 100644 (file)
@@ -123,7 +123,6 @@ static int drm_fbdev_dma_helper_fb_probe(struct drm_fb_helper *fb_helper,
        drm_fb_helper_fill_info(info, fb_helper, sizes);
 
        info->fbops = &drm_fbdev_dma_fb_ops;
-       info->flags = FBINFO_DEFAULT;
 
        /* screen */
        info->flags |= FBINFO_VIRTFB; /* system memory */
index b9343fb..a0ea042 100644 (file)
@@ -109,7 +109,6 @@ static int drm_fbdev_generic_helper_fb_probe(struct drm_fb_helper *fb_helper,
        drm_fb_helper_fill_info(info, fb_helper, sizes);
 
        info->fbops = &drm_fbdev_generic_fb_ops;
-       info->flags = FBINFO_DEFAULT;
 
        /* screen */
        info->flags |= FBINFO_VIRTFB | FBINFO_READS_FAST;
index 0544265..be8f5fb 100644 (file)
@@ -215,7 +215,7 @@ static int psb_fbdev_fb_probe(struct drm_fb_helper *fb_helper,
        }
 
        info->fbops = &psb_fbdev_fb_ops;
-       info->flags = FBINFO_DEFAULT;
+
        /* Accessed stolen memory directly */
        info->screen_base = dev_priv->vram_addr + backing->offset;
        info->screen_size = size;
index f941e2e..68c06ac 100644 (file)
@@ -253,7 +253,7 @@ static int radeon_fbdev_fb_helper_fb_probe(struct drm_fb_helper *fb_helper,
        }
 
        info->fbops = &radeon_fbdev_fb_ops;
-       info->flags = FBINFO_DEFAULT;
+
        /* radeon resume is fragile and needs a vt switch to help it along */
        info->skip_vt_switch = false;