On Fri, Jan 17, 2020 at 05:13:25PM +0200, Ville Syrjälä wrote: > On Fri, Jan 17, 2020 at 05:12:38PM +0200, Ville Syrjälä wrote: > > On Fri, Jan 17, 2020 at 04:29:28PM +0200, Jani Nikula wrote: > > > Affects only two calls in output setup, and ddi init will check the > > > features in more fine grained way. > > > > > > This will make future changes easier. > > > > > > Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> > > > --- > > > drivers/gpu/drm/i915/display/intel_bios.c | 4 +--- > > > 1 file changed, 1 insertion(+), 3 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c > > > index 4c69253739ec..70fb87e7afb6 100644 > > > --- a/drivers/gpu/drm/i915/display/intel_bios.c > > > +++ b/drivers/gpu/drm/i915/display/intel_bios.c > > > @@ -2236,9 +2236,7 @@ bool intel_bios_is_port_present(struct drm_i915_private *dev_priv, enum port por > > > const struct ddi_vbt_port_info *port_info = > > > &dev_priv->vbt.ddi_port_info[port]; > > > > > > - return port_info->supports_dp || > > > - port_info->supports_dvi || > > > - port_info->supports_hdmi; > > > + return port_info->child; > > > > Pondering what happens if there's a non-DP/DVI/HDMI port declared in the > > VBT... I guess those should not have their dvo port set to anything we > > accept? > > Umm, no. We accept DVO_PORT_CRT as PORT_E. Maybe it doesn't matter though. Hopefully no VBT has that on skl+, and on hsw/bdw the current CRT init code doesn't seem to care what this says. > > > > > > } > > > > > > /* FIXME maybe deal with port A as well? */ > > > -- > > > 2.20.1 > > > > > > _______________________________________________ > > > Intel-gfx mailing list > > > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx > > > > -- > > Ville Syrjälä > > Intel > > -- > Ville Syrjälä > Intel -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
