[Intel-gfx] [PATCH] [V3] drm/i915: Enable VBT based BL control for DP
Mustaffa, Mustamin B
mustamin.b.mustaffa at intel.com
Wed Feb 28 10:40:19 UTC 2018
Thanks Jani.
Best regard
Mustamin
-----Original Message-----
From: Jani Nikula [mailto:jani.nikula at linux.intel.com]
Sent: Wednesday, February 28, 2018 5:42 PM
To: Mustaffa, Mustamin B <mustamin.b.mustaffa at intel.com>; intel-gfx at lists.freedesktop.org
Cc: Mustaffa, Mustamin B <mustamin.b.mustaffa at intel.com>
Subject: Re: [Intel-gfx] [PATCH] [V3] drm/i915: Enable VBT based BL control for DP
On Tue, 27 Feb 2018, Mustamin B Mustaffa <mustamin.b.mustaffa at intel.com> wrote:
> Currently, BXT_PP is hardcoded with value '0'.
> It practically disabled eDP backlight on MRB (BXT) platform.
>
> This patch will tell which BXT_PP registers (there are two set of
> PP_CONTROL in the spec) to be used as defined in VBT (Video Bios
> Timing
> table) and this will enabled eDP backlight controller on MRB (BXT)
> platform.
>
> v2:
> - Remove unnecessary information in commit message.
> - Assign vbt.backlight.controller to a backlight_controller variable and
> return the variable value.
> v3:
> - Rebased to latest code base.
> - updated commit title.
>
> Signed-off-by: Mustamin B Mustaffa <mustamin.b.mustaffa at intel.com>
Pushed to dinq, thanks for the patch.
BR,
Jani.
> ---
> drivers/gpu/drm/i915/intel_dp.c | 11 ++++-------
> 1 file changed, 4 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_dp.c
> b/drivers/gpu/drm/i915/intel_dp.c index 475a19d76a49..20e3fcd2bf7e
> 100644
> --- a/drivers/gpu/drm/i915/intel_dp.c
> +++ b/drivers/gpu/drm/i915/intel_dp.c
> @@ -655,18 +655,15 @@ bxt_power_sequencer_idx(struct intel_dp
> *intel_dp) {
> struct drm_i915_private *dev_priv =
> to_i915(intel_dp_to_dev(intel_dp));
>
> + int backlight_controller = dev_priv->vbt.backlight.controller;
> +
> lockdep_assert_held(&dev_priv->pps_mutex);
>
> /* We should never land here with regular DP ports */
> WARN_ON(!intel_dp_is_edp(intel_dp));
>
> - /*
> - * TODO: BXT has 2 PPS instances. The correct port->PPS instance
> - * mapping needs to be retrieved from VBT, for now just hard-code to
> - * use instance #0 always.
> - */
> if (!intel_dp->pps_reset)
> - return 0;
> + return backlight_controller;
>
> intel_dp->pps_reset = false;
>
> @@ -676,7 +673,7 @@ bxt_power_sequencer_idx(struct intel_dp *intel_dp)
> */
> intel_dp_init_panel_power_sequencer_registers(intel_dp, false);
>
> - return 0;
> + return backlight_controller;
> }
>
> typedef bool (*vlv_pipe_check)(struct drm_i915_private *dev_priv,
--
Jani Nikula, Intel Open Source Technology Center
More information about the Intel-gfx
mailing list