[Intel-gfx] [PATCH] drm/i915/guc: Removed unused GuC parameters.

[Chris Wilson]

Quoting Michel Thierry (2018-02-28 22:07:51)
> On 28/02/18 12:26, Michel Thierry wrote:
> > On 28/02/18 10:42, Piotr Piórkowski wrote:
> >> In the i915 driver, there is a function, intel_guc_init_params(),
> >> which initializes the GuC parameter block which is passed into
> >> the GuC. There is parameter GUC_CTL_DEVICE_INFO with values
> >> GfxGtType and GfxCoreFamily unused by GuC.
> >>
> >> This patch remove GUC_CTL_DEVICE_INFO with GfxGtType and
> >> GfxCoreFamily parameters and also unnecessary functions
> >> get_gt_type() and get_core_family().
> >>
> > 
> > Hi,
> > 
> > Looking at the fw code, you're partially right, GfxGtType is ignored... 
> > but GfxCoreFamily isn't.
> > 
> 
> Unless whoever wrote the fw was smart enough to forget to call the 
> function that is reading GfxCoreFamily... I didn't count on that.

Is the intention to use GfxCoreFamily documented, i.e. are they
expecting it part of the interface and may re-instantiate the check
"because it was always supposed to exist" in some future version?
-Chris