isa8 zxbus card

Author: holub

scripts/src/bus.lua

@@ -1896,6 +1896,8 @@ if (BUSES["ISA"]~=null) then
 		MAME_DIR .. "src/devices/bus/isa/xsu_cards.h",
 		MAME_DIR .. "src/devices/bus/isa/xtide.cpp",
 		MAME_DIR .. "src/devices/bus/isa/xtide.h",
+		MAME_DIR .. "src/devices/bus/isa/zxbus_adapter.cpp",
+		MAME_DIR .. "src/devices/bus/isa/zxbus_adapter.h",
 	}
 end
 

src/devices/bus/isa/isa_cards.cpp

@@ -90,6 +90,7 @@
 #include "finalchs.h"
 #include "hpblp.h"
 #include "opus100pm.h"
+#include "zxbus_adapter.h"
 
 
 void pc_isa8_cards(device_slot_interface &device)
@@ -148,6 +149,7 @@ void pc_isa8_cards(device_slot_interface &device)
 	device.option_add("acb2072", ACB2072);
 	device.option_add("3xtwin", ISA8_3XTWIN);
 	device.option_add("opus108pm", ISA8_OPUS108PM);
+	device.option_add("zxbus_adapter", ISA8_ZXBUS);
 }
 
 void pc_isa16_cards(device_slot_interface &device)

src/devices/bus/isa/zxbus_adapter.cpp

@@ -0,0 +1,34 @@
+// license:BSD-3-Clause
+// copyright-holders:Andrei I. Holub
+#include "emu.h"
+#include "zxbus_adapter.h"
+
+DEFINE_DEVICE_TYPE(ISA8_ZXBUS, zxbus_adapter_device, "zxbus_adapter", "ISA8 to ZXBUS Adapter")
+
+zxbus_adapter_device::zxbus_adapter_device(const machine_config &mconfig, const char *tag, device_t *owner, uint32_t clock)
+	: device_t(mconfig, ISA8_ZXBUS, tag, owner, clock)
+	, device_isa8_card_interface(mconfig, *this)
+	, m_isa_io_view(*this, "isa_io_view")
+	, m_zxbus(*this, "zxbus")
+{
+}
+
+void zxbus_adapter_device::map_io(address_map &map)
+{
+	map.unmap_value_high();
+	map(0x0000, 0xffff).view(m_isa_io_view);
+	m_zxbus->set_io_space(m_isa_io_view[0], m_isa_io_view[0]);
+	m_isa_io_view.select(0);
+}
+
+void zxbus_adapter_device::device_start()
+{
+	set_isa_device();
+	m_isa->install_device(0x0000, 0xffff, *this, &zxbus_adapter_device::map_io);
+}
+
+void zxbus_adapter_device::device_add_mconfig(machine_config &config)
+{
+	ZXBUS(config, m_zxbus, 0);
+	ZXBUS_SLOT(config, "slot", 0, m_zxbus, zxbus_cards, nullptr);
+}

src/devices/bus/isa/zxbus_adapter.h

@@ -0,0 +1,29 @@
+// license:BSD-3-Clause
+// copyright-holders:Andrei I. Holub
+#ifndef MAME_BUS_ISA_ZXBUS_H
+#define MAME_BUS_ISA_ZXBUS_H
+
+#include "isa.h"
+
+#include "bus/spectrum/zxbus.h"
+
+class zxbus_adapter_device: public device_t, public device_isa8_card_interface
+{
+public:
+	zxbus_adapter_device(const machine_config &mconfig, const char *tag, device_t *owner, uint32_t clock);
+
+protected:
+	virtual void device_start() override ATTR_COLD;
+	virtual void device_add_mconfig(machine_config &config) override ATTR_COLD;
+
+	void map_io(address_map &map) ATTR_COLD;
+
+private:
+	memory_view m_isa_io_view;
+	required_device<zxbus_device> m_zxbus;
+
+};
+
+DECLARE_DEVICE_TYPE(ISA8_ZXBUS, zxbus_adapter_device)
+
+#endif // MAME_BUS_ISA_ZXBUS_H

