On Fri, Nov 22, 2019 at 03:46:02PM +0200, Kai Vehmanen wrote: > Starting with gen12, PORT_A can be connected to a transcoder > with audio support. Modify the existing logic that disabled > audio on PORT_A unconditionally. > > Signed-off-by: Kai Vehmanen <kai.vehmanen@xxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_dp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > index 0a023f3e633b..c50cf809b44c 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -2341,7 +2341,7 @@ intel_dp_compute_config(struct intel_encoder *encoder, > return ret; > > pipe_config->has_drrs = false; > - if (IS_G4X(dev_priv) || port == PORT_A) > + if (IS_G4X(dev_priv) || (INTEL_GEN(dev_priv) < 12 && port == PORT_A)) Getting a bit messy. Hoovering that into something like static bool intel_dp_port_has_audio(struct intel_encoder *encoder); or static bool intel_dp_port_has_audio(struct drm_i915_private *dev_priv, enum port port); might be nice. > pipe_config->has_audio = false; > else if (intel_conn_state->force_audio == HDMI_AUDIO_AUTO) > pipe_config->has_audio = intel_dp->has_audio; > -- > 2.17.1 -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
