Re: [PATCH 04/10] drm/i915: Pass the level to intel_prepare_hdmi_ddi_buffers()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 16, 2017 at 7:56 AM, Ville Syrjala
<ville.syrjala@xxxxxxxxxxxxxxx> wrote:
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>
> The caller of intel_prepare_hdmi_ddi_buffers() alreday figured out the
> level, so let's just pass it in instead if figuring it out again.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>

Reviewed-by: James Ausmus <james.ausmus@xxxxxxxxx>

> ---
>  drivers/gpu/drm/i915/intel_ddi.c | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c
> index cf0b2d3de15f..f61b6c20005e 100644
> --- a/drivers/gpu/drm/i915/intel_ddi.c
> +++ b/drivers/gpu/drm/i915/intel_ddi.c
> @@ -817,16 +817,15 @@ static void intel_prepare_dp_ddi_buffers(struct intel_encoder *encoder)
>   * values in advance. This function programs the correct values for
>   * HDMI/DVI use cases.
>   */
> -static void intel_prepare_hdmi_ddi_buffers(struct intel_encoder *encoder)
> +static void intel_prepare_hdmi_ddi_buffers(struct intel_encoder *encoder,
> +                                          int hdmi_level)
>  {
>         struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
>         u32 iboost_bit = 0;
> -       int n_hdmi_entries, hdmi_level;
> +       int n_hdmi_entries;
>         enum port port = intel_ddi_get_encoder_port(encoder);
>         const struct ddi_buf_trans *ddi_translations_hdmi;
>
> -       hdmi_level = intel_ddi_hdmi_level(dev_priv, port);
> -
>         ddi_translations_hdmi = intel_ddi_get_buf_trans_hdmi(dev_priv, &n_hdmi_entries);
>
>         /* If we're boosting the current, set bit 31 of trans1 */
> @@ -2203,7 +2202,7 @@ static void intel_ddi_pre_enable_hdmi(struct intel_encoder *encoder,
>                 bxt_ddi_vswing_sequence(dev_priv, level, port,
>                                         INTEL_OUTPUT_HDMI);
>         else
> -               intel_prepare_hdmi_ddi_buffers(encoder);
> +               intel_prepare_hdmi_ddi_buffers(encoder, level);
>
>         if (IS_GEN9_BC(dev_priv))
>                 skl_ddi_set_iboost(encoder, level, INTEL_OUTPUT_HDMI);
> --
> 2.13.6
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 


James Ausmus
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux