drm: s/enum_blob_list/enum_list/ in drm_property
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 19 Nov 2014 17:38:10 +0000 (18:38 +0100)
committerDave Airlie <airlied@redhat.com>
Thu, 20 Nov 2014 01:35:21 +0000 (11:35 +1000)
commit3758b34193638f664177565f1692faa1bec7d9ed
treea6fcfa895a2ae65c5be1650d6a1f6ab300a590e3
parent1a49863349929f5a85fd7c7ffb06213d4fdc8c15
drm: s/enum_blob_list/enum_list/ in drm_property

I guess for hysterical raisins this was meant to be the way to read
blob properties. But that's done with the two-stage approach which
uses separate blob kms object and the special-purpose get_blob ioctl.

Shipping userspace seems to have never relied on this, and the kernel
also never put any blob thing onto that property. And nowadays it
would blow up, e.g. in drm_property_destroy. Also it makes no sense to
return values in an ioctl that only returns metadata about everything.

So let's ditch all the internal code for the blob list, rename the
list to be unambiguous and sprinkle comments all over the place to
explain this peculiar piece of api.

v2: Squash in fixup from Rob to remove now unused variables.

Cc: Rob Clark <robdclark@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Reviewed-by: Rob Clark <robdclark@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/drm_crtc.c
include/drm/drm_crtc.h
include/uapi/drm/drm_mode.h