Re: [PATCH 00/12] Gen9: Changes to add/merge/update/remove WA

From: Daniel Vetter <daniel@ffwll.ch>
To: Arun Siluvery <arun.siluvery@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org, Mika Kuoppala <mika.kuoppala@intel.com>
Subject: Re: [PATCH 00/12] Gen9: Changes to add/merge/update/remove WA
Date: Mon, 28 Sep 2015 16:03:06 +0200	[thread overview]
Message-ID: <20150928140306.GU3383@phenom.ffwll.local> (raw)
In-Reply-To: <560922C0.80509@linux.intel.com>

On Mon, Sep 28, 2015 at 12:21:36PM +0100, Arun Siluvery wrote:
> On 25/09/2015 14:33, Arun Siluvery wrote:
> >Changes that add new WA, merge WA that are applied for the same register,
> >update stepping checks and remove pre-production ones .
> >
> >Arun Siluvery (12):
> >   drm/i915/gen9: Handle error returned by gen9_init_workarounds
> >   drm/i915/gen9: Add WaOCLCoherentLineFlush
> >   drm/i915/gen9: Merge two WA as they part of same register
> >   drm/i915/gen9: Merge HALF_SLICE_CHICKEN3 WA
> >   drm/i915/bxt: update WaSetHDCunitClckGatingDisable
> >   drm/i915/bxt: Add WaStoreMultiplePTEenable name
> >   drm/i915/bxt: Update stepping check for WaDisableSDEUnitClockGating
> >   drm/i915/skl: Remove WaDisableSDEUnitClockGating
> >   drm/i915/skl: Remove WaSetGAPSunitClckGateDisable
> >   drm/i915/skl: Remove WaDisableVFUnitClockGating
> >   drm/i915/skl: Remove
> >     WaSetDisablePixMaskCammingAndRhwoInCommonSliceChicken
> >   drm/i915:skl: Remove WaDisablePowerCompilerClockGating
> >
> >  drivers/gpu/drm/i915/i915_reg.h         |  2 --
> >  drivers/gpu/drm/i915/intel_lrc.c        |  5 ++-
> >  drivers/gpu/drm/i915/intel_pm.c         | 48 ++++++++++++--------------
> >  drivers/gpu/drm/i915/intel_ringbuffer.c | 61 ++++++++++++++++-----------------
> >  4 files changed, 52 insertions(+), 64 deletions(-)
> >
> 
> Hi Daniel,
> 
> Some of the patches in this series are reviewed but I am seeing some
> inconsistency with patch 9 (WaSetGAPSunitClckGateDisable) and in the process
> of getting it clarified. I will re-post all patches again with r-b tags for
> appropriate patches.

I just pulled in a bunch which were reviewed (including minor bikesheds to
keep #defines on 2 patches). Please don't resend those. But yes please
chase the bspec inconsistencies and please file bspec bugs where the docs
are wrong.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx