From 32945af3f271506d49e17ab0709abfeaec309c79 Mon Sep 17 00:00:00 2001
From: Kevin O'Connor <kevin@koconnor.net>
Date: Fri, 27 Feb 2009 21:23:01 -0500
Subject: [PATCH] Don't 'autodetect' ATA PIO32 mode - use compile def instead.

The PIO32 detection appears to use a hack built for emulators.  It
    wont work on real hardware.
Implement with a compile time define instead.
This also improves the quality of the compiled code.
---
 src/ata.c    | 16 ++++++----------
 src/config.h | 10 ++++++----
 src/disk.c   |  3 +--
 src/disk.h   |  1 -
 4 files changed, 13 insertions(+), 17 deletions(-)

diff --git a/src/ata.c b/src/ata.c
index adf3929..e9f9c7c 100644
--- a/src/ata.c
+++ b/src/ata.c
@@ -220,10 +220,10 @@ send_cmd(int driveid, struct ata_pio_command *cmd)
 
 // Read and discard x number of bytes from an io channel.
 static void
-insx_discard(int mode, int iobase1, int bytes)
+insx_discard(int iobase1, int bytes)
 {
     int count, i;
-    if (mode == ATA_MODE_PIO32) {
+    if (CONFIG_ATA_PIO32) {
         count = bytes / 4;
         for (i=0; i<count; i++)
             inl(iobase1);
@@ -254,13 +254,12 @@ ata_transfer(int driveid, int iswrite, int count, int blocksize
     u8 channel  = driveid / 2;
     u16 iobase1 = GET_GLOBAL(ATA.channels[channel].iobase1);
     u16 iobase2 = GET_GLOBAL(ATA.channels[channel].iobase2);
-    u8 mode     = GET_GLOBAL(ATA.devices[driveid].mode);
     int current = 0;
     int status;
     for (;;) {
         int bsize = blocksize;
         if (skipfirst && current == 0) {
-            insx_discard(mode, iobase1, skipfirst);
+            insx_discard(iobase1, skipfirst);
             bsize -= skipfirst;
         }
         if (skiplast && current == count-1)
@@ -269,14 +268,14 @@ ata_transfer(int driveid, int iswrite, int count, int blocksize
         if (iswrite) {
             // Write data to controller
             dprintf(16, "Write sector id=%d dest=%p\n", driveid, buf_fl);
-            if (mode == ATA_MODE_PIO32)
+            if (CONFIG_ATA_PIO32)
                 outsl_fl(iobase1, buf_fl, bsize / 4);
             else
                 outsw_fl(iobase1, buf_fl, bsize / 2);
         } else {
             // Read data from controller
             dprintf(16, "Read sector id=%d dest=%p\n", driveid, buf_fl);
-            if (mode == ATA_MODE_PIO32)
+            if (CONFIG_ATA_PIO32)
                 insl_fl(iobase1, buf_fl, bsize / 4);
             else
                 insw_fl(iobase1, buf_fl, bsize / 2);
@@ -284,7 +283,7 @@ ata_transfer(int driveid, int iswrite, int count, int blocksize
         buf_fl += bsize;
 
         if (skiplast && current == count-1)
-            insx_discard(mode, iobase1, skiplast);
+            insx_discard(iobase1, skiplast);
 
         status = pause_await_not_bsy(iobase1, iobase2);
         if (status < 0)
@@ -642,9 +641,6 @@ extract_identify(int driveid, u16 *buffer)
 
     // Common flags.
     SET_GLOBAL(ATA.devices[driveid].removable, (buffer[0] & 0x80) ? 1 : 0);
-    // XXX - what is buffer[48]?
-    SET_GLOBAL(ATA.devices[driveid].mode
-               , buffer[48] ? ATA_MODE_PIO32 : ATA_MODE_PIO16);
 }
 
 static int
diff --git a/src/config.h b/src/config.h
index ac99dae..13b8472 100644
--- a/src/config.h
+++ b/src/config.h
@@ -25,12 +25,10 @@
 
 // Support for int13 floppy drive access
 #define CONFIG_FLOPPY_SUPPORT 1
-// Support for int15c2 mouse calls
-#define CONFIG_PS2_MOUSE 1
 // Support for IDE disk code
 #define CONFIG_ATA 1
-// Support calling int155f on each keyboard event
-#define CONFIG_KBD_CALL_INT15_4F 1
+// Use 32bit PIO accesses on ATA (minor optimization on PCI transfers)
+#define CONFIG_ATA_PIO32 0
 // Support for booting from a CD
 #define CONFIG_CDROM_BOOT 1
 // Support for emulating a boot CD as a floppy/harddrive
@@ -49,6 +47,10 @@
 #define CONFIG_LPT 1
 // Support int 16 keyboard calls
 #define CONFIG_KEYBOARD 1
+// Support calling int155f on each keyboard event
+#define CONFIG_KBD_CALL_INT15_4F 1
+// Support for int15c2 mouse calls
+#define CONFIG_PS2_MOUSE 1
 // Support finding and running option roms during post.
 #define CONFIG_OPTIONROMS 1
 // Set if option roms are already copied to 0xc0000-0xf0000
diff --git a/src/disk.c b/src/disk.c
index 9bad1d7..898fe49 100644
--- a/src/disk.c
+++ b/src/disk.c
@@ -452,7 +452,6 @@ disk_1348(struct bregs *regs, u8 device)
     u16 iobase1 = GET_GLOBAL(ATA.channels[channel].iobase1);
     u16 iobase2 = GET_GLOBAL(ATA.channels[channel].iobase2);
     u8 irq = GET_GLOBAL(ATA.channels[channel].irq);
-    u8 mode = GET_GLOBAL(ATA.devices[device].mode);
 
     u16 options = 0;
     if (type == ATA_TYPE_ATA) {
@@ -470,7 +469,7 @@ disk_1348(struct bregs *regs, u8 device)
         options |= 1<<6; // atapi device
     }
     options |= 1<<4; // lba translation
-    if (mode == ATA_MODE_PIO32)
+    if (CONFIG_ATA_PIO32)
         options |= 1<<7;
 
     SET_EBDA2(ebda_seg, dpte.iobase1, iobase1);
diff --git a/src/disk.h b/src/disk.h
index 3661d27..8e9f1cb 100644
--- a/src/disk.h
+++ b/src/disk.h
@@ -174,7 +174,6 @@ struct ata_device_s {
     u8  type;         // Detected type of ata (ata/atapi/none/unknown)
     u8  device;       // Detected type of attached devices (hd/cd/none)
     u8  removable;    // Removable device flag
-    u8  mode;         // transfer mode : PIO 16/32 bits - IRQ - ISADMA - PCIDMA
     u16 blksize;      // block size
     u8  version;      // ATA/ATAPI version
 
-- 
2.25.1