Thomas Zimmermann <tzimmermann@xxxxxxx> writes: > 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@xxxxxxx> > Cc: Bernie Thompson <bernie@xxxxxxxxxxxx> > --- Acked-by: Javier Martinez Canillas <javierm@xxxxxxxxxx> [...] > +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); > +} > + These two are very similar to the helpers you added for the smscufx driver in patch #1. I guess there's room for further consolidation as follow-up ? -- Best regards, Javier Martinez Canillas Core Platforms Red Hat