diff options
| author | Hong Liu <hong.liu@intel.com> | 2008-07-15 10:14:17 -0700 |
|---|---|---|
| committer | Ian Romanick <ian.d.romanick@intel.com> | 2008-07-15 10:14:17 -0700 |
| commit | dfd441cf964f20e4a761cb8490d7cd82cf32e7b9 (patch) | |
| tree | 152309de7821177bf9073ab8cb9337fc5421a321 /linux-core/i915_drv.c | |
| parent | 764573f3b8bec90f20c3bdd58c4b55490fbbdaf4 (diff) | |
This is a modified version of Hong's patch from last month, with a few
modifications to make it work correctly on my test hardware (altered the
backlight write function, made it enable the legacy backlight controller
interrupts on mobile hardware, sorted the interrupt function so we don't
get an excessive number of vblank interrupts). This lets the backlight
keys on my T61 work properly, though there's a 750msec or so delay
between the request and the brightness actually changing - this sounds
awfully like the hardware spinning waiting for a status flag to become
ready, but as far as I can tell they're all set correctly. If anyone can
figure out what's wrong here, it'd be nice to know.
Some of the functions are still stubs and just tell the hardware that
the request was successful. These can be filled in as kernel modesetting
gets integrated. I think it's worth getting this in anyway, since it's
required for backlight control to work properly on some new platforms.
Signed-off-by: Matthew Garrett <mjg@redhat.com>
Diffstat (limited to 'linux-core/i915_drv.c')
| -rw-r--r-- | linux-core/i915_drv.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/linux-core/i915_drv.c b/linux-core/i915_drv.c index be25239d..abb45de7 100644 --- a/linux-core/i915_drv.c +++ b/linux-core/i915_drv.c @@ -80,6 +80,8 @@ static int i915_suspend(struct drm_device *dev, pm_message_t state) i915_save_state(dev); + intel_opregion_free(dev); + if (state.event == PM_EVENT_SUSPEND) { /* Shut down the device */ pci_disable_device(dev->pdev); @@ -99,6 +101,8 @@ static int i915_resume(struct drm_device *dev) i915_restore_state(dev); + intel_opregion_init(dev); + return 0; } |