[Intel-gfx] [PATCH] drm/i915: Enable DP audio for Haswell

[Paulo Zanoni]

Hi

drivers/gpu/drm/i915/intel_ddi.c: In function ‘intel_ddi_mode_set’:
drivers/gpu/drm/i915/intel_ddi.c:672:7: error: ‘intel_hdmi’ undeclared
(first use in this function)
drivers/gpu/drm/i915/intel_ddi.c:672:7: note: each undeclared
identifier is reported only once for each function it appears in
make[4]: ** [drivers/gpu/drm/i915/intel_ddi.o] Erro 1


2012/11/16 Takashi Iwai <tiwai at suse.de>:
> This patch adds the missing code to send ELD for Haswell DisplayPort,
> based on Xingchao's original patch.
>
> A test was performed with HSW-D machine and NEC EA232Wmi DP monitor.
>
> Cc: Xingchao Wang <xingchao.wang at intel.com>
> Signed-off-by: Takashi Iwai <tiwai at suse.de>
> ---
>  drivers/gpu/drm/i915/intel_ddi.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c
> index 58f50eb..d8dbd99 100644
> --- a/drivers/gpu/drm/i915/intel_ddi.c
> +++ b/drivers/gpu/drm/i915/intel_ddi.c
> @@ -669,6 +669,15 @@ static void intel_ddi_mode_set(struct drm_encoder *encoder,
>                         break;
>                 }
>
> +               if (intel_hdmi->has_audio) {

Replacing it with intel_dp->has_audio makes it look correct.

You could also try to "unify" the 2 "has_audio" checks of this
function into one single check at the end of the function.

> +                       DRM_DEBUG_DRIVER("DP audio on pipe %c on DDI\n",
> +                                        pipe_name(intel_crtc->pipe));
> +
> +                       /* write eld */
> +                       DRM_DEBUG_DRIVER("DP audio: write eld information\n");
> +                       intel_write_eld(encoder, adjusted_mode);
> +               }
> +
>                 intel_dp_init_link_config(intel_dp);
>
>         } else if (type == INTEL_OUTPUT_HDMI) {
> --
> 1.8.0
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Paulo Zanoni