From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rodrigo Vivi Subject: Re: [PATCH] drm/i915/bdw: Remove BDW preproduction W/As until C stepping. Date: Tue, 30 Sep 2014 15:14:48 -0700 Message-ID: References: <1411172190-1642-5-git-send-email-rodrigo.vivi@intel.com> <1411598766-3441-1-git-send-email-rodrigo.vivi@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wg0-f46.google.com (mail-wg0-f46.google.com [74.125.82.46]) by gabe.freedesktop.org (Postfix) with ESMTP id DEFD56E072 for ; Tue, 30 Sep 2014 15:14:49 -0700 (PDT) Received: by mail-wg0-f46.google.com with SMTP id k14so6114795wgh.17 for ; Tue, 30 Sep 2014 15:14:49 -0700 (PDT) In-Reply-To: <1411598766-3441-1-git-send-email-rodrigo.vivi@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Rodrigo Vivi , Mika Kuoppala Cc: intel-gfx List-Id: intel-gfx@lists.freedesktop.org Just a note that v2 here can be useful and applied even without the patch 4/5 which I asked to ignore for now. Mika, could you please help on review of this? On Wed, Sep 24, 2014 at 3:46 PM, Rodrigo Vivi wrote: > Let's clean this a bit > > v2: Rebase after other Mika's patch that removed some BDW production workarounds. > > Signed-off-by: Rodrigo Vivi > --- > drivers/gpu/drm/i915/intel_pm.c | 10 ---------- > drivers/gpu/drm/i915/intel_ringbuffer.c | 5 ++--- > 2 files changed, 2 insertions(+), 13 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c > index ae61b45..aaab056 100644 > --- a/drivers/gpu/drm/i915/intel_pm.c > +++ b/drivers/gpu/drm/i915/intel_pm.c > @@ -5803,16 +5803,6 @@ static void broadwell_init_clock_gating(struct drm_device *dev) > I915_WRITE(WM2_LP_ILK, 0); > I915_WRITE(WM1_LP_ILK, 0); > > - /* FIXME(BDW): Check all the w/a, some might only apply to > - * pre-production hw. */ > - > - > - I915_WRITE(GAMTARBMODE, _MASKED_BIT_ENABLE(ARB_MODE_BWGTLB_DISABLE)); > - > - I915_WRITE(_3D_CHICKEN3, > - _MASKED_BIT_ENABLE(_3D_CHICKEN_SDE_LIMIT_FIFO_POLY_DEPTH(2))); > - > - > /* WaSwitchSolVfFArbitrationPriority:bdw */ > I915_WRITE(GAM_ECOCHK, I915_READ(GAM_ECOCHK) | HSW_ECOCHK_ARB_PRIO_SOL); > > diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c > index 0a4fd37..896f564 100644 > --- a/drivers/gpu/drm/i915/intel_ringbuffer.c > +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c > @@ -714,13 +714,12 @@ static int bdw_init_workarounds(struct intel_engine_cs *ring) > return ret; > > /* WaDisablePartialInstShootdown:bdw */ > - /* WaDisableThreadStallDopClockGating:bdw */ > - /* FIXME: Unclear whether we really need this on production bdw. */ > + /* WaDisableThreadStallDopClockGating:bdw (pre-production until D) */ > intel_ring_emit_wa(ring, GEN8_ROW_CHICKEN, > _MASKED_BIT_ENABLE(PARTIAL_INSTRUCTION_SHOOTDOWN_DISABLE > | STALL_DOP_GATING_DISABLE)); > > - /* WaDisableDopClockGating:bdw May not be needed for production */ > + /* WaDisableDopClockGating:bdw */ > intel_ring_emit_wa(ring, GEN7_ROW_CHICKEN2, > _MASKED_BIT_ENABLE(DOP_CLOCK_GATING_DISABLE)); > > -- > 1.9.3 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Rodrigo Vivi Blog: http://blog.vivi.eng.br