* Jocelyn Falempe (jfalempe@xxxxxxxxxx) wrote: > On 23/10/2024 08:46, Thomas Zimmermann wrote: > > Hi > > > > Am 23.10.24 um 01:29 schrieb linux@xxxxxxxxxxx: > > > From: "Dr. David Alan Gilbert" <linux@xxxxxxxxxxx> > > > > > > drm_client_framebuffer_flush() was explicitly added in 2020 > > > by > > > commit c9c03e3cf072 ("drm/client: Add drm_client_framebuffer_flush()") > > > but has never been used. > > > > > > Remove it. > > > > I had a patchset to use this helper for fbdev emulation. It just needs > > preparation in a number of drivers. > > It is used by drm_log, which is under review. > Please don't remove it. OK, no problem. Reviews on the rest of the series would be great. Dave > https://patchwork.freedesktop.org/series/136789/ > > -- > > Jocelyn > > > > > > Best regards > > Thomas > > > > > > > > Signed-off-by: Dr. David Alan Gilbert <linux@xxxxxxxxxxx> > > > --- > > > drivers/gpu/drm/drm_client.c | 33 --------------------------------- > > > include/drm/drm_client.h | 1 - > > > 2 files changed, 34 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/drm_client.c b/drivers/gpu/drm/drm_client.c > > > index bfedcbf516db..5d10ad3c2ca5 100644 > > > --- a/drivers/gpu/drm/drm_client.c > > > +++ b/drivers/gpu/drm/drm_client.c > > > @@ -552,39 +552,6 @@ void drm_client_framebuffer_delete(struct > > > drm_client_buffer *buffer) > > > } > > > EXPORT_SYMBOL(drm_client_framebuffer_delete); > > > -/** > > > - * drm_client_framebuffer_flush - Manually flush client framebuffer > > > - * @buffer: DRM client buffer (can be NULL) > > > - * @rect: Damage rectangle (if NULL flushes all) > > > - * > > > - * This calls &drm_framebuffer_funcs->dirty (if present) to flush > > > buffer changes > > > - * for drivers that need it. > > > - * > > > - * Returns: > > > - * Zero on success or negative error code on failure. > > > - */ > > > -int drm_client_framebuffer_flush(struct drm_client_buffer *buffer, > > > struct drm_rect *rect) > > > -{ > > > - if (!buffer || !buffer->fb || !buffer->fb->funcs->dirty) > > > - return 0; > > > - > > > - if (rect) { > > > - struct drm_clip_rect clip = { > > > - .x1 = rect->x1, > > > - .y1 = rect->y1, > > > - .x2 = rect->x2, > > > - .y2 = rect->y2, > > > - }; > > > - > > > - return buffer->fb->funcs->dirty(buffer->fb, buffer->client- > > > >file, > > > - 0, 0, &clip, 1); > > > - } > > > - > > > - return buffer->fb->funcs->dirty(buffer->fb, buffer->client->file, > > > - 0, 0, NULL, 0); > > > -} > > > -EXPORT_SYMBOL(drm_client_framebuffer_flush); > > > - > > > #ifdef CONFIG_DEBUG_FS > > > static int drm_client_debugfs_internal_clients(struct seq_file *m, > > > void *data) > > > { > > > diff --git a/include/drm/drm_client.h b/include/drm/drm_client.h > > > index bc0e66f9c425..560aae47e06d 100644 > > > --- a/include/drm/drm_client.h > > > +++ b/include/drm/drm_client.h > > > @@ -165,7 +165,6 @@ struct drm_client_buffer { > > > struct drm_client_buffer * > > > drm_client_framebuffer_create(struct drm_client_dev *client, u32 > > > width, u32 height, u32 format); > > > void drm_client_framebuffer_delete(struct drm_client_buffer *buffer); > > > -int drm_client_framebuffer_flush(struct drm_client_buffer *buffer, > > > struct drm_rect *rect); > > > int drm_client_buffer_vmap_local(struct drm_client_buffer *buffer, > > > struct iosys_map *map_copy); > > > void drm_client_buffer_vunmap_local(struct drm_client_buffer *buffer); > > > -- -----Open up your eyes, open up your mind, open up your code ------- / Dr. David Alan Gilbert | Running GNU/Linux | Happy \ \ dave @ treblig.org | | In Hex / \ _________________________|_____ http://www.treblig.org |_______/