Re: [PATCH 8/9] drm/i915/bios: check port presence based on child device

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

 



On Fri, 17 Jan 2020, Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> wrote:
> On Fri, Jan 17, 2020 at 05:28:23PM +0200, Jani Nikula wrote:
>> On Fri, 17 Jan 2020, Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> 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.
>> 
>> My argument is this:
>> 
>> - Shouldn't intel_bios_is_port_present(PORT_E) return true in that case?
>
> Only on hsw/bdw really.
>
>> 
>> - Where does the change make a functional difference in the
>>   intel_bios_is_port_present() users anyway? AFAICT all the cases will
>>   also check the supports_X flags anyway (or have checked before).
>
> After some trawling I can't see anything that should break from this,
> so probably fine. But the commit message should probably elaborate
> on this a bit more, just in case there is a regression.
>
> Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>

Thanks for all the reviews! Expanded on the commit message on this one,
and pushed the lot.

BR,
Jani.


>
>> 
>> 
>> BR,
>> Jani.
>> 
>> 
>> >
>> >> 
>> >> >  	}
>> >> >  
>> >> >  	/* 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
>> 
>> -- 
>> Jani Nikula, Intel Open Source Graphics Center

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux