diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-19 10:33:17 -0500 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2014-11-20 21:16:27 -0500 |
commit | 9084acf19bb98afb0ed90337fd39541103fe3494 (patch) | |
tree | 047501ec96cb3c4fa99a90ed9d098b487241a095 /drivers/gpu/drm/drm_edid_load.c | |
parent | fc2d2bc1f1f8d8351f884bb65b5cc4ad2411b2a7 (diff) |
drm/edid: Deletion of an unnecessary check before the function call "release_firmware"
The release_firmware() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Reviewed-by: Thierry Reding <thierry.reding@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/drm_edid_load.c')
-rw-r--r-- | drivers/gpu/drm/drm_edid_load.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c index 0a235fe61c9b..732cb6f8e653 100644 --- a/drivers/gpu/drm/drm_edid_load.c +++ b/drivers/gpu/drm/drm_edid_load.c | |||
@@ -254,8 +254,7 @@ static void *edid_load(struct drm_connector *connector, const char *name, | |||
254 | name, connector_name); | 254 | name, connector_name); |
255 | 255 | ||
256 | out: | 256 | out: |
257 | if (fw) | 257 | release_firmware(fw); |
258 | release_firmware(fw); | ||
259 | return edid; | 258 | return edid; |
260 | } | 259 | } |
261 | 260 | ||