ps1: separate /ack and transfer delays into two counters

ares/ps1/peripheral/io.cpp

@@ -13,36 +13,30 @@ auto Peripheral::transmit(u8 data) -> void {
 
   //Calculate the number of cycles required to transfer a byte at the current baud rate
   //This is added to the /ACK delay to determine the total duration until /ACK is asserted
-  //NOTE: transfer time is only emulated for memory cards for now, not controllers
-  //enabling it for controllers causes games to drop input; needs further investigation
   u8 factors[4] = {1, 1, 16, 64};
-  auto transferCycles = ((io.baudrateReloadValue * factors[io.baudrateReloadFactor])) * 8;
+  io.transferCounter = ((io.baudrateReloadValue * factors[io.baudrateReloadFactor])) * 8;
 
   if(io.slotNumber == 0) {
     if(!memoryCardPort1.active()) {
-      io.receiveSize = 1;
       io.receiveData = controllerPort1.bus(data);
-      if(controllerPort1.acknowledge()) io.counter = /*transferCycles*/ + 338; // approx 9.98us
+      if(controllerPort1.acknowledge()) io.ackCounter = 338; // approx 9.98us
     }
 
     if(!controllerPort1.active()) {
-      io.receiveSize = 1;
       io.receiveData = memoryCardPort1.bus(data);
-      if(memoryCardPort1.acknowledge()) io.counter = transferCycles + 170; // approx 5us
+      if(memoryCardPort1.acknowledge()) io.ackCounter = 170; // approx 5us
     }
   }
 
   if(io.slotNumber == 1) {
     if(!memoryCardPort2.active()) {
-      io.receiveSize = 1;
       io.receiveData = controllerPort2.bus(data);
-      if(controllerPort2.acknowledge()) io.counter = /*transferCycles*/ + 338; // approx 9.98us
+      if(controllerPort2.acknowledge()) io.ackCounter = 338; // approx 9.98us
     }
 
     if(!controllerPort2.active()) {
-      io.receiveSize = 1;
       io.receiveData = memoryCardPort2.bus(data);
-      if(memoryCardPort2.acknowledge()) io.counter = transferCycles + 170; // approx 5us
+      if(memoryCardPort2.acknowledge()) io.ackCounter = 170; // approx 5us
     }
   }
 }

ares/ps1/peripheral/peripheral.cpp

@@ -18,19 +18,26 @@ auto Peripheral::unload() -> void {
 }
 
 auto Peripheral::main() -> void {
-  if(io.counter > 0) {
-    if(--io.counter == 0) {
-      if(!io.acknowledgeAsserted) {
+  if(io.transferCounter > 0) {
+    if(--io.transferCounter == 0) {
+      //transfer complete, set receive size to 1 byte
+      io.receiveSize = 1;
+    }
+  }
+
+  if(io.transferCounter == 0 && io.ackCounter > 0) {
+    if(--io.ackCounter == 0) {
+      if (!io.acknowledgeAsserted) {
         //Assert /ACK and fire the IRQ
         io.interruptRequest = 1;
         io.acknowledgeAsserted = 1;
         if(io.acknowledgeInterruptEnable) {
           interrupt.raise(Interrupt::Peripheral);
         }
 
-        io.counter = 96; // ACK duration is approx 96 cycles (2.84us)
+        io.ackCounter = 96; // ACK duration is approx 96 cycles (2.84us)
       } else {
-        //Deassert /ACK
+        //De-assert /ACK
         io.acknowledgeAsserted = 0;
       }
     }

ares/ps1/peripheral/peripheral.hpp

@@ -77,7 +77,8 @@ struct Peripheral : Thread, Memory::Interface {
     n16 baudrateReloadValue;
 
     //internal
-    i32 counter;
+    i32 transferCounter;
+    i32 ackCounter;
   } io;
 };
 

ares/ps1/peripheral/serialization.cpp

@@ -31,5 +31,6 @@ auto Peripheral::serialize(serializer& s) -> void {
   s(io.slotNumber);
   s(io.unknownCtrl_14_15);
   s(io.baudrateReloadValue);
-  s(io.counter);
+  s(io.transferCounter);
+  s(io.ackCounter);
 }

ares/ps1/system/serialization.cpp

@@ -1,4 +1,4 @@
-static const string SerializerVersion = "v131";
+static const string SerializerVersion = "v133";
 
 auto System::serialize(bool synchronize) -> serializer {
   serializer s;