drm/omap: remove extra manager checks on disconnect
authorTomi Valkeinen <tomi.valkeinen@ti.com>
Thu, 5 Nov 2015 07:44:53 +0000 (09:44 +0200)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Thu, 3 Mar 2016 15:38:23 +0000 (17:38 +0200)
The DSS output drivers check 'dssdev->manager' in disconnect()
functions. This check is not needed as the manager must always be set if
the output device was connected. Remove the check.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
drivers/gpu/drm/omapdrm/dss/dpi.c
drivers/gpu/drm/omapdrm/dss/dsi.c
drivers/gpu/drm/omapdrm/dss/hdmi4.c
drivers/gpu/drm/omapdrm/dss/hdmi5.c
drivers/gpu/drm/omapdrm/dss/sdi.c
drivers/gpu/drm/omapdrm/dss/venc.c

index d4be2444b2eaa63d7eb5f452ca1c8e002428f487..c73d3651ca10cc35cb829504bc2205993703966a 100644 (file)
@@ -701,8 +701,7 @@ static void dpi_disconnect(struct omap_dss_device *dssdev,
 
        omapdss_output_unset_device(dssdev);
 
-       if (dssdev->manager)
-               dss_mgr_disconnect(dssdev->manager->id, dssdev);
+       dss_mgr_disconnect(dssdev->manager->id, dssdev);
 }
 
 static const struct omapdss_dpi_ops dpi_ops = {
index 10099f75364df55094c26ee963278aa547fffe74..d00c6a31602d74241f4d88f846ff1dfa5f206350 100644 (file)
@@ -5019,8 +5019,7 @@ static void dsi_disconnect(struct omap_dss_device *dssdev,
 
        omapdss_output_unset_device(dssdev);
 
-       if (dssdev->manager)
-               dss_mgr_disconnect(dssdev->manager->id, dssdev);
+       dss_mgr_disconnect(dssdev->manager->id, dssdev);
 }
 
 static const struct omapdss_dsi_ops dsi_ops = {
index f8ed407f148e3e3ddb5101f6b4aa0c5c7985a77c..a222fc5089a546fa78fe9e47ebf10c98a1fdb0ec 100644 (file)
@@ -474,8 +474,7 @@ static void hdmi_disconnect(struct omap_dss_device *dssdev,
 
        omapdss_output_unset_device(dssdev);
 
-       if (dssdev->manager)
-               dss_mgr_disconnect(dssdev->manager->id, dssdev);
+       dss_mgr_disconnect(dssdev->manager->id, dssdev);
 }
 
 static int hdmi_read_edid(struct omap_dss_device *dssdev,
index b75f15e15797598bc184831a8ef2b2e9acb45de8..33cda9bcf952262fee4888ec7bc53ad3b9db4064 100644 (file)
@@ -500,8 +500,7 @@ static void hdmi_disconnect(struct omap_dss_device *dssdev,
 
        omapdss_output_unset_device(dssdev);
 
-       if (dssdev->manager)
-               dss_mgr_disconnect(dssdev->manager->id, dssdev);
+       dss_mgr_disconnect(dssdev->manager->id, dssdev);
 }
 
 static int hdmi_read_edid(struct omap_dss_device *dssdev,
index 0526f24ed8a9a13c6f89eb350f2c9811ec9b7e07..dee40ba9a2963bdfe58773e02ee623d1f6244a64 100644 (file)
@@ -316,8 +316,7 @@ static void sdi_disconnect(struct omap_dss_device *dssdev,
 
        omapdss_output_unset_device(dssdev);
 
-       if (dssdev->manager)
-               dss_mgr_disconnect(dssdev->manager->id, dssdev);
+       dss_mgr_disconnect(dssdev->manager->id, dssdev);
 }
 
 static const struct omapdss_sdi_ops sdi_ops = {
index 401c477242b100f8cc73b04d909b7080f2804d3b..d514c9878ae2561340a37b3210f7cb072a826fef 100644 (file)
@@ -778,8 +778,7 @@ static void venc_disconnect(struct omap_dss_device *dssdev,
 
        omapdss_output_unset_device(dssdev);
 
-       if (dssdev->manager)
-               dss_mgr_disconnect(dssdev->manager->id, dssdev);
+       dss_mgr_disconnect(dssdev->manager->id, dssdev);
 }
 
 static const struct omapdss_atv_ops venc_ops = {