Hi Thomas, On Tue, Apr 25, 2023 at 4:28 PM Thomas Zimmermann <tzimmermann@xxxxxxx> wrote: > The file-op entry points fb_read() and fb_write() verify that > info->state has been set to FBINFO_STATE_RUNNING. Remove the same > test from the implementations of struct fb_ops.{fb_read,fb_write}. > > Signed-off-by: Thomas Zimmermann <tzimmermann@xxxxxxx> Thanks for your patch! > drivers/video/fbdev/core/fb_sys_fops.c | 6 ------ > drivers/video/fbdev/sm712fb.c | 6 ------ Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> But there are several other fbdev drivers to fix: drivers/media/pci/ivtv/ivtvfb.c drivers/video/fbdev/broadsheetfb.c drivers/video/fbdev/hecubafb.c drivers/video/fbdev/metronomefb.c Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds