diff options
| author | Thierry Reding <treding@nvidia.com> | 2014-12-10 07:03:39 -0500 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-12-10 08:14:53 -0500 |
| commit | 2ca651d15aff0f613d11913e8a10e3b86b3dc44a (patch) | |
| tree | 9b3be1531ade77a67a4108537e0a819b05603726 | |
| parent | 01073b0846c7c13438ef9b4d62e0747aed2db15d (diff) | |
drm: Remove useless else block
All prior conditional blocks return from the function, so the else block
can be at the top level of the function.
Signed-off-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
| -rw-r--r-- | drivers/gpu/drm/drm_crtc.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index fa1b1a5b8330..4f4fcd37f110 100644 --- a/drivers/gpu/drm/drm_crtc.c +++ b/drivers/gpu/drm/drm_crtc.c | |||
| @@ -4135,6 +4135,8 @@ EXPORT_SYMBOL(drm_mode_connector_update_edid_property); | |||
| 4135 | static bool drm_property_change_is_valid(struct drm_property *property, | 4135 | static bool drm_property_change_is_valid(struct drm_property *property, |
| 4136 | uint64_t value) | 4136 | uint64_t value) |
| 4137 | { | 4137 | { |
| 4138 | int i; | ||
| 4139 | |||
| 4138 | if (property->flags & DRM_MODE_PROP_IMMUTABLE) | 4140 | if (property->flags & DRM_MODE_PROP_IMMUTABLE) |
| 4139 | return false; | 4141 | return false; |
| 4140 | 4142 | ||
| @@ -4150,7 +4152,6 @@ static bool drm_property_change_is_valid(struct drm_property *property, | |||
| 4150 | return false; | 4152 | return false; |
| 4151 | return true; | 4153 | return true; |
| 4152 | } else if (drm_property_type_is(property, DRM_MODE_PROP_BITMASK)) { | 4154 | } else if (drm_property_type_is(property, DRM_MODE_PROP_BITMASK)) { |
| 4153 | int i; | ||
| 4154 | uint64_t valid_mask = 0; | 4155 | uint64_t valid_mask = 0; |
| 4155 | 4156 | ||
| 4156 | for (i = 0; i < property->num_values; i++) | 4157 | for (i = 0; i < property->num_values; i++) |
| @@ -4174,13 +4175,12 @@ static bool drm_property_change_is_valid(struct drm_property *property, | |||
| 4174 | */ | 4175 | */ |
| 4175 | obj = _object_find(property->dev, value, property->values[0]); | 4176 | obj = _object_find(property->dev, value, property->values[0]); |
| 4176 | return obj != NULL; | 4177 | return obj != NULL; |
| 4177 | } else { | ||
| 4178 | int i; | ||
| 4179 | for (i = 0; i < property->num_values; i++) | ||
| 4180 | if (property->values[i] == value) | ||
| 4181 | return true; | ||
| 4182 | return false; | ||
| 4183 | } | 4178 | } |
| 4179 | |||
| 4180 | for (i = 0; i < property->num_values; i++) | ||
| 4181 | if (property->values[i] == value) | ||
| 4182 | return true; | ||
| 4183 | return false; | ||
| 4184 | } | 4184 | } |
| 4185 | 4185 | ||
| 4186 | /** | 4186 | /** |
