[Intel-gfx] [PATCH 4/7] drm/i915/icl: Add Voltage swing table for MG PHY DDI Buffer

Chris Wilson chris at chris-wilson.co.uk
Tue Apr 10 21:07:24 UTC 2018


Quoting Chris Wilson (2018-04-10 22:01:33)
> Quoting Paulo Zanoni (2018-04-10 21:39:31)
> > Em Ter, 2018-04-10 às 09:51 +0100, Chris Wilson escreveu:
> > > Quoting Paulo Zanoni (2018-03-23 17:24:16)
> > > > From: Manasi Navare <manasi.d.navare at intel.com>
> > > > 
> > > > This table is used for voltage swing programming sequence during
> > > > DDI
> > > > Buffer initialization for MG PHY DDI Buffers on Icelake.
> > > 
> > > Except it is not used at all...
> > 
> > It's going to be used later in the series.

In the context of this patch in merged series, it means that we have
added unexecuted, untested code. Please justify that. (Unused data
tables have in the past remain unused, leaving the code using old
tables; a bug that would have been caught by the compiler.)

We know our test coverage is lacking, this makes it worse. Yes, it's
icl, yes we won't even have sim coverage for the entire set of freq
tables, so what's stopping us from writing unittests to double check the
code at least conforms to our expectations and is exercised?
-Chris


More information about the Intel-gfx mailing list