drm/i915/dsi: Init panel-enable GPIO to low when the LCD is initially off (v2)
authorHans de Goede <hdegoede@redhat.com>
Mon, 16 Dec 2019 20:51:20 +0000 (21:51 +0100)
committerHans de Goede <hdegoede@redhat.com>
Fri, 3 Jan 2020 10:46:59 +0000 (11:46 +0100)
commit6c0a878ebaa43fc9d8d84ac4eacce9f6a9b974ac
tree99af3fc7667664beacb3e1e4dbb77f6bb241f63e
parentea0fe6725141788ec0bb5a1bcb68d136cb5c6970
drm/i915/dsi: Init panel-enable GPIO to low when the LCD is initially off (v2)

When the LCD has not been turned on by the firmware/GOP, because e.g. the
device was booted with an external monitor connected over HDMI, we should
not turn on the panel-enable GPIO when we request it.

Turning on the panel-enable GPIO when we request it, means we turn it on
too early in the init-sequence, which causes some panels to not correctly
light up.

This commits adds a panel_is_on parameter to intel_dsi_vbt_gpio_init()
and makes intel_dsi_vbt_gpio_init() set the initial GPIO value accordingly.

This fixes the panel not lighting up on a Thundersoft TST168 tablet when
booted with an external monitor connected over HDMI.

Changes in v2:
- Call intel_dsi_get_hw_state() to check if the panel is on instead of
  relying on the current_mode pointer

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20191216205122.1850923-4-hdegoede@redhat.com
drivers/gpu/drm/i915/display/intel_dsi.h
drivers/gpu/drm/i915/display/intel_dsi_vbt.c
drivers/gpu/drm/i915/display/vlv_dsi.c