]> git.dujemihanovic.xyz Git - linux.git/commitdiff
fbdev: Remove FBINFO_FLAG_DEFAULT from kzalloc()'ed structs
authorThomas Zimmermann <tzimmermann@suse.de>
Sat, 15 Jul 2023 18:51:55 +0000 (20:51 +0200)
committerThomas Zimmermann <tzimmermann@suse.de>
Mon, 24 Jul 2023 14:50:39 +0000 (16:50 +0200)
The flag FBINFO_FLAG_DEFAULT is 0 and has no effect, as struct
fbinfo.flags has been allocated to zero by kzalloc(). 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.

v4:
* clarify commit message (Geert, Dan)
v2:
* fix commit message (Miguel)

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Helge Deller <deller@gmx.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20230715185343.7193-14-tzimmermann@suse.de
drivers/video/fbdev/amba-clcd.c
drivers/video/fbdev/matrox/matroxfb_crtc2.c

index e45338227be6e03e102b2cf06a95d5afffa21d13..24d89e6fb7805791f2d8b52e031ad6cd35595224 100644 (file)
@@ -461,7 +461,6 @@ static int clcdfb_register(struct clcd_fb *fb)
        }
 
        fb->fb.fbops            = &clcdfb_ops;
-       fb->fb.flags            = FBINFO_FLAG_DEFAULT;
        fb->fb.pseudo_palette   = fb->cmap;
 
        strncpy(fb->fb.fix.id, clcd_name, sizeof(fb->fb.fix.id));
index 7655afa3fd500b672ff1dae94b59472e5e75528e..372197c124dec4caadd72e27517b9f493eb6115c 100644 (file)
@@ -603,9 +603,8 @@ static int matroxfb_dh_regit(const struct matrox_fb_info *minfo,
        void* oldcrtc2;
 
        m2info->fbcon.fbops = &matroxfb_dh_ops;
-       m2info->fbcon.flags = FBINFO_FLAG_DEFAULT;
-       m2info->fbcon.flags |= FBINFO_HWACCEL_XPAN |
-                              FBINFO_HWACCEL_YPAN;
+       m2info->fbcon.flags = FBINFO_HWACCEL_XPAN |
+                             FBINFO_HWACCEL_YPAN;
        m2info->fbcon.pseudo_palette = m2info->cmap;
        fb_alloc_cmap(&m2info->fbcon.cmap, 256, 1);