From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 0/5] Make num_sprites a per pipe value (v3) Date: Wed, 5 Mar 2014 18:48:38 +0100 Message-ID: <20140305174838.GQ17001@phenom.ffwll.local> References: <1393867908-4604-1-git-send-email-damien.lespiau@intel.com> <20140303211638.GB17720@nuc-i3427.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ea0-f176.google.com (mail-ea0-f176.google.com [209.85.215.176]) by gabe.freedesktop.org (Postfix) with ESMTP id C5D93FA890 for ; Wed, 5 Mar 2014 09:48:42 -0800 (PST) Received: by mail-ea0-f176.google.com with SMTP id o10so1200704eaj.7 for ; Wed, 05 Mar 2014 09:48:42 -0800 (PST) Content-Disposition: inline In-Reply-To: <20140303211638.GB17720@nuc-i3427.alporthouse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Chris Wilson , Damien Lespiau , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Mon, Mar 03, 2014 at 09:16:38PM +0000, Chris Wilson wrote: > On Mon, Mar 03, 2014 at 05:31:43PM +0000, Damien Lespiau wrote: > > Follow up of: > > http://lists.freedesktop.org/archives/intel-gfx/2014-February/040789.html > > > > With a couple of changes: > > > > - Fix up the poor style when initializing the per-pipe num_sprites > > - Added more for_each_pipe(pipe) conversion > > - Added a small cleanup patch spotted when doing the previous patch > > Nothing upset me (apart from the concept of variable number of planes > per pipe and the inflexible assumption that it is constant in certain > client facing APIs), not even the expansion of intel_info, so > Reviewed-by: Chris Wilson All pulled in, thanks for patches&review. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch