On Mon, 21 Mar 2011 17:27:13 +0800
Zhenyu Wang <[email protected]> wrote:

> we don't use it anywhere.
> 
> Signed-off-by: Zhenyu Wang <[email protected]>
> ---
>  drivers/gpu/drm/i915/i915_suspend.c |    4 ----
>  1 files changed, 0 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_suspend.c 
> b/drivers/gpu/drm/i915/i915_suspend.c
> index 0521ecf..422981b 100644
> --- a/drivers/gpu/drm/i915/i915_suspend.c
> +++ b/drivers/gpu/drm/i915/i915_suspend.c
> @@ -795,8 +795,6 @@ int i915_save_state(struct drm_device *dev)
>       struct drm_i915_private *dev_priv = dev->dev_private;
>       int i;
>  
> -     pci_read_config_byte(dev->pdev, LBB, &dev_priv->saveLBB);
> -
>       /* Hardware status page */
>       dev_priv->saveHWS = I915_READ(HWS_PGA);
>  
> @@ -844,8 +842,6 @@ int i915_restore_state(struct drm_device *dev)
>       struct drm_i915_private *dev_priv = dev->dev_private;
>       int i;
>  
> -     pci_write_config_byte(dev->pdev, LBB, dev_priv->saveLBB);
> -
>       /* Hardware status page */
>       I915_WRITE(HWS_PGA, dev_priv->saveHWS);

This may affect older platforms and prevent backlight save/restore?  At
least I think that's why we added it in the first place...

-- 
Jesse Barnes, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to