Call acpi_video_register() in intel_opregion_init() failure path
authorDavid Woodhouse <David.Woodhouse@intel.com>
Thu, 12 Aug 2010 23:56:54 +0000 (00:56 +0100)
committerDavid Woodhouse <David.Woodhouse@intel.com>
Fri, 13 Aug 2010 00:10:10 +0000 (01:10 +0100)
If i915 opregion is present, the acpi_video driver doesn't register
itself immediately; it defers that until the i915 opregion code is done.
But if that *fails*, the acpi_video driver was never getting registered.
And thus I have no backlight support on my Lenovo IdeaPad S10-3.

Call acpi_video_register() on the failure path, and it works again.

Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Acked-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Acked-by: Matthew Garrett <mjg@redhat.com>
drivers/gpu/drm/i915/i915_opregion.c

index 8fcc75c1aa2831be0bac489f9dd2043907d2cf69..d1bf92b99788807759b70436c7f53c48c4487f1e 100644 (file)
@@ -535,6 +535,7 @@ int intel_opregion_init(struct drm_device *dev, int resume)
 err_out:
        iounmap(opregion->header);
        opregion->header = NULL;
+       acpi_video_register();
        return err;
 }