drivers/gpu/drm/openchrome/openchrome_crtc.c    |  295 +++++++++---------------
 drivers/gpu/drm/openchrome/openchrome_display.c |    4 
 drivers/gpu/drm/openchrome/openchrome_drv.h     |   10 
 3 files changed, 132 insertions(+), 177 deletions(-)

New commits:
commit 8fe47f76b3f2822a6a7a0898b2fdde5964388ba6
Author: Kevin Brace <kevinbr...@gmx.com>
Date:   Sat Oct 19 19:16:38 2019 -0700

    drm/openchrome: Version bumped to 3.1.19
    
    Extensive refactoring of DRM plane and CRTC code.
    
    Signed-off-by: Kevin Brace <kevinbr...@gmx.com>

diff --git a/drivers/gpu/drm/openchrome/openchrome_drv.h 
b/drivers/gpu/drm/openchrome/openchrome_drv.h
index a91744aa9a50..2c5d8a71dbc7 100644
--- a/drivers/gpu/drm/openchrome/openchrome_drv.h
+++ b/drivers/gpu/drm/openchrome/openchrome_drv.h
@@ -61,10 +61,10 @@
 
 #define DRIVER_MAJOR           3
 #define DRIVER_MINOR           1
-#define DRIVER_PATCHLEVEL      18
+#define DRIVER_PATCHLEVEL      19
 #define DRIVER_NAME            "openchrome"
 #define DRIVER_DESC            "OpenChrome DRM for VIA Technologies Chrome IGP"
-#define DRIVER_DATE            "20191016"
+#define DRIVER_DATE            "20191019"
 #define DRIVER_AUTHOR          "OpenChrome Project"
 
 
commit 0e8ba736159565ff57fabbff4946861f5052ea30
Author: Kevin Brace <kevinbr...@gmx.com>
Date:   Sat Oct 19 11:20:44 2019 -0700

    drm/openchrome: Rename via_crtc_init()
    
    The new name is openchrome_crtc_param_init().
    
    Signed-off-by: Kevin Brace <kevinbr...@gmx.com>

diff --git a/drivers/gpu/drm/openchrome/openchrome_crtc.c 
b/drivers/gpu/drm/openchrome/openchrome_crtc.c
index dbed11a66ea3..5bb3eb1b6d95 100644
--- a/drivers/gpu/drm/openchrome/openchrome_crtc.c
+++ b/drivers/gpu/drm/openchrome/openchrome_crtc.c
@@ -2164,15 +2164,15 @@ exit:
        return ret;
 }
 
-int via_crtc_init(struct drm_device *dev, uint32_t index)
+void openchrome_crtc_param_init(
+               struct openchrome_drm_private *dev_private,
+               uint32_t index)
 {
-       struct openchrome_drm_private *dev_private =
-                                               dev->dev_private;
+       struct drm_device *dev = dev_private->dev;
        struct via_crtc *iga = &dev_private->iga[index];
        struct drm_crtc *crtc = &iga->base;
        u16 *gamma;
        uint32_t i;
-       int ret = 0;
 
        if (iga->index) {
                iga->timings.htotal.count = ARRAY_SIZE(iga2_hor_total);
@@ -2332,6 +2332,4 @@ int via_crtc_init(struct drm_device *dev, uint32_t index)
                gamma[i + 256] = i << 8 | i;
                gamma[i + 512] = i << 8 | i;
        }
-
-       return ret;
 }
diff --git a/drivers/gpu/drm/openchrome/openchrome_display.c 
b/drivers/gpu/drm/openchrome/openchrome_display.c
index e8482e9a207a..91ff28861703 100644
--- a/drivers/gpu/drm/openchrome/openchrome_display.c
+++ b/drivers/gpu/drm/openchrome/openchrome_display.c
@@ -499,10 +499,7 @@ via_modeset_init(struct drm_device *dev)
                        goto exit;
                }
 
-               ret = via_crtc_init(dev, i);
-               if (ret) {
-                       goto exit;
-               }
+               openchrome_crtc_param_init(dev_private, i);
        }
 
        openchrome_ext_dvi_probe(dev);
