Hello Tomi Valkeinen,

This is a semi-automatic email about new static checker warnings.

The patch 5391e87d1259: "OMAPDSS: remove dispc's dependency to 
VENC/HDMI" from May 16, 2013, leads to the following Smatch complaint:

drivers/video/omap2/dss/hdmi.c:672 omapdss_hdmi_display_set_timing()
         error: we previously assumed 't' could be null (see line 669)

drivers/video/omap2/dss/hdmi.c
   668          t = hdmi_get_timings();
   669          if (t != NULL)
                    ^^^^^^^^^
Existing check.

   670                  hdmi.ip_data.cfg = *t;
   671  
   672          dispc_set_tv_pclk(t->timings.pixel_clock * 1000);
                                  ^^^^^^^^^^^^^^^^^^^^^^
Patch added a dereference.

   673  
   674          mutex_unlock(&hdmi.lock);

regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to