Re: [PATCH 1/1] Work around Dell E520 BIOS reboot bug.

[Andrew Morton]

On Fri, 25 May 2007 21:58:09 -0500
Matt Domsch <Matt_Domsch@xxxxxxxx> wrote:

> On Wed, May 23, 2007 at 01:12:44PM -0600, Tim Gardner wrote:
> > From: Tim Gardner <tim.gardner@xxxxxxxxxx>
> > Subject: [PATCH] Work around Dell E520 BIOS reboot bug.
> > 
> > Force Dell E520 to use the BIOS to shutdown/reboot.
> > 
> > Signed-off-by: Tim Gardner <tim.gardner@xxxxxxxxxx>
> 
> Acked-by: Matt Domsch <Matt_Domsch@xxxxxxxx>
> 
> Thanks Tim for submitting this.

Do you think this is also required in 2.6.21.x?

Andi: ack for 2.6.22 inclusion?

Thanks.



From: Tim Gardner <tim.gardner@xxxxxxxxxx>

Force Dell E520 to use the BIOS to shutdown/reboot.

I have at least one report that this patch fixes shutdown/reboot
problems on the Dell E520 platform.

Signed-off-by: Tim Gardner <tim.gardner@xxxxxxxxxx>
Cc: Andi Kleen <ak@xxxxxxx>
Acked-by: Matt Domsch <Matt_Domsch@xxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---

 arch/i386/kernel/reboot.c |    8 ++++++++
 1 files changed, 8 insertions(+)

diff -puN arch/i386/kernel/reboot.c~work-around-dell-e520-bios-reboot-bug arch/i386/kernel/reboot.c
--- a/arch/i386/kernel/reboot.c~work-around-dell-e520-bios-reboot-bug
+++ a/arch/i386/kernel/reboot.c
@@ -89,6 +89,14 @@ static int __init set_bios_reboot(struct
 }
 
 static struct dmi_system_id __initdata reboot_dmi_table[] = {
+	{	/* Handle problems with rebooting on Dell E520's */
+		.callback = set_bios_reboot,
+		.ident = "Dell E520",
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+			DMI_MATCH(DMI_PRODUCT_NAME, "Dell DM061"),
+		},
+	},
 	{	/* Handle problems with rebooting on Dell 1300's */
 		.callback = set_bios_reboot,
 		.ident = "Dell PowerEdge 1300",
_

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/