diff --git a/drivers/gpu/drm/openchrome/openchrome_drv.h 
b/drivers/gpu/drm/openchrome/openchrome_drv.h
index 922b6ca964f9..a91744aa9a50 100644
--- a/drivers/gpu/drm/openchrome/openchrome_drv.h
+++ b/drivers/gpu/drm/openchrome/openchrome_drv.h
@@ -468,7 +468,9 @@ extern void via_load_crtc_pixel_timing(struct drm_crtc 
*crtc,
                                        struct drm_display_mode *mode);
 int openchrome_crtc_init(struct openchrome_drm_private *dev_private,
                                uint32_t index);
-extern int via_crtc_init(struct drm_device *dev, uint32_t index);
+void openchrome_crtc_param_init(
+                       struct openchrome_drm_private *dev_private,
+                       uint32_t index);
 
 /* encoders */
 extern void via_set_sync_polarity(struct drm_encoder *encoder,
commit 76a37faf5de6efd4a37517a1658019b42b860c89
Author: Kevin Brace <kevinbr...@gmx.com>
Date:   Sat Oct 19 11:05:44 2019 -0700

    drm/openchrome: Move plane and CRTC initialization code into a new function
    
    Signed-off-by: Kevin Brace <kevinbr...@gmx.com>

diff --git a/drivers/gpu/drm/openchrome/openchrome_crtc.c 
b/drivers/gpu/drm/openchrome/openchrome_crtc.c
index c31ad8acb693..dbed11a66ea3 100644
--- a/drivers/gpu/drm/openchrome/openchrome_crtc.c
+++ b/drivers/gpu/drm/openchrome/openchrome_crtc.c
@@ -2073,17 +2073,16 @@ static const uint32_t openchrome_primary_formats[] = {
        DRM_FORMAT_RGB332,
 };
 
-int via_crtc_init(struct drm_device *dev, uint32_t index)
+int openchrome_crtc_init(struct openchrome_drm_private *dev_private,
+                               uint32_t index)
 {
-       struct openchrome_drm_private *dev_private =
-                                               dev->dev_private;
+       struct drm_device *dev = dev_private->dev;
        struct via_crtc *iga = &dev_private->iga[index];
        struct drm_plane *primary;
        struct drm_plane *cursor;
        struct drm_crtc *crtc = &iga->base;
        uint32_t possible_crtcs;
-       int cursor_size = 64 * 64 * 4, i;
-       u16 *gamma;
+       uint64_t cursor_size = 64 * 64 * 4;
        int ret;
 
        iga->index = index;
@@ -2152,6 +2151,29 @@ int via_crtc_init(struct drm_device *dev, uint32_t index)
                goto cleanup_cursor;
        }
 
+       goto exit;
+cleanup_cursor:
+       drm_plane_cleanup(cursor);
+free_cursor:
+       kfree(cursor);
+cleanup_primary:
+       drm_plane_cleanup(primary);
+free_primary:
+       kfree(primary);
+exit:
+       return ret;
+}
+
+int via_crtc_init(struct drm_device *dev, uint32_t index)
+{
+       struct openchrome_drm_private *dev_private =
+                                               dev->dev_private;
+       struct via_crtc *iga = &dev_private->iga[index];
+       struct drm_crtc *crtc = &iga->base;
+       u16 *gamma;
+       uint32_t i;
+       int ret = 0;
+
        if (iga->index) {
                iga->timings.htotal.count = ARRAY_SIZE(iga2_hor_total);
                iga->timings.htotal.regs = iga2_hor_total;
@@ -2311,15 +2333,5 @@ int via_crtc_init(struct drm_device *dev, uint32_t index)
                gamma[i + 512] = i << 8 | i;
        }
 
-       goto exit;
-cleanup_cursor:
-       drm_plane_cleanup(cursor);
-free_cursor:
-       kfree(cursor);
-cleanup_primary:
-       drm_plane_cleanup(primary);
-free_primary:
-       kfree(primary);
-exit:
        return ret;
 }
