drm/omap: Don't pass display pointer to encoder init function
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Wed, 12 Sep 2018 23:23:26 +0000 (02:23 +0300)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Mon, 18 Mar 2019 09:42:12 +0000 (11:42 +0200)
The display isn't used by the encoder implementation, don't pass it to
the initialization function and store it internally needlessly.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.com>
Tested-by: Sebastian Reichel <sebastian.reichel@collabora.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
drivers/gpu/drm/omapdrm/omap_drv.c
drivers/gpu/drm/omapdrm/omap_encoder.c
drivers/gpu/drm/omapdrm/omap_encoder.h

index bddd4c1..3b8f0fd 100644 (file)
@@ -296,7 +296,7 @@ static int omap_modeset_init(struct drm_device *dev)
                struct drm_encoder *encoder;
                struct drm_crtc *crtc;
 
-               encoder = omap_encoder_init(dev, pipe->output, display);
+               encoder = omap_encoder_init(dev, pipe->output);
                if (!encoder)
                        return -ENOMEM;
 
index acdfd21..1f4172f 100644 (file)
@@ -37,7 +37,6 @@
 struct omap_encoder {
        struct drm_encoder base;
        struct omap_dss_device *output;
-       struct omap_dss_device *display;
 };
 
 static void omap_encoder_destroy(struct drm_encoder *encoder)
@@ -247,8 +246,7 @@ static const struct drm_encoder_helper_funcs omap_encoder_helper_funcs = {
 
 /* initialize encoder */
 struct drm_encoder *omap_encoder_init(struct drm_device *dev,
-                                     struct omap_dss_device *output,
-                                     struct omap_dss_device *display)
+                                     struct omap_dss_device *output)
 {
        struct drm_encoder *encoder = NULL;
        struct omap_encoder *omap_encoder;
@@ -258,7 +256,6 @@ struct drm_encoder *omap_encoder_init(struct drm_device *dev,
                goto fail;
 
        omap_encoder->output = output;
-       omap_encoder->display = display;
 
        encoder = &omap_encoder->base;
 
index a7b5dde..4aefb31 100644 (file)
@@ -25,7 +25,6 @@ struct drm_encoder;
 struct omap_dss_device;
 
 struct drm_encoder *omap_encoder_init(struct drm_device *dev,
-                                     struct omap_dss_device *output,
-                                     struct omap_dss_device *display);
+                                     struct omap_dss_device *output);
 
 #endif /* __OMAPDRM_ENCODER_H__ */