- Move timerreg.h to <arch>/include and split i8253 specific defines into

  i8253reg.h, and add some defines to control a speaker.
- Move PPI related defines from i386/isa/spkr.c into ppireg.h and use them.
- Move IO_{PPI,TIMER} defines into ppireg.h and timerreg.h respectively.
- Use isa/isareg.h rather than <arch>/isa/isa.h.

Tested on: i386, pc98

4 files changed, 21 insertions, 26 deletions

diff --git a/sys/pc98/cbus/cbus.h b/sys/pc98/cbus/cbus.h
index 31a35b0..8adcce8 100644
--- a/sys/pc98/cbus/cbus.h
+++ b/sys/pc98/cbus/cbus.h
@@ -57,9 +57,7 @@
 #define	IO_ICU2		0x008		/* 8259A Interrupt Controller #2 */
 #define	IO_RTC		0x020		/* 4990A RTC */
 #define	IO_SYSPORT	0x031		/* 8255A System Port */
-#define	IO_PPI		0x035		/* Programmable Peripheral Interface */
 #define	IO_KBD		0x041		/* 8251A Keyboard */
-#define	IO_TIMER1	0x071		/* 8253C Timer */
 #define	IO_COM2		0x0B1		/* 8251A RS232C serial I/O (ext) */
 #define	IO_COM3		0x0B9		/* 8251A RS232C serial I/O (ext) */
 #define	IO_FDPORT	0x0BE		/* FD I/F port (1M<->640K,EMTON) */
diff --git a/sys/pc98/cbus/clock.c b/sys/pc98/cbus/clock.c
index f3d956b..ee75914 100644
--- a/sys/pc98/cbus/clock.c
+++ b/sys/pc98/cbus/clock.c
@@ -79,6 +79,8 @@
 #include <machine/apicvar.h>
 #endif
 #include <machine/specialreg.h>
+#include <machine/ppireg.h>
+#include <machine/timerreg.h>
 
 #include <i386/isa/icu.h>
 #include <pc98/cbus/cbus.h>
@@ -86,7 +88,6 @@
 #ifdef DEV_ISA
 #include <isa/isavar.h>
 #endif
-#include <i386/isa/timerreg.h>
 
 /*
  * 32-bit time_t's can't reach leap years before 1904 or after 2036, so we
@@ -357,8 +358,8 @@ DELAY(int n)
 static void
 sysbeepstop(void *chan)
 {
-	outb(IO_PPI, inb(IO_PPI)|0x08);	/* disable counter1 output to speaker */
-	release_timer1();
+	ppi_spkr_off();		/* disable counter1 output to speaker */
+	timer_spkr_release();
 	beeping = 0;
 }
 
@@ -367,19 +368,18 @@ sysbeep(int pitch, int period)
 {
 	int x = splclock();
 
-	if (acquire_timer1(TIMER_SQWAVE|TIMER_16BIT))
+	if (timer_spkr_acquire())
 		if (!beeping) {
 			/* Something else owns it. */
 			splx(x);
 			return (-1); /* XXX Should be EBUSY, but nobody cares anyway. */
 		}
 	disable_intr();
-	outb(0x3fdb, pitch);
-	outb(0x3fdb, (pitch>>8));
+	spkr_set_pitch(pitch);
 	enable_intr();
 	if (!beeping) {
 		/* enable counter1 output to speaker */
-		outb(IO_PPI, (inb(IO_PPI) & 0xf7));
+		ppi_spkr_on();
 		beeping = period;
 		timeout(sysbeepstop, (void *)NULL, period);
 	}
diff --git a/sys/pc98/cbus/pcrtc.c b/sys/pc98/cbus/pcrtc.c
index f3d956b..ee75914 100644
--- a/sys/pc98/cbus/pcrtc.c
+++ b/sys/pc98/cbus/pcrtc.c
@@ -79,6 +79,8 @@
 #include <machine/apicvar.h>
 #endif
 #include <machine/specialreg.h>
+#include <machine/ppireg.h>
+#include <machine/timerreg.h>
 
 #include <i386/isa/icu.h>
 #include <pc98/cbus/cbus.h>
@@ -86,7 +88,6 @@
 #ifdef DEV_ISA
 #include <isa/isavar.h>
 #endif
-#include <i386/isa/timerreg.h>
 
 /*
  * 32-bit time_t's can't reach leap years before 1904 or after 2036, so we
@@ -357,8 +358,8 @@ DELAY(int n)
 static void
 sysbeepstop(void *chan)
 {
-	outb(IO_PPI, inb(IO_PPI)|0x08);	/* disable counter1 output to speaker */
-	release_timer1();
+	ppi_spkr_off();		/* disable counter1 output to speaker */
+	timer_spkr_release();
 	beeping = 0;
 }
 
@@ -367,19 +368,18 @@ sysbeep(int pitch, int period)
 {
 	int x = splclock();
 
-	if (acquire_timer1(TIMER_SQWAVE|TIMER_16BIT))
+	if (timer_spkr_acquire())
 		if (!beeping) {
 			/* Something else owns it. */
 			splx(x);
 			return (-1); /* XXX Should be EBUSY, but nobody cares anyway. */
 		}
 	disable_intr();
-	outb(0x3fdb, pitch);
-	outb(0x3fdb, (pitch>>8));
+	spkr_set_pitch(pitch);
 	enable_intr();
 	if (!beeping) {
 		/* enable counter1 output to speaker */
-		outb(IO_PPI, (inb(IO_PPI) & 0xf7));
+		ppi_spkr_on();
 		beeping = period;
 		timeout(sysbeepstop, (void *)NULL, period);
 	}
diff --git a/sys/pc98/cbus/syscons_cbus.c b/sys/pc98/cbus/syscons_cbus.c
index fc430e9..7247f48 100644
--- a/sys/pc98/cbus/syscons_cbus.c
+++ b/sys/pc98/cbus/syscons_cbus.c
@@ -38,14 +38,14 @@
 #include <sys/sysctl.h>
 
 #include <machine/clock.h>
+#include <machine/ppireg.h>
+#include <machine/timerreg.h>
 
 #include <pc98/cbus/cbus.h>
 #include <pc98/pc98/pc98_machdep.h>
 
 #include <dev/syscons/syscons.h>
 
-#include <i386/isa/timerreg.h>
-
 #include <isa/isavar.h>
 
 static devclass_t	sc_devclass;
@@ -227,22 +227,19 @@ sc_get_bios_values(bios_values_t *values)
 int
 sc_tone(int herz)
 {
-	int pitch;
 
 	if (herz) {
 		/* enable counter 1 */
-		outb(0x35, inb(0x35) & 0xf7);
+		ppi_spkr_on();
 		/* set command for counter 1, 2 byte write */
-		if (acquire_timer1(TIMER_16BIT | TIMER_SQWAVE))
+		if (timer_spkr_acquire())
 			return EBUSY;
 		/* set pitch */
-		pitch = timer_freq/herz;
-		outb(TIMER_CNTR1, pitch);
-		outb(TIMER_CNTR1, pitch >> 8);
+		spkr_set_pitch(timer_freq / herz);
 	} else {
 		/* disable counter 1 */
-		outb(0x35, inb(0x35) | 0x08);
-		release_timer1();
+		ppi_spkr_off();
+		timer_spkr_release();
 	}
 	return 0;
 }