[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4EB73A6A.5010900@kernel.org>
Date: Sun, 06 Nov 2011 20:54:50 -0500
From: Len Brown <lenb@...nel.org>
To: Thomas Renninger <trenn@...e.de>, x86@...nel.org
CC: linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org,
stable@...nel.org
Subject: Re: [PATCH] X86: Solve Dell Latitudes do not reboot on x86_64 more
generally
On 11/04/2011 08:44 PM, Thomas Renninger wrote:
> There are quite some reports that those do not reboot:
> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/833705
> ->E6520, E6220
> http://www.linux-club.de/viewtopic.php?f=29&t=114459&start=20
> ->E5520
> and some more already are blacklisted
>
> This patch improves the blacklist for all E6xxx and E5xxx Latitudes.
>
> Strange is that on above Ubuntu bug it was mentioned that 32 bit
> reboots just fine, which should use the same mechanism than 64 bit
> by default (ACPI).
>
> While this should solve the problem on short term for quite some users,
> hopefully a more generic way is found sooner or later...
>
>
> Signed-off-by: Thomas Renninger <trenn@...e.de>
> CC: stable@...nel.org
Acked-by: Len Brown <len.brown@...el.com>
Although this may be an issue with ACPI,
looks like this patch should go though TIP
because of the file changed.
thanks,
-Len
> ---
> arch/x86/kernel/reboot.c | 20 ++++++--------------
> 1 files changed, 6 insertions(+), 14 deletions(-)
>
> diff --git a/arch/x86/kernel/reboot.c b/arch/x86/kernel/reboot.c
> index 42f8cd7..7fcb20a 100644
> --- a/arch/x86/kernel/reboot.c
> +++ b/arch/x86/kernel/reboot.c
> @@ -419,28 +419,20 @@ static struct dmi_system_id __initdata pci_reboot_dmi_table[] = {
> DMI_MATCH(DMI_PRODUCT_NAME, "iMac9,1"),
> },
> },
> - { /* Handle problems with rebooting on the Latitude E6320. */
> - .callback = set_pci_reboot,
> - .ident = "Dell Latitude E6320",
> - .matches = {
> - DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> - DMI_MATCH(DMI_PRODUCT_NAME, "Latitude E6320"),
> - },
> - },
> - { /* Handle problems with rebooting on the Latitude E5420. */
> + {
> .callback = set_pci_reboot,
> - .ident = "Dell Latitude E5420",
> + .ident = "Dell Latitude E5xxx",
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> - DMI_MATCH(DMI_PRODUCT_NAME, "Latitude E5420"),
> + DMI_MATCH(DMI_PRODUCT_NAME, "Latitude E5"),
> },
> },
> - { /* Handle problems with rebooting on the Latitude E6420. */
> + {
> .callback = set_pci_reboot,
> - .ident = "Dell Latitude E6420",
> + .ident = "Dell Latitude E6xxx",
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> - DMI_MATCH(DMI_PRODUCT_NAME, "Latitude E6420"),
> + DMI_MATCH(DMI_PRODUCT_NAME, "Latitude E6"),
> },
> },
> {
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists