drm: omapdrm: displays: panel-dsi-cm: Fix field access before set
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Thu, 1 Mar 2018 20:02:30 +0000 (22:02 +0200)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Fri, 2 Mar 2018 13:11:07 +0000 (15:11 +0200)
The driver accesses the ddata->in field before it gets set in the
dsicm_connect() function. Use the local in pointer variable instead.

Fixes: 7877632b4cd0 ("drm: omapdrm: displays: Get panel source at connect time")
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c

index f960e55d64eab351c99442353af3c4c74640b895..428de90fced18b1077dc7b9c6ab4a1a393b92739 100644 (file)
@@ -778,13 +778,13 @@ static int dsicm_connect(struct omap_dss_device *dssdev)
                goto err_connect;
        }
 
-       r = in->ops.dsi->request_vc(ddata->in, &ddata->channel);
+       r = in->ops.dsi->request_vc(in, &ddata->channel);
        if (r) {
                dev_err(dev, "failed to get virtual channel\n");
                goto err_req_vc;
        }
 
-       r = in->ops.dsi->set_vc_id(ddata->in, ddata->channel, TCH);
+       r = in->ops.dsi->set_vc_id(in, ddata->channel, TCH);
        if (r) {
                dev_err(dev, "failed to set VC_ID\n");
                goto err_vc_id;
@@ -794,7 +794,7 @@ static int dsicm_connect(struct omap_dss_device *dssdev)
        return 0;
 
 err_vc_id:
-       in->ops.dsi->release_vc(ddata->in, ddata->channel);
+       in->ops.dsi->release_vc(in, ddata->channel);
 err_req_vc:
        in->ops.dsi->disconnect(in, dssdev);
 err_connect: