RE: [PATCH] drm/i915/backlight: Honor VESA eDP backlight luminance control capability

[Kandpal, Suraj]

> Subject: RE: [PATCH] drm/i915/backlight: Honor VESA eDP backlight luminance
> control capability
> 
> > Subject: [PATCH] drm/i915/backlight: Honor VESA eDP backlight
> > luminance control capability
> >
> > The VESA AUX backlight failed to be enabled becaused luminance_set is
> > false always.

Fix the typo on 'because' here. Also fix the grammar before merging this
to something like "AUX backlight fails to enable because ..." 

On an unrelated note @Nikula, Jani the commits that this patch sits on exists in
Drm-misc-next but not drm-intel-next. These said commits were merged I think a 
month or two back.
Now do we proceed merging these in drm-misc-next or get a pull of them in 
drm-intel-next

Regards,
Suraj Kandpal

> > Fix it by using luminance support control capabitliy.
> >
> > Fixes: e13af5166a359 ("drm/i915/backlight: Use drm helper to
> > initialize edp
> > backlight")
> > Signed-off-by: Aaron Ma <aaron...@canonical.com>
> 
> LGTM,
> Reviewed-by: Suraj Kandpal <suraj.kand...@intel.com>
> 
> > ---
> >  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 41228478b21c7..0a3a3f6a5f9d8 100644
> > --- a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
> > +++ b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
> > @@ -546,7 +546,7 @@ static int
> > intel_dp_aux_vesa_setup_backlight(struct
> > intel_connector *connector,
> >                                  luminance_range->max_luminance,
> >                                  panel->vbt.backlight.pwm_freq_hz,
> >                                  intel_dp->edp_dpcd, &current_level,
> &current_mode,
> > -                                false);
> > +                                panel-
> > >backlight.edp.vesa.luminance_control_support);
> >     if (ret < 0)
> >             return ret;
> >
> > --
> > 2.43.0