This is useful when there is a bridge between the SoC and the
panel.

Signed-off-by: Jitao Shi <jitao....@mediatek.com>
---
 drivers/gpu/drm/panel/panel-innolux-p079zca.c | 39 ++++++++++++-------
 1 file changed, 24 insertions(+), 15 deletions(-)

diff --git a/drivers/gpu/drm/panel/panel-innolux-p079zca.c 
b/drivers/gpu/drm/panel/panel-innolux-p079zca.c
index d92d1c98878c..8db404fb5eeb 100644
--- a/drivers/gpu/drm/panel/panel-innolux-p079zca.c
+++ b/drivers/gpu/drm/panel/panel-innolux-p079zca.c
@@ -54,7 +54,7 @@ struct innolux_panel {
 
        struct backlight_device *backlight;
        struct regulator_bulk_data *supplies;
-       struct gpio_desc *enable_gpio;
+       struct gpio_desc *enable_gpio[2];
 
        bool prepared;
        bool enabled;
@@ -82,7 +82,7 @@ static int innolux_panel_disable(struct drm_panel *panel)
 static int innolux_panel_unprepare(struct drm_panel *panel)
 {
        struct innolux_panel *innolux = to_innolux_panel(panel);
-       int err;
+       int err, i;
 
        if (!innolux->prepared)
                return 0;
@@ -102,7 +102,8 @@ static int innolux_panel_unprepare(struct drm_panel *panel)
        if (innolux->desc->sleep_mode_delay)
                msleep(innolux->desc->sleep_mode_delay);
 
-       gpiod_set_value_cansleep(innolux->enable_gpio, 0);
+       for (i = 0; i < ARRAY_SIZE(innolux->enable_gpio); i++)
+               gpiod_set_value_cansleep(innolux->enable_gpio[i], 0);
 
        if (innolux->desc->power_down_delay)
                msleep(innolux->desc->power_down_delay);
@@ -120,22 +121,27 @@ static int innolux_panel_unprepare(struct drm_panel 
*panel)
 static int innolux_panel_prepare(struct drm_panel *panel)
 {
        struct innolux_panel *innolux = to_innolux_panel(panel);
-       int err;
+       int err, i;
 
        if (innolux->prepared)
                return 0;
 
-       gpiod_set_value_cansleep(innolux->enable_gpio, 0);
+       for (i = 0; i < ARRAY_SIZE(innolux->enable_gpio); i++)
+               gpiod_set_value_cansleep(innolux->enable_gpio[i], 0);
 
        err = regulator_bulk_enable(innolux->desc->num_supplies,
                                    innolux->supplies);
        if (err < 0)
                return err;
 
-       /* p079zca: t2 (20ms), p097pfg: t4 (15ms) */
-       usleep_range(20000, 21000);
+       for (i = 0; i < ARRAY_SIZE(innolux->enable_gpio); i++) {
+               if (!innolux->enable_gpio[i])
+                       break;
 
-       gpiod_set_value_cansleep(innolux->enable_gpio, 1);
+               /* p079zca: t2 (20ms), p097pfg: t4 (15ms) */
+               usleep_range(20000, 21000);
+               gpiod_set_value_cansleep(innolux->enable_gpio[i], 1);
+       }
 
        /* p079zca: t4, p097pfg: t5 */
        usleep_range(20000, 21000);
@@ -195,7 +201,8 @@ static int innolux_panel_prepare(struct drm_panel *panel)
        return 0;
 
 poweroff:
-       gpiod_set_value_cansleep(innolux->enable_gpio, 0);
+       for (i = 0; i < ARRAY_SIZE(innolux->enable_gpio); i++)
+               gpiod_set_value_cansleep(innolux->enable_gpio[i], 0);
        regulator_bulk_disable(innolux->desc->num_supplies, innolux->supplies);
 
        return err;
@@ -475,12 +482,14 @@ static int innolux_panel_add(struct mipi_dsi_device *dsi,
        if (err < 0)
                return err;
 
-       innolux->enable_gpio = devm_gpiod_get_optional(dev, "enable",
-                                                      GPIOD_OUT_HIGH);
-       if (IS_ERR(innolux->enable_gpio)) {
-               err = PTR_ERR(innolux->enable_gpio);
-               dev_dbg(dev, "failed to get enable gpio: %d\n", err);
-               innolux->enable_gpio = NULL;
+       for (i = 0; i < ARRAY_SIZE(innolux->enable_gpio); i++) {
+               innolux->enable_gpio[i] = devm_gpiod_get_index_optional(dev,
+                                               "enable", i, GPIOD_OUT_HIGH);
+               if (IS_ERR(innolux->enable_gpio[i])) {
+                       err = PTR_ERR(innolux->enable_gpio[i]);
+                       dev_err(dev, "failed to get enable gpio: %d\n", err);
+                       innolux->enable_gpio[i] = NULL;
+               }
        }
 
        innolux->backlight = devm_of_find_backlight(dev);
-- 
2.21.0

Reply via email to