From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 0/5] A few fixes on top of the wa_regs patches Date: Wed, 3 Sep 2014 12:40:13 +0200 Message-ID: <20140903104013.GL15520@phenom.ffwll.local> References: <1409413863-7681-1-git-send-email-damien.lespiau@intel.com> <5406DCF7.2060206@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wg0-f47.google.com (mail-wg0-f47.google.com [74.125.82.47]) by gabe.freedesktop.org (Postfix) with ESMTP id 5A5026E2BD for ; Wed, 3 Sep 2014 03:39:51 -0700 (PDT) Received: by mail-wg0-f47.google.com with SMTP id z12so8285289wgg.30 for ; Wed, 03 Sep 2014 03:39:50 -0700 (PDT) Content-Disposition: inline In-Reply-To: <5406DCF7.2060206@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: "Siluvery, Arun" Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Wed, Sep 03, 2014 at 10:18:47AM +0100, Siluvery, Arun wrote: > On 30/08/2014 16:50, Damien Lespiau wrote: > >Hi Arun, > > > >I've compiled a few patches that I think solve some small-ish issues around > >your wa_regs series. Could you please have a look at them and comment/give your > >r-b tag if you judge appropriate? > > > >On top of those patches, I'd love some comments on the issues I raised in the > >other mail and possible follow up patches to address them. > > > > http://lists.freedesktop.org/archives/intel-gfx/2014-August/051514.html > > > >At some point, we'll also need a bit of coherence with what Mika has been doing: > > > > http://lists.freedesktop.org/archives/intel-gfx/2014-August/050000.html > > > Hi Daniel, > > Since the new workaround design/implementation takes time could you please > pull the patches in this series to fix the issues and also the patch to > change filename in igt. Yes this is the best approach since clarifying what exactly the full blown w/a infrastructure should do will take a bit of time. > for the series, > Reviewed-by: Arun Siluvery Thanks, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch