Don't include <isa/isavar.h> or compile code depending on it when isa

is not configured.  Including <isa/isavar.h> when it is not used is
harmful as well as bogus, since it includes "isa_if.h" which is not
generated when isa is not configured.

This was fixed in 1999 but was broken by unconditionalizing PNPBIOS.

4 files changed, 14 insertions, 1 deletions

diff --git a/sys/amd64/isa/clock.c b/sys/amd64/isa/clock.c
index a39f773..ae56051 100644
--- a/sys/amd64/isa/clock.c
+++ b/sys/amd64/isa/clock.c
@@ -49,6 +49,7 @@
  */
 
 #include "opt_clock.h"
+#include "opt_isa.h"
 #include "opt_mca.h"
 
 #include <sys/param.h>
@@ -83,7 +84,9 @@
 #include <i386/isa/icu.h>
 #include <i386/isa/isa.h>
 #include <isa/rtc.h>
+#ifdef DEV_ISA
 #include <isa/isavar.h>
+#endif
 #include <i386/isa/timerreg.h>
 
 #include <i386/isa/intr_machdep.h>
@@ -1260,6 +1263,7 @@ tsc_get_timecount(struct timecounter *tc)
 	return (rdtsc());
 }
 
+#ifdef DEV_ISA
 /*
  * Attach to the ISA PnP descriptors for the timer and realtime clock.
  */
@@ -1306,3 +1310,4 @@ static devclass_t attimer_devclass;
 
 DRIVER_MODULE(attimer, isa, attimer_driver, attimer_devclass, 0, 0);
 DRIVER_MODULE(attimer, acpi, attimer_driver, attimer_devclass, 0, 0);
+#endif /* DEV_ISA */
diff --git a/sys/amd64/isa/intr_machdep.c b/sys/amd64/isa/intr_machdep.c
index 3d20f9f..98194a1 100644
--- a/sys/amd64/isa/intr_machdep.c
+++ b/sys/amd64/isa/intr_machdep.c
@@ -148,6 +148,7 @@ static void	init_i8259(void);
 #define ENMI_IOSTATUS (1 << 5)
 #endif
 
+#ifdef DEV_ISA
 /*
  * Bus attachment for the ISA PIC.
  */
@@ -212,6 +213,7 @@ static devclass_t atpic_devclass;
 
 DRIVER_MODULE(atpic, isa, atpic_driver, atpic_devclass, 0, 0);
 DRIVER_MODULE(atpic, acpi, atpic_driver, atpic_devclass, 0, 0);
+#endif /* DEV_ISA */
 
 /*
  * Handle a NMI, possibly a machine check.
diff --git a/sys/amd64/isa/nmi.c b/sys/amd64/isa/nmi.c
index 3d20f9f..98194a1 100644
--- a/sys/amd64/isa/nmi.c
+++ b/sys/amd64/isa/nmi.c
@@ -148,6 +148,7 @@ static void	init_i8259(void);
 #define ENMI_IOSTATUS (1 << 5)
 #endif
 
+#ifdef DEV_ISA
 /*
  * Bus attachment for the ISA PIC.
  */
@@ -212,6 +213,7 @@ static devclass_t atpic_devclass;
 
 DRIVER_MODULE(atpic, isa, atpic_driver, atpic_devclass, 0, 0);
 DRIVER_MODULE(atpic, acpi, atpic_driver, atpic_devclass, 0, 0);
+#endif /* DEV_ISA */
 
 /*
  * Handle a NMI, possibly a machine check.
diff --git a/sys/amd64/isa/npx.c b/sys/amd64/isa/npx.c
index 5d44aaa..07bd2f5 100644
--- a/sys/amd64/isa/npx.c
+++ b/sys/amd64/isa/npx.c
@@ -37,6 +37,7 @@
 
 #include "opt_cpu.h"
 #include "opt_debug_npx.h"
+#include "opt_isa.h"
 #include "opt_math_emulate.h"
 #include "opt_npx.h"
 
@@ -83,7 +84,9 @@
 #endif
 #endif
 #include <i386/isa/intr_machdep.h>
+#ifdef DEV_ISA
 #include <isa/isavar.h>
+#endif
 
 /*
  * 387 and 287 Numeric Coprocessor Extension (NPX) Driver.
@@ -949,6 +952,7 @@ static driver_t npx_driver = {
 
 static devclass_t npx_devclass;
 
+#ifdef DEV_ISA
 /*
  * We prefer to attach to the root nexus so that the usual case (exception 16)
  * doesn't describe the processor as being `on isa'.
@@ -1003,4 +1007,4 @@ DRIVER_MODULE(npxisa, isa, npxisa_driver, npxisa_devclass, 0, 0);
 #ifndef PC98
 DRIVER_MODULE(npxisa, acpi, npxisa_driver, npxisa_devclass, 0, 0);
 #endif
-
+#endif /* DEV_ISA */