diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-19 18:33:17 +0300 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2014-11-21 05:16:27 +0300 |
| commit | 9084acf19bb98afb0ed90337fd39541103fe3494 (patch) | |
| tree | 047501ec96cb3c4fa99a90ed9d098b487241a095 /drivers/gpu/drm | |
| parent | fc2d2bc1f1f8d8351f884bb65b5cc4ad2411b2a7 (diff) | |
| download | linux-9084acf19bb98afb0ed90337fd39541103fe3494.tar.xz | |
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')
| -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, name, connector_name); out: - if (fw) - release_firmware(fw); + release_firmware(fw); return edid; } |