drm/panel: s6e63m0: Set up some display info
authorLinus Walleij <linus.walleij@linaro.org>
Tue, 10 Nov 2020 23:46:53 +0000 (00:46 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Sun, 15 Nov 2020 23:48:16 +0000 (00:48 +0100)
Copy over the width/height in millimeters to the
(somewhat redundant) display info, and set up the
bus format and bus flags for the display.

When used as DPI this display requires DE to be
active low and pixel data to be output on the
negative edge. It might be that it was previously
used with a display controller that either does
not support these settings or was hardcoded to use
these as default. This information comes from the
source code of the Samsung GT-I9070 mobile phone.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Stephan Gerhold <stephan@gerhold.net>
Cc: Paweł Chmiel <pawel.mikolaj.chmiel@gmail.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20201110234653.2248594-6-linus.walleij@linaro.org
drivers/gpu/drm/panel/panel-samsung-s6e63m0.c

index 3eee67e2d86a4eae3a37d19eacbbbb6bdc1c4235..210e70da3a15350ad70d988cbc67e901f49be82d 100644 (file)
@@ -16,6 +16,7 @@
 #include <linux/gpio/consumer.h>
 #include <linux/module.h>
 #include <linux/regulator/consumer.h>
+#include <linux/media-bus-format.h>
 
 #include <video/mipi_display.h>
 
@@ -410,6 +411,7 @@ static int s6e63m0_get_modes(struct drm_panel *panel,
                             struct drm_connector *connector)
 {
        struct drm_display_mode *mode;
+       static const u32 bus_format = MEDIA_BUS_FMT_RGB888_1X24;
 
        mode = drm_mode_duplicate(connector->dev, &default_mode);
        if (!mode) {
@@ -419,6 +421,13 @@ static int s6e63m0_get_modes(struct drm_panel *panel,
                return -ENOMEM;
        }
 
+       connector->display_info.width_mm = mode->width_mm;
+       connector->display_info.height_mm = mode->height_mm;
+       drm_display_info_set_bus_formats(&connector->display_info,
+                                        &bus_format, 1);
+       connector->display_info.bus_flags = DRM_BUS_FLAG_DE_LOW |
+               DRM_BUS_FLAG_PIXDATA_DRIVE_NEGEDGE;
+
        drm_mode_set_name(mode);
 
        mode->type = DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED;