On 20-02-2026 10:32, Suraj Kandpal wrote:
With luminance_set which represents PANEL_LUMINANCE_OVERRIDE, we
have another variable other than aux_enable to decide if we use
"other than" is redundant
Leads to a new thinking that is there another variable other than the two mentioned above.
PWM or DPCD. Make drm_dbg_kms log represent that.

Signed-off-by: Suraj Kandpal <[email protected]>
---
With the above change

Reviewed-by: Arun R Murthy <[email protected]>

Thanks and Regards,
Arun R Murthy
-------------------

  drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c | 3 ++-
  1 file changed, 2 insertions(+), 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 c7143869bafd..043c9aef2ea6 100644
--- a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
+++ b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
@@ -539,7 +539,8 @@ static int intel_dp_aux_vesa_setup_backlight(struct 
intel_connector *connector,
        drm_dbg_kms(display->drm,
                    "[CONNECTOR:%d:%s] AUX VESA backlight enable is controlled 
through %s\n",
                    connector->base.base.id, connector->base.name,
-                   dpcd_vs_pwm_str(panel->backlight.edp.vesa.info.aux_enable));
+                   dpcd_vs_pwm_str(panel->backlight.edp.vesa.info.aux_enable ||
+                                   
panel->backlight.edp.vesa.info.luminance_set));
        drm_dbg_kms(display->drm,
                    "[CONNECTOR:%d:%s] AUX VESA backlight level is controlled 
through %s\n",
                    connector->base.base.id, connector->base.name,

Reply via email to