diff --git a/drivers/gpu/drm/openchrome/openchrome_display.c 
b/drivers/gpu/drm/openchrome/openchrome_display.c
index f4be439cb72f..e8482e9a207a 100644
--- a/drivers/gpu/drm/openchrome/openchrome_display.c
+++ b/drivers/gpu/drm/openchrome/openchrome_display.c
@@ -494,6 +494,11 @@ via_modeset_init(struct drm_device *dev)
        via_hwcursor_init(dev_private);
 
        for (i = 0; i < OPENCHROME_MAX_CRTC; i++) {
+               ret = openchrome_crtc_init(dev_private, i);
+               if (ret) {
+                       goto exit;
+               }
+
                ret = via_crtc_init(dev, i);
                if (ret) {
                        goto exit;
diff --git a/drivers/gpu/drm/openchrome/openchrome_drv.h 
b/drivers/gpu/drm/openchrome/openchrome_drv.h
index d27469d32084..922b6ca964f9 100644
--- a/drivers/gpu/drm/openchrome/openchrome_drv.h
+++ b/drivers/gpu/drm/openchrome/openchrome_drv.h
@@ -466,6 +466,8 @@ extern void via_fbdev_fini(struct drm_device *dev);
 /* crtc */
 extern void via_load_crtc_pixel_timing(struct drm_crtc *crtc,
                                        struct drm_display_mode *mode);
+int openchrome_crtc_init(struct openchrome_drm_private *dev_private,
+                               uint32_t index);
 extern int via_crtc_init(struct drm_device *dev, uint32_t index);
 
 /* encoders */
commit cb738635fe9759681af125856609850259d0e682
Author: Kevin Brace <kevinbr...@gmx.com>
Date:   Sat Oct 19 10:37:43 2019 -0700

    drm/openchrome: Invoke drm_crtc_helper_add() only once
    
    Consolidate two separate drm_crtc_helper_funcs structs into one.
    
    Signed-off-by: Kevin Brace <kevinbr...@gmx.com>

diff --git a/drivers/gpu/drm/openchrome/openchrome_crtc.c 
b/drivers/gpu/drm/openchrome/openchrome_crtc.c
index f8b5562ef1a4..c31ad8acb693 100644
--- a/drivers/gpu/drm/openchrome/openchrome_crtc.c
+++ b/drivers/gpu/drm/openchrome/openchrome_crtc.c
@@ -2055,18 +2055,7 @@ exit:
 }
 
 static const struct
-drm_crtc_helper_funcs openchrome_iga1_drm_crtc_helper_funcs = {
-       .dpms = openchrome_crtc_dpms,
-       .disable = openchrome_crtc_disable,
-       .prepare = openchrome_crtc_prepare,
-       .commit = openchrome_crtc_commit,
-       .mode_fixup = openchrome_crtc_mode_fixup,
-       .mode_set = openchrome_crtc_mode_set,
-       .mode_set_base_atomic = openchrome_crtc_mode_set_base_atomic,
-};
-
-static const struct
-drm_crtc_helper_funcs openchrome_iga2_drm_crtc_helper_funcs = {
+drm_crtc_helper_funcs openchrome_drm_crtc_helper_funcs = {
        .dpms = openchrome_crtc_dpms,
        .disable = openchrome_crtc_disable,
        .prepare = openchrome_crtc_prepare,
@@ -2137,14 +2126,8 @@ int via_crtc_init(struct drm_device *dev, uint32_t index)
                goto free_cursor;
        }
 
-       if (iga->index) {
-               drm_crtc_helper_add(crtc,
-                       &openchrome_iga2_drm_crtc_helper_funcs);
-       } else {
-               drm_crtc_helper_add(crtc,
-                       &openchrome_iga1_drm_crtc_helper_funcs);
-       }
-
+       drm_crtc_helper_add(crtc,
+                       &openchrome_drm_crtc_helper_funcs);
        ret = drm_crtc_init_with_planes(dev, crtc, primary, cursor,
                                        &openchrome_drm_crtc_funcs,
                                        NULL);
commit 3930b52d6a99b9fa6ddd6715e6ba5fcad8a713bd
Author: Kevin Brace <kevinbr...@gmx.com>
Date:   Sat Oct 19 10:11:38 2019 -0700

    drm/openchrome: Consolidate CRTC dpms code
    
    It is being merged into openchrome_crtc_dpms().
    
    Signed-off-by: Kevin Brace <kevinbr...@gmx.com>

diff --git a/drivers/gpu/drm/openchrome/openchrome_crtc.c 
b/drivers/gpu/drm/openchrome/openchrome_crtc.c
index 0b0a80538a88..f8b5562ef1a4 100644
--- a/drivers/gpu/drm/openchrome/openchrome_crtc.c
+++ b/drivers/gpu/drm/openchrome/openchrome_crtc.c
@@ -1582,6 +1582,63 @@ drm_mode_crtc_load_lut(struct drm_crtc *crtc)
        }
 }
 
