[PATCH] xf86drm: remove to open the DRM device unnecessarily

Daniel Kurtz djkurtz at google.com
Thu May 28 07:15:52 PDT 2015


It's not necessary if we are about to skip the rest of the if clause anyway
because name is NULL.
On May 28, 2015 9:14 PM, "Emil Velikov" <emil.l.velikov at gmail.com> wrote:

> On 28 May 2015 at 00:57, Joonyoung Shim <jy0922.shim at samsung.com> wrote:
> > This is to remove to open the DRM device unnecessarily as call
> > drmAvailable() when name is NULL or drm_server_info is NULL in
> > drmOpenWithType function.
> >
> Why do you believe that calling drmAvailable() is not necessary ? If
> that's the case should one just nuke the call all together ?
>
> -Emil
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150528/4ff9b577/attachment-0001.html>


More information about the dri-devel mailing list