[PATCH] drmOpenByName: remove redundant drmAvailable check

2014-05-13 Thread Thierry Reding
On Tue, May 13, 2014 at 11:46:54AM +0800, Daniel Kurtz wrote: > Rob, > Can you take a look at this one too, now that I have your attention :-). Oh, and now I read that you wanted Rob to look at this. Rob, I hope you don't mind me jumping the queue. Thierry -- next part --

[PATCH] drmOpenByName: remove redundant drmAvailable check

2014-05-13 Thread Thierry Reding
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. > > Signed-off-by: Daniel Kurtz > --- > xf86drm.c | 13 -

[PATCH] drmOpenByName: remove redundant drmAvailable check

2014-05-13 Thread Daniel Kurtz
Rob, Can you take a look at this one too, now that I have your attention :-). -djk On Thu, Apr 17, 2014 at 12:08 PM, Daniel Kurtz wrote: > drmOpenByName() is a static function that is only called by drmOpen(). > drmOpen() already checks drmAvailable(), so the checki in > drmOpenByName() is redund

[PATCH] drmOpenByName: remove redundant drmAvailable check

2014-05-13 Thread Rob Clark
On Tue, May 13, 2014 at 7:49 AM, Thierry Reding wrote: > On Tue, May 13, 2014 at 11:46:54AM +0800, Daniel Kurtz wrote: >> Rob, >> Can you take a look at this one too, now that I have your attention :-). > > Oh, and now I read that you wanted Rob to look at this. Rob, I hope you > don't mind me jum

[PATCH] drmOpenByName: remove redundant drmAvailable check

2014-04-17 Thread Daniel Kurtz
drmOpenByName() is a static function that is only called by drmOpen(). drmOpen() already checks drmAvailable(), so the checki in drmOpenByName() is redundant. Signed-off-by: Daniel Kurtz --- xf86drm.c | 13 - 1 file changed, 13 deletions(-) diff --git a/xf86drm.c b/xf86drm.c index 7