lkml.org 
[lkml]   [2011]   [Nov]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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@suse.de>
> CC: stable@kernel.org


Acked-by: Len Brown <len.brown@intel.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"),
> },
> },
> {




\
 
 \ /
  Last update: 2011-11-07 02:57    [from the cache]
©2003-2011 Jasper Spaans