On Tue, Jun 24, 2014 at 02:52:11PM -0500, Rob Herring wrote:

On Tue, Jun 24, 2014 at 10:06 AM, Russell King - ARM Linux [off-list ref] wrote:

[...]

quoted

On Mon, Jun 16, 2014 at 12:11:22PM +0200, Denis Carikli wrote:

[...]

quoted

quoted

diff --git a/Documentation/devicetree/bindings/panel/eukrea,mbimxsd51-dvi-vga.txt b/Documentation/devicetree/bindings/panel/eukrea,mbimxsd51-dvi-vga.txt

[...]

quoted

quoted

@@ -0,0 +1,7 @@
+Eukrea DVI-VGA (640x480 pixels) DVI output.
+
+Required properties:
+- compatible: should be "eukrea,mbimxsd51-dvi-vga"
+
+This binding is compatible with the simple-panel binding, which is specified
+in simple-panel.txt in this directory.

Seems like we could just have a list of compatible strings rather than
a mostly duplicated file.

We've been doing it this way for all other panels.

quoted

quoted

diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
index a251361..adc40a7 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c

@@ -403,6 +403,80 @@ static const struct panel_desc edt_etm0700g0dh6 = {
      },
 };

+static const struct drm_display_mode eukrea_mbimxsd51_cmoqvga_mode = {
+     .clock = 6500,
+     .hdisplay = 320,
+     .hsync_start = 320 + 38,
+     .hsync_end = 320 + 38 + 20,
+     .htotal = 320 + 38 + 20 + 30,
+     .vdisplay = 240,
+     .vsync_start = 240 + 15,
+     .vsync_end = 240 + 15 + 4,
+     .vtotal = 240 + 15 + 4 + 3,
+     .vrefresh = 60,
+     .pol_flags = DRM_MODE_FLAG_POL_PIXDATA_NEGEDGE |
+                  DRM_MODE_FLAG_POL_DE_LOW,

Why aren't you using:

Documentation/devicetree/bindings/video/display-timing.txt

Because it's redundant information. We need to have a compatible for the
panel in the device tree anyway and that already implicitly defines the
display mode.

Thierry