drm/gem-fb-helper: drm_gem_fbdev_fb_create() make funcs optional
authorNoralf Trønnes <noralf@tronnes.org>
Wed, 15 Nov 2017 14:19:40 +0000 (15:19 +0100)
committerNoralf Trønnes <noralf@tronnes.org>
Fri, 8 Dec 2017 13:26:00 +0000 (14:26 +0100)
Make the drm_framebuffer_funcs argument optional for drivers that
don't need to set the dirty callback.

Signed-off-by: Noralf Trønnes <noralf@tronnes.org>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20171115142001.45358-2-noralf@tronnes.org
drivers/gpu/drm/drm_gem_framebuffer_helper.c

index aa8cb9b..4d682a6 100644 (file)
@@ -272,7 +272,8 @@ EXPORT_SYMBOL_GPL(drm_gem_fb_prepare_fb);
  * @sizes: fbdev size description
  * @pitch_align: Optional pitch alignment
  * @obj: GEM object backing the framebuffer
- * @funcs: vtable to be used for the new framebuffer object
+ * @funcs: Optional vtable to be used for the new framebuffer object when the
+ *         dirty callback is needed.
  *
  * This function creates a framebuffer from a &drm_fb_helper_surface_size
  * description for use in the &drm_fb_helper_funcs.fb_probe callback.
@@ -300,6 +301,9 @@ drm_gem_fbdev_fb_create(struct drm_device *dev,
        if (obj->size < mode_cmd.pitches[0] * mode_cmd.height)
                return ERR_PTR(-EINVAL);
 
+       if (!funcs)
+               funcs = &drm_gem_fb_funcs;
+
        return drm_gem_fb_alloc(dev, &mode_cmd, &obj, 1, funcs);
 }
 EXPORT_SYMBOL(drm_gem_fbdev_fb_create);