From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thierry Reding <thierry.reding@gmail.com>
Subject: Re: [PATCH] drmOpenByName: remove redundant drmAvailable check
Date: Tue, 13 May 2014 12:16:34 +0200
Message-ID: <20140513101632.GA8407@ulmo>
References: <1397707681-27587-1-git-send-email-djkurtz@chromium.org>
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1438967245=="
Return-path: <dri-devel-bounces@lists.freedesktop.org>
Received: from mail-ee0-f50.google.com (mail-ee0-f50.google.com [74.125.83.50])
 by gabe.freedesktop.org (Postfix) with ESMTP id F19E36E637
 for <dri-devel@lists.freedesktop.org>; Tue, 13 May 2014 03:18:34 -0700 (PDT)
Received: by mail-ee0-f50.google.com with SMTP id e51so235004eek.37
 for <dri-devel@lists.freedesktop.org>; Tue, 13 May 2014 03:18:34 -0700 (PDT)
In-Reply-To: <1397707681-27587-1-git-send-email-djkurtz@chromium.org>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/dri-devel>,
 <mailto:dri-devel-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/dri-devel>
List-Post: <mailto:dri-devel@lists.freedesktop.org>
List-Help: <mailto:dri-devel-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/dri-devel>,
 <mailto:dri-devel-request@lists.freedesktop.org?subject=subscribe>
Errors-To: dri-devel-bounces@lists.freedesktop.org
Sender: "dri-devel" <dri-devel-bounces@lists.freedesktop.org>
To: Daniel Kurtz <djkurtz@chromium.org>
Cc: Dave Airlie <airlied@redhat.com>, dri-devel@lists.freedesktop.org
List-Id: dri-devel@lists.freedesktop.org


--===============1438967245==
Content-Type: multipart/signed; micalg=pgp-sha1;
	protocol="application/pgp-signature"; boundary="bp/iNruPH9dso1Pn"
Content-Disposition: inline


--bp/iNruPH9dso1Pn
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Apr 17, 2014 at 12:08:01PM +0800, Daniel Kurtz wrote:
> drmOpenByName() is a static function that is only called by drmOpen().
> drmOpen() already checks drmAvailable(), so the checki in
> drmOpenByName() is redundant.
>=20
> Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
> ---
>  xf86drm.c | 13 -------------
>  1 file changed, 13 deletions(-)

Looks good to me. I've fixed up the typo (checki -> check) in the commit
message and pushed this.

Thierry

--bp/iNruPH9dso1Pn
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (GNU/Linux)

iQIcBAEBAgAGBQJTcfEAAAoJEN0jrNd/PrOhvbIP/0zF/f9rB6bV5yIDjwwad4L6
sDyt7Wc1mssRy37UQw0SJs4Is/e+ez322S5HFTAEQxZoy+EFtydlJeFYWDaftbV5
sDS/9U6ARI5ocecrVGs3yH+43kC/jcqVNi3e6ig3N5BEKcrsVh6wgusTO0M1mcWB
vm6rNzD3WvLAgfOqRMsi9uj4X/3hOWbcK4BpJwjotTUbmdTEpbA91cCCrtWQhoO6
wUc8FltNIUpTkMd4sZdeepQFsZcgLHUQPvqdUrs/JZCKXp/UXxrCC/P2lD3hHMvz
T6SslN+bL1dIBbJRhb7MaFlLIKuvWK/laneyIn+RdnHerJoCWamTSGpP3SAYGyRO
CCQo3InxEmGQUMbtXNB9fCntK5Wz2sbHMkrFUsFO+3xxFC0CoiHaqksVWHQB9vCk
8BQbrjfqw1Dn9oVFEUIww6EA90OaFklQ/uRjc8u7nhfSwU9Udj1CSqx1Ip1MPT7O
0u51iX8nufVSK91iMoHh4Dwa/smj1T6Pax7gy/pRhS98CGL76Pm+Ih8zI09FcJvX
/Js8mdGlrLwPdtMk6W7gyVOqgTmK1Au01UbmD6c/Xju/con8hh1eUmFF3PflS5ez
58k2b/s5kESY+Jv/zaDG+TqCd5wCmq1JKCH8CmSc+WCfj/EDT3HemAORHy3KvNHG
d9IKM/lIkW7xj17KA8uk
=d/px
-----END PGP SIGNATURE-----

--bp/iNruPH9dso1Pn--

--===============1438967245==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

--===============1438967245==--