#include <drm/drmP.h>
@@ -630,6 +631,10 @@ void intel_audio_codec_enable(struct intel_encoder *intel_encoder,
if (acomp && acomp->audio_ops && acomp->audio_ops->pin_eld_notify)
acomp->audio_ops->pin_eld_notify(acomp->audio_ops->audio_ptr,
(int) port, (int) pipe);
+
+ if (HAS_LPE_AUDIO(dev_priv))
+ intel_lpe_audio_notify(dev_priv, connector->eld, port,
+ crtc_state->port_clock);
Seems unnecessary to check for HAS_LPE_AUDIO (which you'll change to
dev_priv->lpe_audio.platdev, right ;) both in the caller and
callee. Pick one.
If we test inside of the function, it'd mean an unconditional jump to
test a feature that exists on only two platforms out of the dozen or so
that this i915 driver handles. No objection to do the change but is this
really desired?
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
