Skip to contentSkip to conversation

es7000 broken without acpi

PATCH In review Eric Sesterhenn / snakebyte 3 participants 3 messages Dec 12, 2005 – Jan 4, 2006

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 es7000 broken without acpi

hi,

a make randconfig gave me a situation where es7000 was enabled, but acpi wasnt ( dont know if this makes sense ), gcc gave me some compiling errors, which the following patch fixes. Please cc me as i am not on the list. Thanks.

arch/i386/mach-es7000/es7000.h renamed from arch/i386/mach-es7000.orig/es7000.h +2 -0
diff -up linux-2.6.15-rc5-git2/arch/i386/mach-es7000.orig/es7000.h linux-2.6.15-rc5-git2/arch/i386/mach-es7000/es7000.h
@@ -83,6 +83,7 @@ struct es7000_oem_table {
 	struct psai psai;
 };
 
+#if defined(CONFIG_X86_IO_APIC) && defined(CONFIG_ACPI)
 struct acpi_table_sdt {
 	unsigned long pa;
 	unsigned long count;
@@ -98,6 +99,7 @@ struct oem_table {
 	u32 OEMTableAddr;
 	u32 OEMTableSize;
 };
+#endif
 
 struct mip_reg {
 	unsigned long long off_0;
arch/i386/mach-es7000/es7000plat.c renamed from arch/i386/mach-es7000.orig/es7000plat.c +4 -0
diff -up linux-2.6.15-rc5-git2/arch/i386/mach-es7000.orig/es7000plat.c linux-2.6.15-rc5-git2/arch/i386/mach-es7000/es7000plat.c
@@ -92,7 +92,9 @@ setup_unisys(void)
 		es7000_plat = ES7000_ZORRO;
 	else
 		es7000_plat = ES7000_CLASSIC;
+#if defined(CONFIG_X86_IO_APIC) && defined(CONFIG_ACPI)
 	ioapic_renumber_irq = es7000_rename_gsi;
+#endif
 }
 
 /*
@@ -160,6 +162,7 @@ parse_unisys_oem (char *oemptr)
 	return es7000_plat;
 }
 
+#if defined(CONFIG_X86_IO_APIC) && defined(CONFIG_ACPI)
 int __init
 find_unisys_acpi_oem_table(unsigned long *oem_addr)
 {
@@ -212,6 +215,7 @@ find_unisys_acpi_oem_table(unsigned long
 	}
 	return -1;
 }
+#endif
 
 static void
 es7000_spin(int n)
#1 lore raw
Quoted 8 lines · Eric Sesterhenn / snakebyte

hi,

a make randconfig gave me a situation where es7000 was enabled, but acpi wasnt ( dont know if this makes sense ), gcc gave me some compiling errors, which the following patch fixes. Please cc me as i am not on the list. Thanks.

I believe that es7000 requires ACPI, so a better fix would be to enforce that within Kconfig.

Natalie, can you please comment?

Quoted 57 lines

diff -up linux-2.6.15-rc5-git2/arch/i386/mach-es7000.orig/es7000.h linux-2.6.15-rc5-git2/arch/i386/mach-es7000/es7000.h

arch/i386/mach-es7000/es7000.h renamed from arch/i386/mach-es7000.orig/es7000.h +2 -0
@@ -83,6 +83,7 @@ struct es7000_oem_table {
 	struct psai psai;
 };
 
+#if defined(CONFIG_X86_IO_APIC) && defined(CONFIG_ACPI)
 struct acpi_table_sdt {
 	unsigned long pa;
 	unsigned long count;
@@ -98,6 +99,7 @@ struct oem_table {
 	u32 OEMTableAddr;
 	u32 OEMTableSize;
 };
+#endif
 
 struct mip_reg {
 	unsigned long long off_0;

diff -up linux-2.6.15-rc5-git2/arch/i386/mach-es7000.orig/es7000plat.c linux-2.6.15-rc5-git2/arch/i386/mach-es7000/es7000plat.c

arch/i386/mach-es7000/es7000plat.c renamed from arch/i386/mach-es7000.orig/es7000plat.c +4 -1
@@ -92,7 +92,9 @@ setup_unisys(void)
 		es7000_plat = ES7000_ZORRO;
 	else
 		es7000_plat = ES7000_CLASSIC;
+#if defined(CONFIG_X86_IO_APIC) && defined(CONFIG_ACPI)
 	ioapic_renumber_irq = es7000_rename_gsi;
+#endif
 }
 
 /*
@@ -160,6 +162,7 @@ parse_unisys_oem (char *oemptr)
 	return es7000_plat;
 }
 
+#if defined(CONFIG_X86_IO_APIC) && defined(CONFIG_ACPI)
 int __init
 find_unisys_acpi_oem_table(unsigned long *oem_addr)
 {
@@ -212,6 +215,7 @@ find_unisys_acpi_oem_table(unsigned long
 	}
 	return -1;
 }
+#endif
 
 static void
 es7000_spin(int n)
 
 
-

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

#2 lore raw
Quoted 6 lines · Andrew Morton

I believe that es7000 requires ACPI, so a better fix would be to enforce that within Kconfig.

Natalie, can you please comment?

This was discussed back in October [1], but nothing became of it. Should I resend the patch perhaps?

Peter

[1] http://marc.theaimsgroup.com/?t=112928755800002&r=1&w=2

Signature · 3 lines
Peter Hagervall......................email: [email protected]
Department of Computing Science........tel: +46(0)90 786 7018
University of Umeå, SE-901 87 Umeå.....fax: +46(0)90 786 6126