diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2012-12-13 17:07:50 -0500 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-01-20 09:57:57 -0500 |
commit | af26ef3b3978349cfbd864163a6ebb4906b733b5 (patch) | |
tree | 38efb55b2aac85582180caa018c26b7c3202e77c /drivers/gpu/drm/udl/udl_fb.c | |
parent | 0ae6d7bc0e70dafc1739d50b2b8d9d7c61968395 (diff) |
drm/<drivers>: Unified handling of unimplemented fb->create_handle
Some drivers don't have real ->create_handle callbacks.
- cirrus/ast/mga200: Returns either 0 or -EINVAL.
- udl: Didn't even bother with a callback, leading to a nice
userspace-triggerable OOPS.
- vmwgfx: This driver bothered with an implementation to return 0 as
the handle (which is the canonical no-obj gem handle).
All have in common that ->create_handle doesn't really make too much
sense for them - that ioctl is used only for seamless fb takeover in
the radeon/nouveau/i915 ddx drivers. So allow drivers to not implement
this and return a consistent -ENODEV.
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/udl/udl_fb.c')
-rw-r--r-- | drivers/gpu/drm/udl/udl_fb.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/gpu/drm/udl/udl_fb.c b/drivers/gpu/drm/udl/udl_fb.c index 829c4a76938c..f8904b4e68de 100644 --- a/drivers/gpu/drm/udl/udl_fb.c +++ b/drivers/gpu/drm/udl/udl_fb.c | |||
@@ -422,7 +422,6 @@ static void udl_user_framebuffer_destroy(struct drm_framebuffer *fb) | |||
422 | static const struct drm_framebuffer_funcs udlfb_funcs = { | 422 | static const struct drm_framebuffer_funcs udlfb_funcs = { |
423 | .destroy = udl_user_framebuffer_destroy, | 423 | .destroy = udl_user_framebuffer_destroy, |
424 | .dirty = udl_user_framebuffer_dirty, | 424 | .dirty = udl_user_framebuffer_dirty, |
425 | .create_handle = NULL, | ||
426 | }; | 425 | }; |
427 | 426 | ||
428 | 427 | ||