[PATCH 3/4] drm: panel: nt35560: Move DSI commands to enable/disable

[Linus Walleij]

Due to semantic changes in the bridge core, panels cannot send
any DSI commands in the prepare/unprepare callbacks: there is
no guarantee that the DSI transmitter is available at this
point.

Cc: Aradhya Bhatia <a-bhat...@ti.com>
Fixes: c9b1150a68d9 ("drm/atomic-helper: Re-order bridge chain pre-enable and 
post-disable")
Signed-off-by: Linus Walleij <linus.wall...@linaro.org>
---
 drivers/gpu/drm/panel/panel-novatek-nt35560.c | 24 +++++++++++++++++-------
 1 file changed, 17 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/panel/panel-novatek-nt35560.c 
b/drivers/gpu/drm/panel/panel-novatek-nt35560.c
index 561e6643dcbb..b0b11d3e26fe 100644
--- a/drivers/gpu/drm/panel/panel-novatek-nt35560.c
+++ b/drivers/gpu/drm/panel/panel-novatek-nt35560.c
@@ -278,16 +278,18 @@ static void nt35560_power_off(struct nt35560 *nt)
 }
 
 static int nt35560_prepare(struct drm_panel *panel)
+{
+       struct nt35560 *nt = panel_to_nt35560(panel);
+
+       return nt35560_power_on(nt);
+}
+
+static int nt35560_enable(struct drm_panel *panel)
 {
        struct nt35560 *nt = panel_to_nt35560(panel);
        struct mipi_dsi_multi_context dsi_ctx = {
                .dsi = to_mipi_dsi_device(nt->dev)
        };
-       int ret;
-
-       ret = nt35560_power_on(nt);
-       if (ret)
-               return ret;
 
        nt35560_read_id(&dsi_ctx);
 
@@ -317,7 +319,7 @@ static int nt35560_prepare(struct drm_panel *panel)
        return dsi_ctx.accum_err;
 }
 
-static int nt35560_unprepare(struct drm_panel *panel)
+static int nt35560_disable(struct drm_panel *panel)
 {
        struct nt35560 *nt = panel_to_nt35560(panel);
        struct mipi_dsi_multi_context dsi_ctx = {
@@ -332,12 +334,18 @@ static int nt35560_unprepare(struct drm_panel *panel)
 
        msleep(85);
 
+       return 0;
+}
+
+static int nt35560_unprepare(struct drm_panel *panel)
+{
+       struct nt35560 *nt = panel_to_nt35560(panel);
+
        nt35560_power_off(nt);
 
        return 0;
 }
 
-
 static int nt35560_get_modes(struct drm_panel *panel,
                             struct drm_connector *connector)
 {
@@ -369,6 +377,8 @@ static int nt35560_get_modes(struct drm_panel *panel,
 static const struct drm_panel_funcs nt35560_drm_funcs = {
        .unprepare = nt35560_unprepare,
        .prepare = nt35560_prepare,
+       .enable = nt35560_enable,
+       .disable = nt35560_disable,
        .get_modes = nt35560_get_modes,
 };
 

-- 
2.51.0