bus/pci/vt6306: confirm some HW details [Guru]

mamedev · Feb 17, 2024 · b05ee45 · b05ee45
1 parent 4fb9be2
commit b05ee45
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/scripts/src/bus.lua b/scripts/src/bus.lua
@@ -5510,10 +5510,10 @@ if (BUSES["PCI"]~=null) then
 		MAME_DIR .. "src/devices/bus/pci/sonicvibes.h",
 		MAME_DIR .. "src/devices/bus/pci/sw1000xg.cpp",
 		MAME_DIR .. "src/devices/bus/pci/sw1000xg.h",
-		MAME_DIR .. "src/devices/bus/pci/vt6306.cpp",
-		MAME_DIR .. "src/devices/bus/pci/vt6306.h",
 		MAME_DIR .. "src/devices/bus/pci/virge_pci.cpp",
 		MAME_DIR .. "src/devices/bus/pci/virge_pci.h",
+		MAME_DIR .. "src/devices/bus/pci/vt6306.cpp",
+		MAME_DIR .. "src/devices/bus/pci/vt6306.h",
 		MAME_DIR .. "src/devices/bus/pci/wd9710_pci.cpp",
 		MAME_DIR .. "src/devices/bus/pci/wd9710_pci.h",
 		MAME_DIR .. "src/devices/bus/pci/ymp21.cpp",

diff --git a/src/devices/bus/pci/vt6306.cpp b/src/devices/bus/pci/vt6306.cpp
@@ -5,7 +5,7 @@
 VIA VT6306 generic FireWire IEEE 1394a card
 
 - paired with USB 2.0 controller on a Sunix UFC3212V 4x USB 3x Firewire card
-  (as multifunction, presumably at .3 while USB takes .0/.1/.2)
+  (as multifunction, at .3 while USB takes .0/.1/.2)
 - midway/midzeus.cpp crusnexo/thegrid uses regular IEEE 1394 ports.
 - PHY has default vendor ID=0x004063, device ID=306000, Compliance Level=1
 
@@ -31,6 +31,8 @@ vt6306_device::vt6306_device(const machine_config &mconfig, device_type type, co
 	: pci_card_device(mconfig, type, tag, owner, clock)
 {
 	set_ids(0x11063044, 0x00, 0x0c0010, 0x11063044);
+	// Sunix UFC3212V has these values
+//	set_ids(0x11063044, 0x46, 0x0c0010, 0x1106e8c1);
 }
 
 vt6306_device::vt6306_device(const machine_config &mconfig, const char *tag, device_t *owner, uint32_t clock)
@@ -62,7 +64,7 @@ void vt6306_device::device_reset()
 	pci_card_device::device_reset();
 
 	command = 0x0000;
-	// claims not having an I/O space but it definitely has one defined in BAR, huh?
+	// doc claims not having an I/O space but real HW pci.exe proved otherwise
 	command_mask = 7;
 	// Fast Back-to-Back, medium DEVSEL#
 	status = 0x0280;