drm/msm/hdmi: Delete an unnecessary check before the function call "kfree"
authorMarkus Elfring <elfring@users.sourceforge.net>
Wed, 13 Jul 2016 16:54:11 +0000 (18:54 +0200)
committerRob Clark <robdclark@gmail.com>
Sat, 16 Jul 2016 14:29:33 +0000 (10:29 -0400)
The kfree() 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>
Signed-off-by: Rob Clark <robdclark@gmail.com>
drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c

index 0baaaaabd0027792fcba54d6f9ad0a157eada5ca..6e767979aab3fe5d258849b7aaddcb60f94f7d5a 100644 (file)
@@ -1430,7 +1430,7 @@ struct hdmi_hdcp_ctrl *msm_hdmi_hdcp_init(struct hdmi *hdmi)
 
 void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
 {
-       if (hdmi && hdmi->hdcp_ctrl) {
+       if (hdmi) {
                kfree(hdmi->hdcp_ctrl);
                hdmi->hdcp_ctrl = NULL;
        }