ASoC: hdac_hdmi: Remove 'edev' NULL check
authorFabio Estevam <fabio.estevam@nxp.com>
Wed, 17 Feb 2016 11:42:13 +0000 (09:42 -0200)
committerMark Brown <broonie@kernel.org>
Wed, 17 Feb 2016 13:10:38 +0000 (13:10 +0000)
Commit 4a3478debf36 ("ASoC: hdac_hdmi: Add jack reporting")
leads to the following Smatch complaint:

sound/soc/codecs/hdac_hdmi.c:971 hdac_hdmi_present_sense()
         warn: variable dereferenced before check 'edev' (see line 967)

As explained by Subhransu S. Prusty:

"This NULL check is not required, can be removed.

edev object is expected to be present when the present_sense is
called. This is called from places where this edev object is
already available."

So remove the evdev NULL check.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
Acked-by: Subhransu S. Prusty <subhransu.s.prusty@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/hdac_hdmi.c

index 36f12004ae9c86246a74bc2547ce7a35a338ef2c..2884dbf0bc072ba80d883f475c753d9712998efc 100644 (file)
@@ -968,9 +968,6 @@ static void hdac_hdmi_present_sense(struct hdac_hdmi_pin *pin, int repoll)
        struct hdac_hdmi_pcm *pcm;
        int val;
 
-       if (!edev)
-               return;
-
        pin->repoll_count = repoll;
 
        pm_runtime_get_sync(&edev->hdac.dev);