From mboxrd@z Thu Jan 1 00:00:00 1970 From: Damien Lespiau Subject: Re: [PATCH 04/18] [v2] drm/i915: Add VECS semaphore bits Date: Wed, 29 May 2013 17:06:30 +0100 Message-ID: <20130529160630.GF17389@strange.amr.corp.intel.com> References: <1367110769-1306-1-git-send-email-ben@bwidawsk.net> <1369794154-1639-1-git-send-email-ben@bwidawsk.net> <1369794154-1639-5-git-send-email-ben@bwidawsk.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id E2AE9E63C3 for ; Wed, 29 May 2013 09:06:39 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1369794154-1639-5-git-send-email-ben@bwidawsk.net> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Ben Widawsky Cc: Intel GFX List-Id: intel-gfx@lists.freedesktop.org On Tue, May 28, 2013 at 07:22:20PM -0700, Ben Widawsky wrote: > Like the other rings, the VECS supports semaphores. The semaphore stuff > is a bit wonky so this patch on it's own should be nice for review. > > This patch should have no functional impact. > > v2: Fix the English parts of clarification (again, register names were > right, text was reversed) (Damien) > Restore the still valid invariant. (Damien) > The bsd semaphore register should be MI_SEMAPHORE_SYNC_VVE (Damien) > > Signed-off-by: Ben Widawsky Reviewed-by: Damien Lespiau -- Damien