Re: [PATCH] drm/i915/cnl: Add missing type case.

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

 



On Tue, 2017-07-11 at 03:14 -0700, Dhinakaran Pandiyan wrote:
> On Monday, July 10, 2017 1:58:52 PM PDT Rodrigo Vivi wrote:
> > Paulo had noticed that inside cnl_ddi_vswing_program
> > the case was handling voltage but with no indication
> > of type where a missing type could also take us to that
> > path. So my first attempt was to add a message to
> > let clear who trigger that path.
> > 
> > However DK had a better idea that is to handle the
> > missed type sooner before it might take to that path.
> > So this v2 here uses his approach.
> > 
> > v2: Handle missed type sooner.
> > 
> > Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@xxxxxxxxx>
> > Cc: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> > ---
> >  drivers/gpu/drm/i915/intel_ddi.c | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/intel_ddi.c
> > b/drivers/gpu/drm/i915/intel_ddi.c index 241decf2a7d9..efb13582dc73 100644
> > --- a/drivers/gpu/drm/i915/intel_ddi.c
> > +++ b/drivers/gpu/drm/i915/intel_ddi.c
> > @@ -1979,9 +1979,12 @@ static void cnl_ddi_vswing_sequence(struct
> > intel_encoder *encoder, u32 level) if ((intel_dp) && (type ==
> > INTEL_OUTPUT_EDP || type == INTEL_OUTPUT_DP)) { width =
> > intel_dp->lane_count;
> >  		rate = intel_dp->link_rate;
> > -	} else {
> > +	} else if (type == INTEL_OUTPUT_HDMI) {
> >  		width = 4;
> >  		/* Rate is always < than 6GHz for HDMI */
> > +	} else {
> > +		MISSING_CASE(type);
> > +		return;
> >  	}
> > 
> >  	/*
> 
> The early return in this version avoids register writes for invalid encoder 
> types. So,
> Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@xxxxxxxxx>

merged to dinq, thanks for the suggestion and review!

> 

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux