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    [W:0.026 / U:0.088 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site