drm/omap: Store bus flags in the omap_dss_device structure

Source components in the display pipeline need to configure their output
signals polarities and clock driving edge based on the requirements of
the sink component.

Those requirements are currently shared across the whole pipeline in the
flags of a videomode structure, instead of being local to each bus. This
both prevents multiple buses from having different configurations (when
the hardware supports it), and makes it difficult to move from videomode
to drm_display_mode as the latter doesn't contain bus polarities and
clock edge flags.

Add a bus_flags field to the omap_dss_device structure and move the
DISPLAY_FLAGS_DE_(LOW|HIGH), DISPLAY_FLAGS_PIXDATA_(POS|NEG)EDGE and
DISPLAY_FLAGS_SYNC_(POS|NEG)EDGE videomode flags to bus_flags in all
external encoders, connectors and panels. The videomode flags are still
used internally for internal encoders, this will be addressed in a
second step.

The related videomode flags in the default mode of the DVI connector can
simply be dropped, as they are always overridden by the TFP410 driver.
Note that this results in both the DISPLAY_FLAGS_SYNC_POSEDGE and
DISPLAY_FLAGS_SYNC_NEGEDGE flags being set, which is invalid, but only
the former is tested for when programming the DISPC, so the DVI
connector flags are effectively overridden by the TFP410 flags.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

1 files changed, 31 insertions, 11 deletions

diff --git a/drivers/gpu/drm/omapdrm/omap_crtc.c b/drivers/gpu/drm/omapdrm/omap_crtc.c
index 6e7a777907f5..39693dfe54af 100644
--- a/drivers/gpu/drm/omapdrm/omap_crtc.c
+++ b/drivers/gpu/drm/omapdrm/omap_crtc.c
@@ -419,12 +419,9 @@ static enum drm_mode_status omap_crtc_mode_valid(struct drm_crtc *crtc,
 static void omap_crtc_mode_set_nofb(struct drm_crtc *crtc)
 {
         struct omap_crtc *omap_crtc = to_omap_crtc(crtc);
-        struct omap_dss_device *display = omap_crtc->pipe->display;
         struct drm_display_mode *mode = &crtc->state->adjusted_mode;
-        const u32 flags_mask = DISPLAY_FLAGS_DE_HIGH | DISPLAY_FLAGS_DE_LOW |
-                DISPLAY_FLAGS_PIXDATA_POSEDGE | DISPLAY_FLAGS_PIXDATA_NEGEDGE |
-                DISPLAY_FLAGS_SYNC_POSEDGE | DISPLAY_FLAGS_SYNC_NEGEDGE;
-        struct videomode vm = {0};
+        struct videomode *vm = &omap_crtc->vm;
+        struct omap_dss_device *dssdev;
 
         DBG("%s: set mode: %d:\"%s\" %d %d %d %d %d %d %d %d %d %d 0x%x 0x%x",
             omap_crtc->name, mode->base.id, mode->name,
@@ -433,7 +430,7 @@ static void omap_crtc_mode_set_nofb(struct drm_crtc *crtc)
             mode->vdisplay, mode->vsync_start, mode->vsync_end, mode->vtotal,
             mode->type, mode->flags);
 
-        drm_display_mode_to_videomode(mode, &omap_crtc->vm);
+        drm_display_mode_to_videomode(mode, vm);
 
         /*
          * HACK: This fixes the vm flags.
@@ -442,13 +439,36 @@ static void omap_crtc_mode_set_nofb(struct drm_crtc *crtc)
          * struct drm_display_mode and struct videomode. The hack below
          * goes and fetches the missing flags from the panel drivers.
          *
-         * Correct solution would be to use DRM's bus-flags, but that's not
-         * easily possible before the omapdrm's panel/encoder driver model
-         * has been changed to the DRM model.
+         * A better solution is to use DRM's bus-flags through the whole driver.
          */
 
-        display->ops->get_timings(display, &vm);
-        omap_crtc->vm.flags |= vm.flags & flags_mask;
+        for (dssdev = omap_crtc->pipe->output; dssdev; dssdev = dssdev->next) {
+                unsigned long bus_flags = dssdev->bus_flags;
+
+                if (!(vm->flags & (DISPLAY_FLAGS_DE_LOW |
+                                   DISPLAY_FLAGS_DE_HIGH))) {
+                        if (bus_flags & DRM_BUS_FLAG_DE_LOW)
+                                vm->flags |= DISPLAY_FLAGS_DE_LOW;
+                        else if (bus_flags & DRM_BUS_FLAG_DE_HIGH)
+                                vm->flags |= DISPLAY_FLAGS_DE_HIGH;
+                }
+
+                if (!(vm->flags & (DISPLAY_FLAGS_PIXDATA_POSEDGE |
+                                   DISPLAY_FLAGS_PIXDATA_NEGEDGE))) {
+                        if (bus_flags & DRM_BUS_FLAG_PIXDATA_POSEDGE)
+                                vm->flags |= DISPLAY_FLAGS_PIXDATA_POSEDGE;
+                        else if (bus_flags & DRM_BUS_FLAG_PIXDATA_NEGEDGE)
+                                vm->flags |= DISPLAY_FLAGS_PIXDATA_NEGEDGE;
+                }
+
+                if (!(vm->flags & (DISPLAY_FLAGS_SYNC_POSEDGE |
+                                   DISPLAY_FLAGS_SYNC_NEGEDGE))) {
+                        if (bus_flags & DRM_BUS_FLAG_SYNC_POSEDGE)
+                                vm->flags |= DISPLAY_FLAGS_SYNC_POSEDGE;
+                        else if (bus_flags & DRM_BUS_FLAG_SYNC_NEGEDGE)
+                                vm->flags |= DISPLAY_FLAGS_SYNC_NEGEDGE;
+                }
+        }
 }
 
 static int omap_crtc_atomic_check(struct drm_crtc *crtc,