diff mbox

[v5,7/9] drm/i915: Restore brightness level in aux backlight driver

Message ID 20170504002836.120988-8-puthik@chromium.org (mailing list archive)
State New, archived
Headers show

Commit Message

Puthikorn Voravootivat May 4, 2017, 12:28 a.m. UTC
Some panel will default to zero brightness when turning the
panel off and on again. This patch restores last brightness
level back when panel is turning back on.

Signed-off-by: Puthikorn Voravootivat <puthik@chromium.org>
---
 drivers/gpu/drm/i915/intel_dp_aux_backlight.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Dhinakaran Pandiyan May 8, 2017, 10:28 p.m. UTC | #1
On Wed, 2017-05-03 at 17:28 -0700, Puthikorn Voravootivat wrote:
> Some panel will default to zero brightness when turning the

> panel off and on again. This patch restores last brightness

> level back when panel is turning back on.

> 

> Signed-off-by: Puthikorn Voravootivat <puthik@chromium.org>


Looks correct and all other _enable_backlight() functions are doing it
too. 
Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>


> ---

>  drivers/gpu/drm/i915/intel_dp_aux_backlight.c | 1 +

>  1 file changed, 1 insertion(+)

> 

> diff --git a/drivers/gpu/drm/i915/intel_dp_aux_backlight.c b/drivers/gpu/drm/i915/intel_dp_aux_backlight.c

> index 7d323af96636..fc26fea94fd4 100644

> --- a/drivers/gpu/drm/i915/intel_dp_aux_backlight.c

> +++ b/drivers/gpu/drm/i915/intel_dp_aux_backlight.c

> @@ -158,6 +158,7 @@ static void intel_dp_aux_enable_backlight(struct intel_connector *connector)

>  	}

>  

>  	set_aux_backlight_enable(intel_dp, true);

> +	intel_dp_aux_set_backlight(connector, connector->panel.backlight.level);

>  }

>  

>  static void intel_dp_aux_disable_backlight(struct intel_connector *connector)
diff mbox

Patch

diff --git a/drivers/gpu/drm/i915/intel_dp_aux_backlight.c b/drivers/gpu/drm/i915/intel_dp_aux_backlight.c
index 7d323af96636..fc26fea94fd4 100644
--- a/drivers/gpu/drm/i915/intel_dp_aux_backlight.c
+++ b/drivers/gpu/drm/i915/intel_dp_aux_backlight.c
@@ -158,6 +158,7 @@  static void intel_dp_aux_enable_backlight(struct intel_connector *connector)
 	}
 
 	set_aux_backlight_enable(intel_dp, true);
+	intel_dp_aux_set_backlight(connector, connector->panel.backlight.level);
 }
 
 static void intel_dp_aux_disable_backlight(struct intel_connector *connector)