diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2017-02-28 20:48:43 +0000 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2017-02-28 20:52:19 +0000 |
| commit | 78d7a09b0343829c81257024b164b0b3764392ac (patch) | |
| tree | c2e95d747ce135c9887034cba9847128f3ac138a | |
| parent | e9cd8c211dcaba3bad4bc950a0a06b35766962ba (diff) | |
intel: Fix checking xf86LoadKernelModule for success
Originally introduced in
commit f66e25def3431a900068cc1c23a6b1e8322ef046
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date: Mon Jun 23 16:14:28 2014 +0100
intel: Wait for the DRM device to load
the legacy path was checking the return xf86LoadKernelModule
incorrectly. This error was then copy into the common loader in
commit 6a2dcb388e6b549c3175ccfbcd3f1751e25de40a
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date: Fri Jul 31 10:40:32 2015 +0100
intel: Refactor i915.ko loading support
xf86LoadKernelModule() returns 1 on success and zero on failure.
Reported-by: David Shao <davshao@gmail.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100011
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
| -rw-r--r-- | src/intel_device.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/intel_device.c b/src/intel_device.c index 04ad7b6a..c4910cd8 100644 --- a/src/intel_device.c +++ b/src/intel_device.c @@ -233,7 +233,7 @@ static int load_i915_kernel_module(void) const char **kn; for (kn = kernel_module_names; *kn; kn++) - if (xf86LoadKernelModule(*kn) == 0) + if (xf86LoadKernelModule(*kn)) return 0; return -1; |