From: Thomas Zimmermann <tzimmermann@suse.de>
To: deller@gmx.de, javierm@redhat.com
Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org,
Thomas Zimmermann <tzimmermann@suse.de>
Subject: [PATCH 21/32] fbdev/clps711x-fb: Initialize fb_ops with fbdev macros
Date: Wed, 15 Nov 2023 11:19:26 +0100 [thread overview]
Message-ID: <20231115102954.7102-22-tzimmermann@suse.de> (raw)
In-Reply-To: <20231115102954.7102-1-tzimmermann@suse.de>
Initialize the instance of struct fb_ops with fbdev initializer
macros for framebuffers in I/O address space. Set the read/write,
draw and mmap callbacks to the correct implementation and avoid
implicit defaults. Also select the necessary helpers in Kconfig.
The driver previously selected drawing ops for system memory
although it operates on I/O memory. Fixed now.
Fbdev drivers sometimes rely on the callbacks being NULL for a
default I/O-memory-based implementation to be invoked; hence
requiring the I/O helpers to be built in any case. Setting all
callbacks in all drivers explicitly will allow to make the I/O
helpers optional. This benefits systems that do not use these
functions.
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
---
drivers/video/fbdev/Kconfig | 4 +---
drivers/video/fbdev/clps711x-fb.c | 4 +---
2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig
index 812b3a9a49507..76dc1a0081c17 100644
--- a/drivers/video/fbdev/Kconfig
+++ b/drivers/video/fbdev/Kconfig
@@ -146,10 +146,8 @@ config FB_ACORN
config FB_CLPS711X
tristate "CLPS711X LCD support"
depends on FB && (ARCH_CLPS711X || COMPILE_TEST)
+ select FB_IOMEM_HELPERS
select FB_MODE_HELPERS
- select FB_SYS_FILLRECT
- select FB_SYS_COPYAREA
- select FB_SYS_IMAGEBLIT
select LCD_CLASS_DEVICE
select VIDEOMODE_HELPERS
help
diff --git a/drivers/video/fbdev/clps711x-fb.c b/drivers/video/fbdev/clps711x-fb.c
index e956c90efcdcc..dcfd1fbbc7e10 100644
--- a/drivers/video/fbdev/clps711x-fb.c
+++ b/drivers/video/fbdev/clps711x-fb.c
@@ -155,13 +155,11 @@ static int clps711x_fb_blank(int blank, struct fb_info *info)
static const struct fb_ops clps711x_fb_ops = {
.owner = THIS_MODULE,
+ FB_DEFAULT_IOMEM_OPS,
.fb_setcolreg = clps711x_fb_setcolreg,
.fb_check_var = clps711x_fb_check_var,
.fb_set_par = clps711x_fb_set_par,
.fb_blank = clps711x_fb_blank,
- .fb_fillrect = sys_fillrect,
- .fb_copyarea = sys_copyarea,
- .fb_imageblit = sys_imageblit,
};
static int clps711x_lcd_check_fb(struct lcd_device *lcddev, struct fb_info *fi)
--
2.42.0
next prev parent reply other threads:[~2023-11-15 10:30 UTC|newest]
Thread overview: 77+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-15 10:19 [PATCH 00/32] fbdev: Modularize helpers for struct fb_ops Thomas Zimmermann
2023-11-15 10:19 ` [PATCH 01/32] fbdev/acornfb: Fix name of fb_ops initializer macro Thomas Zimmermann
2023-11-16 9:45 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 02/32] fbdev/sm712fb: Use correct initializer macros for struct fb_ops Thomas Zimmermann
2023-11-16 9:46 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 03/32] fbdev/vfb: Set FBINFO_VIRTFB flag Thomas Zimmermann
2023-11-16 10:00 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 04/32] fbdev/vfb: Initialize fb_ops with fbdev macros Thomas Zimmermann
2023-11-16 10:04 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 05/32] fbdev/arcfb: Set FBINFO_VIRTFB flag Thomas Zimmermann
2023-11-16 10:05 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 06/32] fbdev/arcfb: Use generator macros for deferred I/O Thomas Zimmermann
2023-11-16 10:16 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 07/32] auxdisplay/cfag12864bfb: Set FBINFO_VIRTFB flag Thomas Zimmermann
2023-11-16 10:16 ` Javier Martinez Canillas
2023-11-21 23:44 ` Miguel Ojeda
2023-11-15 10:19 ` [PATCH 08/32] auxdisplay/cfag12864bfb: Initialize fb_ops with fbdev macros Thomas Zimmermann
2023-11-16 10:22 ` Javier Martinez Canillas
2023-11-21 23:45 ` Miguel Ojeda
2023-11-15 10:19 ` [PATCH 09/32] auxdisplay/ht16k33: Set FBINFO_VIRTFB flag Thomas Zimmermann
2023-11-16 10:23 ` Javier Martinez Canillas
2023-11-21 23:50 ` Miguel Ojeda
2023-11-22 9:03 ` Thomas Zimmermann
2023-11-22 7:42 ` Robin van der Gracht
2023-11-15 10:19 ` [PATCH 10/32] auxdisplay/ht16k33: Initialize fb_ops with fbdev macros Thomas Zimmermann
2023-11-16 10:23 ` Javier Martinez Canillas
2023-11-21 23:45 ` Miguel Ojeda
2023-11-22 7:47 ` Robin van der Gracht
2023-11-15 10:19 ` [PATCH 11/32] hid/picolcd_fb: Set FBINFO_VIRTFB flag Thomas Zimmermann
2023-11-16 10:27 ` Javier Martinez Canillas
2023-11-17 8:51 ` Bruno Prémont
2023-11-21 8:42 ` Jiri Kosina
2023-11-21 8:56 ` Thomas Zimmermann
2023-11-15 10:19 ` [PATCH 12/32] fbdev/sh_mobile_lcdcfb: " Thomas Zimmermann
2023-11-16 10:29 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 13/32] fbdev/sh_mobile_lcdcfb: Initialize fb_ops with fbdev macros Thomas Zimmermann
2023-11-16 10:45 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 14/32] fbdev/smscufx: Select correct helpers Thomas Zimmermann
2023-11-16 10:46 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 15/32] fbdev/udlfb: " Thomas Zimmermann
2023-11-16 10:46 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 16/32] fbdev/au1200fb: Set FBINFO_VIRTFB flag Thomas Zimmermann
2023-11-16 10:47 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 17/32] fbdev/au1200fb: Initialize fb_ops with fbdev macros Thomas Zimmermann
2023-11-16 10:48 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 18/32] fbdev/ps3fb: Set FBINFO_VIRTFB flag Thomas Zimmermann
2023-11-16 10:48 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 19/32] fbdev/ps3fb: Initialize fb_ops with fbdev macros Thomas Zimmermann
2023-11-16 10:53 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 20/32] media/ivtvfb: Initialize fb_ops to fbdev I/O-memory helpers Thomas Zimmermann
2023-11-16 11:26 ` Javier Martinez Canillas
2023-11-20 8:52 ` Hans Verkuil
2023-11-20 11:29 ` Thomas Zimmermann
2023-11-15 10:19 ` Thomas Zimmermann [this message]
2023-11-16 11:28 ` [PATCH 21/32] fbdev/clps711x-fb: Initialize fb_ops with fbdev macros Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 22/32] fbdev/vt8500lcdfb: " Thomas Zimmermann
2023-11-16 11:30 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 23/32] fbdev/wm8505fb: Initialize fb_ops to fbdev I/O-memory helpers Thomas Zimmermann
2023-11-16 11:34 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 24/32] fbdev/cyber2000fb: Initialize fb_ops with fbdev macros Thomas Zimmermann
2023-11-16 11:35 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 25/32] staging/sm750fb: Declare fb_ops as constant Thomas Zimmermann
2023-11-16 11:44 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 26/32] staging/sm750fb: Initialize fb_ops with fbdev macros Thomas Zimmermann
2023-11-16 11:57 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 27/32] fbdev: Rename FB_SYS_FOPS token to FB_SYSMEM_FOPS Thomas Zimmermann
2023-11-16 12:05 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 28/32] fbdev: Remove trailing whitespaces Thomas Zimmermann
2023-11-16 11:57 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 29/32] fbdev: Push pgprot_decrypted() into mmap implementations Thomas Zimmermann
2023-11-16 13:04 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 30/32] fbdev: Move default fb_mmap code into helper function Thomas Zimmermann
2023-11-16 13:10 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 31/32] fbdev: Warn on incorrect framebuffer access Thomas Zimmermann
2023-11-16 13:12 ` Javier Martinez Canillas
2023-11-15 10:19 ` [PATCH 32/32] fbdev: Remove default file-I/O implementations Thomas Zimmermann
2023-11-16 13:15 ` Javier Martinez Canillas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20231115102954.7102-22-tzimmermann@suse.de \
--to=tzimmermann@suse.de \
--cc=deller@gmx.de \
--cc=dri-devel@lists.freedesktop.org \
--cc=javierm@redhat.com \
--cc=linux-fbdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).