diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2012-05-17 13:27:21 +0200 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2012-05-22 11:32:58 +0200 |
commit | 4a1b0714275796fdbc35427cf361eb4123e5e9f6 (patch) | |
tree | 1897d2d9cbec66d84d69ace29ddedfc1778f2f4e /drivers/gpu/drm/drm_edid_load.c | |
parent | drm: Delete the vblank timer synchronously at cleanup time (diff) | |
download | linux-4a1b0714275796fdbc35427cf361eb4123e5e9f6.tar.xz linux-4a1b0714275796fdbc35427cf361eb4123e5e9f6.zip |
drm: Don't initialize local ret variable when not needed
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.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 | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c index 48c927c37041..66d4a28ad5a2 100644 --- a/drivers/gpu/drm/drm_edid_load.c +++ b/drivers/gpu/drm/drm_edid_load.c @@ -220,18 +220,18 @@ int drm_load_edid_firmware(struct drm_connector *connector) { char *connector_name = drm_get_connector_name(connector); char *edidname = edid_firmware, *last, *colon; - int ret = 0; + int ret; if (*edidname == '\0') - return ret; + return 0; colon = strchr(edidname, ':'); if (colon != NULL) { if (strncmp(connector_name, edidname, colon - edidname)) - return ret; + return 0; edidname = colon + 1; if (*edidname == '\0') - return ret; + return 0; } last = edidname + strlen(edidname) - 1; |