gpu/drm: delete same check in if condition
authorBernard Zhao <bernard@vivo.com>
Mon, 2 Nov 2020 03:07:36 +0000 (19:07 -0800)
committerThomas Zimmermann <tzimmermann@suse.de>
Mon, 2 Nov 2020 13:04:53 +0000 (14:04 +0100)
In function drm_bridge_connector_get_modes_edid, drm_edid_is_valid
will check weather (!edid), no need to check again in the if
branch.

Signed-off-by: Bernard Zhao <bernard@vivo.com>
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20201102030736.3833-1-bernard@vivo.com
drivers/gpu/drm/drm_bridge_connector.c

index a58cbde..7913798 100644 (file)
@@ -241,7 +241,7 @@ static int drm_bridge_connector_get_modes_edid(struct drm_connector *connector,
                goto no_edid;
 
        edid = bridge->funcs->get_edid(bridge, connector);
-       if (!edid || !drm_edid_is_valid(edid)) {
+       if (!drm_edid_is_valid(edid)) {
                kfree(edid);
                goto no_edid;
        }