On Tue, 25 Jun 2019, Jani Nikula <[email protected]> wrote: >On Tue, 25 Jun 2019, Lee Shawn C <[email protected]> wrote: >> Modify aux backlight enable/disable sequence just like what we did for >> genernal eDP panel. Setup PWM freq and brightness level before enable >> display backlight. >> >> v2: According to spec, it is not necessary to set PWM to 0 if >> backlight enable already off. Remove intel_dp_aux_set_backlight to 0. > >Please think it through again. The same applies here. The brightness change >before enable is ignored according to eDP spec. > >BR, >Jani. >
This is what the problem we see on a new eDP panel from customer. TCON did not turn backlight on successful. After sawp the sequence, backlight works well as expected. In the other words. Looks like this TCON backlight enable flow violate eDP spec. What do you think? >> >> Cc: Ville Syrjälä <[email protected]> >> Cc: Jani Nikula <[email protected]> >> Cc: Jose Roberto de Souza <[email protected]> >> Cc: Cooper Chiou <[email protected]> >> >> Signed-off-by: Lee Shawn C <[email protected]> >> --- >> drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c >> b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c >> index 6b0b73479fb8..70d9b9c9586f 100644 >> --- a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c >> +++ b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c >> @@ -216,8 +216,8 @@ static void intel_dp_aux_enable_backlight(const struct >> intel_crtc_state *crtc_st >> } >> } >> >> - set_aux_backlight_enable(intel_dp, true); >> intel_dp_aux_set_backlight(conn_state, >> connector->panel.backlight.level); >> + set_aux_backlight_enable(intel_dp, true); >> } >> >> static void intel_dp_aux_disable_backlight(const struct >> drm_connector_state *old_conn_state) > >-- >Jani Nikula, Intel Open Source Graphics Center > _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
