From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michel =?ISO-8859-1?Q?D=E4nzer?= Subject: Re: [PATCH 2/3] drm/radeon: add error handling to fence_wait_empty_locked Date: Fri, 29 Jun 2012 17:19:09 +0200 Message-ID: <1340983149.3562.211.camel@thor.local> References: <1340981103-3717-1-git-send-email-deathsimple@vodafone.de> <1340981103-3717-2-git-send-email-deathsimple@vodafone.de> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail.gna.ch (darkcity.gna.ch [195.226.6.51]) by gabe.freedesktop.org (Postfix) with ESMTP id 589BFA0D9D for ; Fri, 29 Jun 2012 08:19:21 -0700 (PDT) In-Reply-To: <1340981103-3717-2-git-send-email-deathsimple@vodafone.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Christian =?ISO-8859-1?Q?K=F6nig?= Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Fre, 2012-06-29 at 16:45 +0200, Christian K=F6nig wrote: = > Instead of returning the error handle it directly > and while at it fix the comments about the ring lock. Sounds like this should really be two separate changes? > Signed-off-by: Christian K=F6nig Either way, Reviewed-by: Michel D=E4nzer -- = Earthling Michel D=E4nzer | http://www.amd.com Libre software enthusiast | Debian, X and DRI developer