+static void openchrome_crtc_dpms(struct drm_crtc *crtc, int mode)
+{
+       struct openchrome_drm_private *dev_private =
+                                               crtc->dev->dev_private;
+       struct via_crtc *iga = container_of(crtc,
+                                               struct via_crtc, base);
+
+       DRM_DEBUG_KMS("Entered %s.\n", __func__);
+
+       if (!iga->index) {
+               switch (mode) {
+               case DRM_MODE_DPMS_SUSPEND:
+               case DRM_MODE_DPMS_STANDBY:
+               case DRM_MODE_DPMS_OFF:
+                       /* turn off CRT screen (IGA1) */
+                       svga_wseq_mask(VGABASE, 0x01, BIT(5), BIT(5));
+
+                       /* clear for TV clock */
+                       svga_wcrt_mask(VGABASE, 0x6C, 0x00, 0xF0);
+                       break;
+
+               case DRM_MODE_DPMS_ON:
+                       /* turn on CRT screen (IGA1) */
+                       svga_wseq_mask(VGABASE, 0x01, 0x00, BIT(5));
+
+                       /* disable simultaneous  */
+                       svga_wcrt_mask(VGABASE, 0x6B, 0x00, BIT(3));
+                       drm_mode_crtc_load_lut(crtc);
+                       break;
+               }
+
+       } else {
+               switch (mode) {
+               case DRM_MODE_DPMS_SUSPEND:
+               case DRM_MODE_DPMS_STANDBY:
+               case DRM_MODE_DPMS_OFF:
+                       /* turn off CRT screen (IGA2) */
+                       svga_wcrt_mask(VGABASE, 0x6B, BIT(2), BIT(2));
+
+                       /* clear for TV clock */
+                       svga_wcrt_mask(VGABASE, 0x6C, 0x00, 0x0F);
+                       break;
+
+               case DRM_MODE_DPMS_ON:
+                       /* turn on CRT screen (IGA2) */
+                       svga_wcrt_mask(VGABASE, 0x6B, 0x00, BIT(2));
+
+                       /* disable simultaneous  */
+                       svga_wcrt_mask(VGABASE, 0x6B, 0x00, BIT(3));
+                       drm_mode_crtc_load_lut(crtc);
+                       break;
+               }
+       }
+
+       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
+}
+
 static void openchrome_crtc_disable(struct drm_crtc *crtc)
 {
        DRM_DEBUG_KMS("Entered %s.\n", __func__);
@@ -1997,73 +2054,9 @@ exit:
        return ret;
 }
 