src/mame/sinclair/sprinter.cpp

@@ -46,7 +46,6 @@ Sprinter Sp2000 (Peters Plus Ltd)
 #include "bus/pc_kbd/pc_kbdc.h"
 #include "bus/rs232/hlemouse.h"
 #include "bus/rs232/rs232.h"
-#include "bus/spectrum/zxbus.h"
 #include "cpu/z80/z84c015.h"
 #include "machine/ds128x.h"
 #include "sound/ay8910.h"
@@ -79,8 +78,7 @@ class sprinter_state : public spectrum_128_state
 	sprinter_state(const machine_config &mconfig, device_type type, const char *tag)
 		: spectrum_128_state(mconfig, type, tag)
 		, m_maincpu(*this, "maincpu")
-		, m_isa(*this, "isa%u", 0U)
-		, m_isa_io_view(*this, "isa_io_view")
+		, m_isa(*this, "isa8%u", 1U)
 		, m_rtc(*this, "rtc")
 		, m_ata(*this, "ata%u", 1U)
 		, m_beta(*this, BETA_DISK_TAG)
@@ -114,7 +112,6 @@ class sprinter_state : public spectrum_128_state
 	void map_io(address_map &map) ATTR_COLD;
 	void map_mem(address_map &map) ATTR_COLD;
 	void map_fetch(address_map &map) ATTR_COLD;
-	void map_isa_io(address_map &map) ATTR_COLD;
 	u8 m1_r(offs_t offset);
 	void cio_dtrb_w(int state);
 	u8 joy_ctrl_r(int num);
@@ -141,7 +138,6 @@ class sprinter_state : public spectrum_128_state
 
 	required_device<z84c015_device> m_maincpu;
 	required_device_array<isa8_device, 2> m_isa;
-	memory_view m_isa_io_view;
 
 private:
 	enum accel_state : u8
@@ -1399,19 +1395,8 @@ void sprinter_state::init_taps()
 	});
 }
 
-void sprinter_state::map_isa_io(address_map &map)
-{
-	map.unmap_value_high();
-	map(0x0000, 0xffff).view(m_isa_io_view);
-	subdevice<zxbus_device>("zxbus")->set_io_space(m_isa_io_view[0], m_isa_io_view[0]);
-	m_isa_io_view.select(0);
-}
-
 void sprinter_state::machine_start()
 {
-	address_space &m_isa_io = m_isa[0]->space(isa8_device::AS_ISA_IO);
-	m_isa_io.install_device(0x0000, 0xffff, *this, &sprinter_state::map_isa_io);
-
 	m_isa[0]->space(isa8_device::AS_ISA_IO).unmap_value_high();
 	m_isa[1]->space(isa8_device::AS_ISA_IO).unmap_value_high();
 
@@ -1907,12 +1892,11 @@ void sprinter_state::sprinter(machine_config &config)
 
 	ISA8(config, m_isa[0], X_SP / 5);
 	m_isa[0]->set_custom_spaces();
-	zxbus_device &zxbus(ZXBUS(config, "zxbus", 0));
-	ZXBUS_SLOT(config, "zxbus2isa", 0, zxbus, zxbus_cards, nullptr);
+	ISA8_SLOT(config, "isa1", 0, m_isa[0], pc_isa8_cards, "zxbus_adapter", false);
 
 	ISA8(config, m_isa[1], X_SP / 5);
 	m_isa[1]->set_custom_spaces();
-	ISA8_SLOT(config, "isa8", 0, m_isa[1], pc_isa8_cards, nullptr, false);
+	ISA8_SLOT(config, "isa2", 0, m_isa[1], pc_isa8_cards, nullptr, false);
 
 	m_screen->set_raw(X_SP / 3, SPRINT_WIDTH, SPRINT_HEIGHT, { 0, SPRINT_XVIS - 1, 0, SPRINT_YVIS - 1 });
 	m_screen->set_screen_update(FUNC(sprinter_state::screen_update));