diff options
author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2012-10-25 14:05:06 -0400 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2012-11-06 19:14:24 -0500 |
commit | c4a56750ecbf2421ebd39f0fec562e1869a38d53 (patch) | |
tree | 0a0c8e0bd36ed4ba4ac87f7ded95300164e47634 | |
parent | 0b3904ab2a48488e23332ac1ecd2d45961ec6718 (diff) |
drm: Ignore blob propertys in drm_property_change_is_valid()
In case of a blob property drm_property_change_is_valid() can't
tell whether the change is valid or not. So just return true
for all blob properties, and leave it up to someone else to
check it.
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
-rw-r--r-- | drivers/gpu/drm/drm_crtc.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index d9a639c870f4..35336097fb47 100644 --- a/drivers/gpu/drm/drm_crtc.c +++ b/drivers/gpu/drm/drm_crtc.c | |||
@@ -3212,6 +3212,9 @@ static bool drm_property_change_is_valid(struct drm_property *property, | |||
3212 | for (i = 0; i < property->num_values; i++) | 3212 | for (i = 0; i < property->num_values; i++) |
3213 | valid_mask |= (1ULL << property->values[i]); | 3213 | valid_mask |= (1ULL << property->values[i]); |
3214 | return !(value & ~valid_mask); | 3214 | return !(value & ~valid_mask); |
3215 | } else if (property->flags & DRM_MODE_PROP_BLOB) { | ||
3216 | /* Only the driver knows */ | ||
3217 | return true; | ||
3215 | } else { | 3218 | } else { |
3216 | int i; | 3219 | int i; |
3217 | for (i = 0; i < property->num_values; i++) | 3220 | for (i = 0; i < property->num_values; i++) |