From 9441b6f2cf04238f51c39d3e875ac76555ae0467 Mon Sep 17 00:00:00 2001
From: OBattler <oubattler@gmail.com>
Date: Sun, 18 May 2025 20:42:28 +0200
Subject: [PATCH 1/2] PC87309: Make the Super I/O chip relocatable.

---
 src/sio/sio_pc87309.c | 45 ++++++++++++++++++++++++++++++-------------
 1 file changed, 32 insertions(+), 13 deletions(-)

diff --git a/src/sio/sio_pc87309.c b/src/sio/sio_pc87309.c
index da53802c1..2219ede17 100644
--- a/src/sio/sio_pc87309.c
+++ b/src/sio/sio_pc87309.c
@@ -41,15 +41,19 @@ typedef struct pc87309_t {
     uint8_t   regs[48];
     uint8_t   ld_regs[256][208];
     uint8_t   pm[8];
+    uint8_t   baddr;
     uint16_t  pm_base;
     int       cur_reg;
     fdc_t    *fdc;
     serial_t *uart[2];
 } pc87309_t;
 
-static void fdc_handler(pc87309_t *dev);
-static void lpt1_handler(pc87309_t *dev);
-static void serial_handler(pc87309_t *dev, int uart);
+static void    fdc_handler(pc87309_t *dev);
+static void    lpt1_handler(pc87309_t *dev);
+static void    serial_handler(pc87309_t *dev, int uart);
+
+static void    pc87309_write(uint16_t port, uint8_t val, void *priv);
+static uint8_t pc87309_read(uint16_t port, void *priv);
 
 static void
 pc87309_pm_write(uint16_t port, uint8_t val, void *priv)
@@ -104,6 +108,26 @@ pc87309_pm_init(pc87309_t *dev, uint16_t addr)
                   pc87309_pm_read, NULL, NULL, pc87309_pm_write, NULL, NULL, dev);
 }
 
+static void
+superio_handler(pc87309_t *dev)
+{
+    io_removehandler(0x15c, 0x0002,
+                     pc87309_read, NULL, NULL, pc87309_write, NULL, NULL, dev);
+    io_removehandler(0x02e, 0x0002,
+                     pc87309_read, NULL, NULL, pc87309_write, NULL, NULL, dev);
+
+    switch (dev->baddr) {
+        case 2:
+            io_sethandler(0x15c, 0x0002,
+                          pc87309_read, NULL, NULL, pc87309_write, NULL, NULL, dev);
+            break;
+        case 3:
+            io_sethandler(0x02e, 0x0002,
+                          pc87309_read, NULL, NULL, pc87309_write, NULL, NULL, dev);
+            break;
+    }
+}
+
 static void
 fdc_handler(pc87309_t *dev)
 {
@@ -197,6 +221,7 @@ pc87309_write(uint16_t port, uint8_t val, void *priv)
                 break;
             case 0x22:
                 dev->regs[dev->cur_reg] = val & 0x7f;
+                superio_handler(dev);
                 break;
             default:
                 if (dev->cur_reg >= 0x30) {
@@ -379,6 +404,7 @@ pc87309_reset(pc87309_t *dev)
 
     dev->regs[0x20] = dev->id;
     dev->regs[0x21] = 0x04;
+    dev->regs[0x22] = dev->baddr;
 
     dev->ld_regs[0x00][0x01] = 0x01;
     dev->ld_regs[0x00][0x30] = 0x03;
@@ -453,6 +479,8 @@ pc87309_reset(pc87309_t *dev)
     serial_remove(dev->uart[0]);
     serial_remove(dev->uart[1]);
     fdc_reset(dev->fdc);
+
+    superio_handler(dev);
 }
 
 static void
@@ -472,19 +500,10 @@ pc87309_init(const device_t *info)
 
     dev->fdc = device_add(&fdc_at_nsc_device);
 
-    dev->uart[0] = device_add_inst(&ns16550_device, 1);
-    dev->uart[1] = device_add_inst(&ns16550_device, 2);
+    dev->baddr = (info->local & 0x100) ? 2 : 3;
 
     pc87309_reset(dev);
 
-    if (info->local & 0x100) {
-        io_sethandler(0x15c, 0x0002,
-                      pc87309_read, NULL, NULL, pc87309_write, NULL, NULL, dev);
-    } else {
-        io_sethandler(0x02e, 0x0002,
-                      pc87309_read, NULL, NULL, pc87309_write, NULL, NULL, dev);
-    }
-
     return dev;
 }
 

From a262a519cbba418fd918b5e1ef2828f34d5006eb Mon Sep 17 00:00:00 2001
From: OBattler <oubattler@gmail.com>
Date: Sun, 18 May 2025 20:48:29 +0200
Subject: [PATCH 2/2] PC87309: Actually fix relocation so it's implemented
 according to the PC87309 datasheet, not the PC87306 one.

---
 src/sio/sio_pc87309.c | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/src/sio/sio_pc87309.c b/src/sio/sio_pc87309.c
index 2219ede17..5e417aefb 100644
--- a/src/sio/sio_pc87309.c
+++ b/src/sio/sio_pc87309.c
@@ -116,12 +116,16 @@ superio_handler(pc87309_t *dev)
     io_removehandler(0x02e, 0x0002,
                      pc87309_read, NULL, NULL, pc87309_write, NULL, NULL, dev);
 
-    switch (dev->baddr) {
-        case 2:
+    switch (dev->regs[0x21] & 0x0b) {
+        case 0x02:
+        case 0x08:
+        case 0x0a:
             io_sethandler(0x15c, 0x0002,
                           pc87309_read, NULL, NULL, pc87309_write, NULL, NULL, dev);
             break;
-        case 3:
+        case 0x03:
+        case 0x09:
+        case 0x0b:
             io_sethandler(0x02e, 0x0002,
                           pc87309_read, NULL, NULL, pc87309_write, NULL, NULL, dev);
             break;
@@ -218,10 +222,10 @@ pc87309_write(uint16_t port, uint8_t val, void *priv)
             case 0x07:
             case 0x21:
                 dev->regs[dev->cur_reg] = val;
+                superio_handler(dev);
                 break;
             case 0x22:
                 dev->regs[dev->cur_reg] = val & 0x7f;
-                superio_handler(dev);
                 break;
             default:
                 if (dev->cur_reg >= 0x30) {
@@ -403,8 +407,7 @@ pc87309_reset(pc87309_t *dev)
     memset(dev->pm, 0x00, 0x08);
 
     dev->regs[0x20] = dev->id;
-    dev->regs[0x21] = 0x04;
-    dev->regs[0x22] = dev->baddr;
+    dev->regs[0x21] = 0x04 | dev->baddr;
 
     dev->ld_regs[0x00][0x01] = 0x01;
     dev->ld_regs[0x00][0x30] = 0x03;
@@ -500,7 +503,7 @@ pc87309_init(const device_t *info)
 
     dev->fdc = device_add(&fdc_at_nsc_device);
 
-    dev->baddr = (info->local & 0x100) ? 2 : 3;
+    dev->baddr = (info->local & 0x100) ? 8 : 9;
 
     pc87309_reset(dev);