<p dir="ltr">On Nov 19, 2014 4:41 PM, "SF Markus Elfring" <<a href="mailto:elfring@users.sourceforge.net">elfring@users.sourceforge.net</a>> wrote:<br>
><br>
> From: Markus Elfring <<a href="mailto:elfring@users.sourceforge.net">elfring@users.sourceforge.net</a>><br>
> Date: Wed, 19 Nov 2014 16:33:17 +0100<br>
><br>
> The release_firmware() function tests whether its argument is NULL and then<br>
> returns immediately. Thus the test around the call is not needed.<br>
><br>
> This issue was detected by using the Coccinelle software.<br>
><br>
> Signed-off-by: Markus Elfring <<a href="mailto:elfring@users.sourceforge.net">elfring@users.sourceforge.net</a>><br>
> ---<br>
>  drivers/gpu/drm/drm_edid_load.c | 3 +--<br>
>  1 file changed, 1 insertion(+), 2 deletions(-)<br>
><br>
> diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c<br>
> index 0a235fe..732cb6f 100644<br>
> --- a/drivers/gpu/drm/drm_edid_load.c<br>
> +++ b/drivers/gpu/drm/drm_edid_load.c<br>
> @@ -254,8 +254,7 @@ static void *edid_load(struct drm_connector *connector, const char *name,<br>
>             name, connector_name);<br>
><br>
>  out:<br>
> -       if (fw)<br>
> -               release_firmware(fw);<br>
> +       release_firmware(fw);<br>
>         return edid;<br>
>  }<br>
></p>
<p dir="ltr">Reviewed-by: Thierry Reding <<a href="mailto:thierry.reding@gmail.com">thierry.reding@gmail.com</a>></p>