> -----Original Message-----
> From: Kandpal, Suraj <[email protected]>
> Sent: Monday, April 14, 2025 9:47 AM
> To: [email protected]; [email protected]; intel-
> [email protected]; [email protected]
> Cc: Nautiyal, Ankit K <[email protected]>; Murthy, Arun R
> <[email protected]>; Kandpal, Suraj <[email protected]>
> Subject: [PATCH 13/13] drm/i915/backlight: Use drm_edp_backlight_enable
>
> Use drm dp helper to enable backlight now that it has been modified to set
> PANEL_LUMINANCE_CONTROL_ENABLE bit based on if capability supports it
> and the driver wants it. Remove the dead code.
>
> Signed-off-by: Suraj Kandpal <[email protected]>
> ---
Reviewed-by: Arun R Murthy <[email protected]>
Thanks and Regards,
Arun R Murthy
--------------------
> .../gpu/drm/i915/display/intel_dp_aux_backlight.c | 14 --------------
> 1 file changed, 14 deletions(-)
>
> 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 95b29d9af335..b8b0ace9e6fd 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
> @@ -499,20 +499,6 @@ intel_dp_aux_vesa_enable_backlight(const struct
> intel_crtc_state *crtc_state,
> struct intel_connector *connector = to_intel_connector(conn_state-
> >connector);
> struct intel_panel *panel = &connector->panel;
> struct intel_dp *intel_dp = enc_to_intel_dp(connector->encoder);
> - int ret;
> -
> - if (panel->backlight.edp.vesa.luminance_control_support) {
> - ret = drm_dp_dpcd_writeb(&intel_dp->aux,
> DP_EDP_BACKLIGHT_MODE_SET_REGISTER,
> -
> DP_EDP_PANEL_LUMINANCE_CONTROL_ENABLE);
> -
> - if (ret == 1)
> - return;
> -
> - if (!drm_edp_backlight_set_level(&intel_dp->aux,
> - &panel-
> >backlight.edp.vesa.info,
> - level))
> - return;
> - }
>
> if (!panel->backlight.edp.vesa.info.aux_enable) {
> u32 pwm_level;
> --
> 2.34.1
