[PATCH] drmOpenByName: remove redundant drmAvailable check

[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 <djkurtz at chromium.org> 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 <djkurtz at chromium.org>
> ---
>  xf86drm.c | 13 -------------
>  1 file changed, 13 deletions(-)
>
> diff --git a/xf86drm.c b/xf86drm.c
> index 720952f..118022c 100644
> --- a/xf86drm.c
> +++ b/xf86drm.c
> @@ -537,19 +537,6 @@ static int drmOpenByName(const char *name)
>      int           fd;
>      drmVersionPtr version;
>      char *        id;
> -
> -    if (!drmAvailable()) {
> -       if (!drm_server_info) {
> -           return -1;
> -       }
> -       else {
> -           /* try to load the kernel module now */
> -           if (!drm_server_info->load_module(name)) {
> -               drmMsg("[drm] failed to load kernel module \"%s\"\n", name);
> -               return -1;
> -           }
> -       }
> -    }
>
>      /*
>       * Open the first minor number that matches the driver name and isn't
> --
> 1.9.1.423.g4596e3a
>