[PATCH] UBUNTU: SAUCE: acpi: Fixes battery power regression on HP laptops (bug 453963)

Stefan Bader stefan.bader at canonical.com
Tue Jan 12 17:18:29 GMT 2010


Gadi Cohen wrote:
> DMI init_set_sci_en_on_resume for varios HP/Compaq laptops.
> Patched against Ubuntu- 2.6.31-17.54. Shipped in linux-2.6.32.
> 
> BugLink: http://bugs.launchpad.net/bugs/453963
> 
> Signed-off-by: Gadi Cohen <dragon at wastelands.net>

Having a quick lock it seems that this patch just merges several upstream
commits into one patch. Which is not how things should happen. Could you rather
send us the upstream commit ids of the patches that should get applied.
The changes itself look like stable material. Depending on the age of them we
should also ask for upstream stable inclusion.
But it helps a lot if you could do the research part on the patches required
from upstream.

Thanks,
Stefan


> ---
>  drivers/acpi/sleep.c |   42 ++++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 42 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/acpi/sleep.c b/drivers/acpi/sleep.c
> index 42159a2..53f7ee8 100644
> --- a/drivers/acpi/sleep.c
> +++ b/drivers/acpi/sleep.c
> @@ -405,6 +405,48 @@ static struct dmi_system_id __initdata
> acpisleep_dmi_table[] = {
>          },
>      },
>      {
> +    .callback = init_set_sci_en_on_resume,
> +    .ident = "Hewlett-Packard HP Pavilion dv3 Notebook PC",
> +    .matches = {
> +        DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
> +        DMI_MATCH(DMI_PRODUCT_NAME, "HP Pavilion dv3 Notebook PC"),
> +        },
> +    },
> +    {
> +    .callback = init_set_sci_en_on_resume,
> +    .ident = "Hewlett-Packard Pavilion dv4",
> +    .matches = {
> +        DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
> +        DMI_MATCH(DMI_PRODUCT_NAME, "HP Pavilion dv4"),
> +        },
> +    },
> +    {
> +    .callback = init_set_sci_en_on_resume,
> +    .ident = "Hewlett-Packard Pavilion dv7",
> +    .matches = {
> +        DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
> +        DMI_MATCH(DMI_PRODUCT_NAME, "HP Pavilion dv7"),
> +        },
> +    },
> +    {
> +    .callback = init_set_sci_en_on_resume,
> +    .ident = "Hewlett-Packard Compaq Presario CQ40 Notebook PC",
> +    .matches = {
> +        DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
> +        DMI_MATCH(DMI_PRODUCT_NAME,
> +                "Compaq Presario CQ40 Notebook PC"),
> +        },
> +    },
> +    {
> +    .callback = init_set_sci_en_on_resume,
> +    .ident = "Hewlett-Packard Compaq Presario C700 Notebook PC",
> +    .matches = {
> +        DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
> +        DMI_MATCH(DMI_PRODUCT_NAME,
> +                "Compaq Presario C700 Notebook PC""),
> +        },
> +    },
> +    {
>      .callback = init_old_suspend_ordering,
>      .ident = "Panasonic CF51-2L",
>      .matches = {




More information about the kernel-team mailing list