[Lucid] [PATCH 1/1] SRU: Dell Studio 1558 hangs on resume from suspend

[Tim Gardner]

On 04/19/2010 06:15 PM, Kamal Mostafa wrote:
> Proposed patch for Lucid SRU-
>
> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/553498
> Dell Studio 1558 (Arrandale) hangs on resume from suspend
>
> SRU Justification/Impact:
>
> Dell Studio 1558 (and presumably other 155x models) hangs on resume from
> suspend.  Attached patch adds the Dell Studio models (1558, 1557, 1555)
> to the list of machines needing 'set_sci_en_on_resume' handling, fixing
> resume on the 1558.  This change affects only the listed machines.
>
> TEST CASE:
>
> I have built and tested a kernel with this patch and confirmed that it
> reliably fixes resume from suspend on my own Dell Studio 1558.
>
> =====
>
>  From d6d16d70aaebc715795ceaa9fbfbc078703c983a Mon Sep 17 00:00:00 2001
> From: Kamal Mostafa<kamal at canonical.com>
> Date: Mon, 19 Apr 2010 17:45:33 -0400
> Subject: [PATCH] UBUNTU: SAUCE: Fix hang on resume for Dell Studio 1558,
> 1557, 1555
>
> BugLink: http://bugs.launchpad.net/bugs/553498
> ---
>   debian.master/changelog |    9 ++++-----
>   drivers/acpi/sleep.c    |   24 ++++++++++++++++++++++++
>   2 files changed, 28 insertions(+), 5 deletions(-)
>
> diff --git a/debian.master/changelog b/debian.master/changelog
> index f661b70..b2b36c8 100644
> --- a/debian.master/changelog
> +++ b/debian.master/changelog
> @@ -1,10 +1,9 @@
> -linux (2.6.32-21.33) UNRELEASED; urgency=low
> +linux (2.6.32-21.33~kamal~dell_155x_resume) lucid; urgency=low
>
> -  CHANGELOG: Do not edit directly. Autogenerated at release.
> -  CHANGELOG: Use the printchanges target to see the curent changes.
> -  CHANGELOG: Use the insertchanges target to create the final log.
> +  * Fix hang on resume for Dell Studio 1558, 1557, 1555
> (set_sci_en_on_resume)
> +    - LP: #553498
>
> - -- Andy Whitcroft<apw at canonical.com>   Mon, 19 Apr 2010 11:06:35 +0100
> + -- Kamal Mostafa<kamal at canonical.com>   Mon, 19 Apr 2010 15:20:45
> -0400
>
>   linux (2.6.32-21.32) lucid; urgency=low
>

The quirk patch looks fine, but we don't patch or update 
debian.master/changelog manually since updates to that file are 
generated by 'debian/rules insertchanges'.

rtg
-- 
Tim Gardner tim.gardner at canonical.com