-static void
-via_iga1_crtc_dpms(struct drm_crtc *crtc, int mode)
-{
-       struct openchrome_drm_private *dev_private =
-                                               crtc->dev->dev_private;
-
-       DRM_DEBUG_KMS("Entered %s.\n", __func__);
-
-       switch (mode) {
-       case DRM_MODE_DPMS_SUSPEND:
-       case DRM_MODE_DPMS_STANDBY:
-       case DRM_MODE_DPMS_OFF:
-               /* turn off CRT screen (IGA1) */
-               svga_wseq_mask(VGABASE, 0x01, BIT(5), BIT(5));
-
-               /* clear for TV clock */
-               svga_wcrt_mask(VGABASE, 0x6C, 0x00, 0xF0);
-               break;
-
-       case DRM_MODE_DPMS_ON:
-               /* turn on CRT screen (IGA1) */
-               svga_wseq_mask(VGABASE, 0x01, 0x00, BIT(5));
-
-               /* disable simultaneous  */
-               svga_wcrt_mask(VGABASE, 0x6B, 0x00, BIT(3));
-               drm_mode_crtc_load_lut(crtc);
-               break;
-       }
-
-       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
-}
-
-static void
-via_iga2_crtc_dpms(struct drm_crtc *crtc, int mode)
-{
-       struct openchrome_drm_private *dev_private =
-                                               crtc->dev->dev_private;
-
-       DRM_DEBUG_KMS("Entered %s.\n", __func__);
-
-       switch (mode) {
-       case DRM_MODE_DPMS_SUSPEND:
-       case DRM_MODE_DPMS_STANDBY:
-       case DRM_MODE_DPMS_OFF:
-               /* turn off CRT screen (IGA2) */
-               svga_wcrt_mask(VGABASE, 0x6B, BIT(2), BIT(2));
-
-               /* clear for TV clock */
-               svga_wcrt_mask(VGABASE, 0x6C, 0x00, 0x0F);
-               break;
-
-       case DRM_MODE_DPMS_ON:
-               /* turn on CRT screen (IGA2) */
-               svga_wcrt_mask(VGABASE, 0x6B, 0x00, BIT(2));
-
-               /* disable simultaneous  */
-               svga_wcrt_mask(VGABASE, 0x6B, 0x00, BIT(3));
-               drm_mode_crtc_load_lut(crtc);
-               break;
-       }
-
-       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
-}
-
 static const struct
 drm_crtc_helper_funcs openchrome_iga1_drm_crtc_helper_funcs = {
-       .dpms = via_iga1_crtc_dpms,
+       .dpms = openchrome_crtc_dpms,
        .disable = openchrome_crtc_disable,
        .prepare = openchrome_crtc_prepare,
        .commit = openchrome_crtc_commit,
@@ -2074,7 +2067,7 @@ drm_crtc_helper_funcs 
openchrome_iga1_drm_crtc_helper_funcs = {
 
 static const struct
 drm_crtc_helper_funcs openchrome_iga2_drm_crtc_helper_funcs = {
-       .dpms = via_iga2_crtc_dpms,
+       .dpms = openchrome_crtc_dpms,
        .disable = openchrome_crtc_disable,
        .prepare = openchrome_crtc_prepare,
        .commit = openchrome_crtc_commit,
commit 999f82852e6c643c9f81f595520b562c64a4a52b
Author: Kevin Brace <kevinbr...@gmx.com>
Date:   Sat Oct 19 09:59:07 2019 -0700

    drm/openchrome: Consolidate CRTC disable code
    
    It is being merged into openchrome_crtc_disable().
    
    Signed-off-by: Kevin Brace <kevinbr...@gmx.com>

diff --git a/drivers/gpu/drm/openchrome/openchrome_crtc.c 
b/drivers/gpu/drm/openchrome/openchrome_crtc.c
index cafd5db5ea5c..0b0a80538a88 100644
--- a/drivers/gpu/drm/openchrome/openchrome_crtc.c
+++ b/drivers/gpu/drm/openchrome/openchrome_crtc.c
@@ -1582,6 +1582,15 @@ drm_mode_crtc_load_lut(struct drm_crtc *crtc)
        }
 }
 
+static void openchrome_crtc_disable(struct drm_crtc *crtc)
+{
+       DRM_DEBUG_KMS("Entered %s.\n", __func__);
+
+       crtc->helper_private->dpms(crtc, DRM_MODE_DPMS_OFF);
+
+       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
+}
+
 static void openchrome_crtc_prepare(struct drm_crtc *crtc)
 {
        DRM_DEBUG_KMS("Entered %s.\n", __func__);
@@ -2021,16 +2030,6 @@ via_iga1_crtc_dpms(struct drm_crtc *crtc, int mode)
 }
 
 static void
-via_iga1_crtc_disable(struct drm_crtc *crtc)
-{
-       DRM_DEBUG_KMS("Entered %s.\n", __func__);
-
-       via_iga1_crtc_dpms(crtc, DRM_MODE_DPMS_OFF);
-
-       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
-}
-
-static void
 via_iga2_crtc_dpms(struct drm_crtc *crtc, int mode)
 {
        struct openchrome_drm_private *dev_private =
@@ -2062,20 +2061,10 @@ via_iga2_crtc_dpms(struct drm_crtc *crtc, int mode)
        DRM_DEBUG_KMS("Exiting %s.\n", __func__);
 }
 
-static void
-via_iga2_crtc_disable(struct drm_crtc *crtc)
-{
-       DRM_DEBUG_KMS("Entered %s.\n", __func__);
-
-       via_iga2_crtc_dpms(crtc, DRM_MODE_DPMS_OFF);
-
-       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
-}
-
 static const struct
 drm_crtc_helper_funcs openchrome_iga1_drm_crtc_helper_funcs = {
        .dpms = via_iga1_crtc_dpms,
-       .disable = via_iga1_crtc_disable,
+       .disable = openchrome_crtc_disable,
        .prepare = openchrome_crtc_prepare,
        .commit = openchrome_crtc_commit,
        .mode_fixup = openchrome_crtc_mode_fixup,
@@ -2086,7 +2075,7 @@ drm_crtc_helper_funcs 
openchrome_iga1_drm_crtc_helper_funcs = {
 static const struct
 drm_crtc_helper_funcs openchrome_iga2_drm_crtc_helper_funcs = {
        .dpms = via_iga2_crtc_dpms,
-       .disable = via_iga2_crtc_disable,
+       .disable = openchrome_crtc_disable,
        .prepare = openchrome_crtc_prepare,
        .commit = openchrome_crtc_commit,
        .mode_fixup = openchrome_crtc_mode_fixup,
commit e0c5c1608d301faf60db1609ef9843963edd9aaa
Author: Kevin Brace <kevinbr...@gmx.com>
Date:   Sat Oct 19 09:40:01 2019 -0700

    drm/openchrome: Consolidate CRTC prepare code
    
    It is being merged into openchrome_crtc_prepare().
    
    Signed-off-by: Kevin Brace <kevinbr...@gmx.com>

diff --git a/drivers/gpu/drm/openchrome/openchrome_crtc.c 
b/drivers/gpu/drm/openchrome/openchrome_crtc.c
index 0c8b71b514c4..cafd5db5ea5c 100644
--- a/drivers/gpu/drm/openchrome/openchrome_crtc.c
+++ b/drivers/gpu/drm/openchrome/openchrome_crtc.c
@@ -1582,6 +1582,17 @@ drm_mode_crtc_load_lut(struct drm_crtc *crtc)
        }
 }
 
+static void openchrome_crtc_prepare(struct drm_crtc *crtc)
+{
+       DRM_DEBUG_KMS("Entered %s.\n", __func__);
+
+       /* Blank the screen */
+       if (crtc->enabled)
+               crtc->helper_private->dpms(crtc, DRM_MODE_DPMS_OFF);
+
+       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
+}
+
 static void openchrome_crtc_commit(struct drm_crtc *crtc)
 {
        DRM_DEBUG_KMS("Entered %s.\n", __func__);
@@ -2020,18 +2031,6 @@ via_iga1_crtc_disable(struct drm_crtc *crtc)
 }
 
 static void
-via_iga1_crtc_prepare(struct drm_crtc *crtc)
-{
-       DRM_DEBUG_KMS("Entered %s.\n", __func__);
-
-       /* Blank the screen */
-       if (crtc->enabled)
-               via_iga1_crtc_dpms(crtc, DRM_MODE_DPMS_OFF);
-
-       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
-}
-
-static void
 via_iga2_crtc_dpms(struct drm_crtc *crtc, int mode)
 {
        struct openchrome_drm_private *dev_private =
@@ -2073,23 +2072,11 @@ via_iga2_crtc_disable(struct drm_crtc *crtc)
        DRM_DEBUG_KMS("Exiting %s.\n", __func__);
 }
 
-static void
-via_iga2_crtc_prepare(struct drm_crtc *crtc)
-{
-       DRM_DEBUG_KMS("Entered %s.\n", __func__);
-
-       /* Blank the screen */
-       if (crtc->enabled)
-               via_iga2_crtc_dpms(crtc, DRM_MODE_DPMS_OFF);
-
-       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
-}
-
 static const struct
 drm_crtc_helper_funcs openchrome_iga1_drm_crtc_helper_funcs = {
        .dpms = via_iga1_crtc_dpms,
        .disable = via_iga1_crtc_disable,
-       .prepare = via_iga1_crtc_prepare,
+       .prepare = openchrome_crtc_prepare,
        .commit = openchrome_crtc_commit,
        .mode_fixup = openchrome_crtc_mode_fixup,
        .mode_set = openchrome_crtc_mode_set,
@@ -2100,7 +2087,7 @@ static const struct
 drm_crtc_helper_funcs openchrome_iga2_drm_crtc_helper_funcs = {
        .dpms = via_iga2_crtc_dpms,
        .disable = via_iga2_crtc_disable,
-       .prepare = via_iga2_crtc_prepare,
+       .prepare = openchrome_crtc_prepare,
        .commit = openchrome_crtc_commit,
        .mode_fixup = openchrome_crtc_mode_fixup,
        .mode_set = openchrome_crtc_mode_set,
commit c56426d6a47633135fb947d7d230a26f01a6ab57
Author: Kevin Brace <kevinbr...@gmx.com>
Date:   Sat Oct 19 09:33:45 2019 -0700

    drm/openchrome: Consolidate CRTC commit code
    
    It is being merged into openchrome_crtc_commit().
    
    Signed-off-by: Kevin Brace <kevinbr...@gmx.com>

diff --git a/drivers/gpu/drm/openchrome/openchrome_crtc.c 
b/drivers/gpu/drm/openchrome/openchrome_crtc.c
index e3f8de6b4471..0c8b71b514c4 100644
--- a/drivers/gpu/drm/openchrome/openchrome_crtc.c
+++ b/drivers/gpu/drm/openchrome/openchrome_crtc.c
@@ -1582,6 +1582,17 @@ drm_mode_crtc_load_lut(struct drm_crtc *crtc)
        }
 }
 
+static void openchrome_crtc_commit(struct drm_crtc *crtc)
+{
+       DRM_DEBUG_KMS("Entered %s.\n", __func__);
+
+       /* Turn on the monitor */
+       if (crtc->enabled)
+               crtc->helper_private->dpms(crtc, DRM_MODE_DPMS_ON);
+
+       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
+}
+
 static bool openchrome_crtc_mode_fixup(struct drm_crtc *crtc,
                                const struct drm_display_mode *mode,
                                struct drm_display_mode *adjusted_mode)
@@ -2021,18 +2032,6 @@ via_iga1_crtc_prepare(struct drm_crtc *crtc)
 }
 
 static void
-via_iga1_crtc_commit(struct drm_crtc *crtc)
-{
-       DRM_DEBUG_KMS("Entered %s.\n", __func__);
-
-       /* Turn on the monitor */
-       if (crtc->enabled)
-               via_iga1_crtc_dpms(crtc, DRM_MODE_DPMS_ON);
-
-       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
-}
-
-static void
 via_iga2_crtc_dpms(struct drm_crtc *crtc, int mode)
 {
        struct openchrome_drm_private *dev_private =
@@ -2086,24 +2085,12 @@ via_iga2_crtc_prepare(struct drm_crtc *crtc)
        DRM_DEBUG_KMS("Exiting %s.\n", __func__);
 }
 
-static void
-via_iga2_crtc_commit(struct drm_crtc *crtc)
-{
-       DRM_DEBUG_KMS("Entered %s.\n", __func__);
-
-       /* Turn on the monitor */
-       if (crtc->enabled)
-               via_iga2_crtc_dpms(crtc, DRM_MODE_DPMS_ON);
-
-       DRM_DEBUG_KMS("Exiting %s.\n", __func__);
-}
-
 static const struct
 drm_crtc_helper_funcs openchrome_iga1_drm_crtc_helper_funcs = {
        .dpms = via_iga1_crtc_dpms,
        .disable = via_iga1_crtc_disable,
        .prepare = via_iga1_crtc_prepare,
-       .commit = via_iga1_crtc_commit,
+       .commit = openchrome_crtc_commit,
        .mode_fixup = openchrome_crtc_mode_fixup,
        .mode_set = openchrome_crtc_mode_set,
        .mode_set_base_atomic = openchrome_crtc_mode_set_base_atomic,
@@ -2114,7 +2101,7 @@ drm_crtc_helper_funcs 
openchrome_iga2_drm_crtc_helper_funcs = {
        .dpms = via_iga2_crtc_dpms,
        .disable = via_iga2_crtc_disable,
        .prepare = via_iga2_crtc_prepare,
-       .commit = via_iga2_crtc_commit,
+       .commit = openchrome_crtc_commit,
        .mode_fixup = openchrome_crtc_mode_fixup,
        .mode_set = openchrome_crtc_mode_set,
        .mode_set_base_atomic = openchrome_crtc_mode_set_base_atomic,
_______________________________________________
openchrome-devel mailing list
openchrome-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/openchrome-devel

Reply via email to