fbdev/udlfb: Use fb_ops helpers for deferred I/O
Generate callback functions for struct fb_ops with the fbdev macro FB_GEN_DEFAULT_DEFERRED_SYSMEM_OPS(). Initialize struct fb_ops to the generated functions with fbdev initializer macros. Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de> Cc: Bernie Thompson <bernie@plugable.com> Acked-by: Javier Martinez Canillas <javierm@redhat.com> Link: https://patchwork.freedesktop.org/patch/msgid/20230828132131.29295-3-tzimmermann@suse.de
This commit is contained in:
parent
2519fceb9f
commit
1e3c721664
|
@ -715,68 +715,6 @@ static void dlfb_offload_damage(struct dlfb_data *dlfb, int x, int y, int width,
|
|||
schedule_work(&dlfb->damage_work);
|
||||
}
|
||||
|
||||
/*
|
||||
* Path triggered by usermode clients who write to filesystem
|
||||
* e.g. cat filename > /dev/fb1
|
||||
* Not used by X Windows or text-mode console. But useful for testing.
|
||||
* Slow because of extra copy and we must assume all pixels dirty.
|
||||
*/
|
||||
static ssize_t dlfb_ops_write(struct fb_info *info, const char __user *buf,
|
||||
size_t count, loff_t *ppos)
|
||||
{
|
||||
ssize_t result;
|
||||
struct dlfb_data *dlfb = info->par;
|
||||
u32 offset = (u32) *ppos;
|
||||
|
||||
result = fb_sys_write(info, buf, count, ppos);
|
||||
|
||||
if (result > 0) {
|
||||
int start = max((int)(offset / info->fix.line_length), 0);
|
||||
int lines = min((u32)((result / info->fix.line_length) + 1),
|
||||
(u32)info->var.yres);
|
||||
|
||||
dlfb_handle_damage(dlfb, 0, start, info->var.xres,
|
||||
lines);
|
||||
}
|
||||
|
||||
return result;
|
||||
}
|
||||
|
||||
/* hardware has native COPY command (see libdlo), but not worth it for fbcon */
|
||||
static void dlfb_ops_copyarea(struct fb_info *info,
|
||||
const struct fb_copyarea *area)
|
||||
{
|
||||
|
||||
struct dlfb_data *dlfb = info->par;
|
||||
|
||||
sys_copyarea(info, area);
|
||||
|
||||
dlfb_offload_damage(dlfb, area->dx, area->dy,
|
||||
area->width, area->height);
|
||||
}
|
||||
|
||||
static void dlfb_ops_imageblit(struct fb_info *info,
|
||||
const struct fb_image *image)
|
||||
{
|
||||
struct dlfb_data *dlfb = info->par;
|
||||
|
||||
sys_imageblit(info, image);
|
||||
|
||||
dlfb_offload_damage(dlfb, image->dx, image->dy,
|
||||
image->width, image->height);
|
||||
}
|
||||
|
||||
static void dlfb_ops_fillrect(struct fb_info *info,
|
||||
const struct fb_fillrect *rect)
|
||||
{
|
||||
struct dlfb_data *dlfb = info->par;
|
||||
|
||||
sys_fillrect(info, rect);
|
||||
|
||||
dlfb_offload_damage(dlfb, rect->dx, rect->dy, rect->width,
|
||||
rect->height);
|
||||
}
|
||||
|
||||
/*
|
||||
* NOTE: fb_defio.c is holding info->fbdefio.mutex
|
||||
* Touching ANY framebuffer memory that triggers a page fault
|
||||
|
@ -1186,14 +1124,31 @@ static int dlfb_ops_blank(int blank_mode, struct fb_info *info)
|
|||
return 0;
|
||||
}
|
||||
|
||||
static void dlfb_ops_damage_range(struct fb_info *info, off_t off, size_t len)
|
||||
{
|
||||
struct dlfb_data *dlfb = info->par;
|
||||
int start = max((int)(off / info->fix.line_length), 0);
|
||||
int lines = min((u32)((len / info->fix.line_length) + 1), (u32)info->var.yres);
|
||||
|
||||
dlfb_handle_damage(dlfb, 0, start, info->var.xres, lines);
|
||||
}
|
||||
|
||||
static void dlfb_ops_damage_area(struct fb_info *info, u32 x, u32 y, u32 width, u32 height)
|
||||
{
|
||||
struct dlfb_data *dlfb = info->par;
|
||||
|
||||
dlfb_offload_damage(dlfb, x, y, width, height);
|
||||
}
|
||||
|
||||
FB_GEN_DEFAULT_DEFERRED_SYSMEM_OPS(dlfb_ops,
|
||||
dlfb_ops_damage_range,
|
||||
dlfb_ops_damage_area)
|
||||
|
||||
static const struct fb_ops dlfb_ops = {
|
||||
.owner = THIS_MODULE,
|
||||
.fb_read = fb_sys_read,
|
||||
.fb_write = dlfb_ops_write,
|
||||
__FB_DEFAULT_DEFERRED_OPS_RDWR(dlfb_ops),
|
||||
.fb_setcolreg = dlfb_ops_setcolreg,
|
||||
.fb_fillrect = dlfb_ops_fillrect,
|
||||
.fb_copyarea = dlfb_ops_copyarea,
|
||||
.fb_imageblit = dlfb_ops_imageblit,
|
||||
__FB_DEFAULT_DEFERRED_OPS_DRAW(dlfb_ops),
|
||||
.fb_mmap = dlfb_ops_mmap,
|
||||
.fb_ioctl = dlfb_ops_ioctl,
|
||||
.fb_open = dlfb_ops_open,
|
||||
|
|
Loading…
Reference in New Issue