--- nono/vm/spc.cpp	2026/04/29 17:05:09	1.1.1.12
+++ nono/vm/spc.cpp	2026/04/29 17:05:28	1.1.1.16
@@ -10,45 +10,26 @@
 
 #include "spc.h"
 #include "config.h"
-#include "interrupt.h"
 #include "mainapp.h"
-#include "mpu.h"
 #include "pedec.h"
 #include "scheduler.h"
-#include "sram.h"
+#include "scsidev.h"
 
 // time 時間後に呼び出すコールバックをセットする。
 // func の書式が面倒なのを省略して書きたいため。
 #define CallAfter(func_, name_, time_)	do { \
-	event.func = ToEventCallback(&SPCDevice::func_); \
-	event.time = time_; \
-	event.SetName(name_); \
-	gScheduler->RestartEvent(event); \
+	phase_event.func = ToEventCallback(&SPCDevice::func_); \
+	phase_event.time = time_; \
+	phase_event.SetName(name_); \
+	scheduler->RestartEvent(phase_event); \
 } while (0)
 
-// グローバル参照用
-SPCDevice *gSPC;
-
 // コンストラクタ
 SPCDevice::SPCDevice()
-	: inherited("SPC")
+	: inherited(OBJ_SPC)
 {
-	// これらは XIOSpace で必要なので、LUNA ではダミー。
-	// もうちょっといい方法を考えたほうがいい。
-	devaddr = 0xe96000;
-	devlen = 0x2000;
-
-	SetConfigName("spc0");
 	myid = -1;
 
-	// 汎用イベント
-	// 名前とコールバック関数は都度セットする。
-	event.Regist("SPC phase");
-
-	// TC カウンタイベント
-	event_tc.code = 0;
-	event_tc.Regist("SPC Timer");
-
 	monitor.func = ToMonitorCallback(&SPCDevice::MonitorUpdate);
 	// サイズは Init で決まる
 	monitor.Regist(ID_MONITOR_SPC);
@@ -57,69 +38,84 @@ SPCDevice::SPCDevice()
 // デストラクタ
 SPCDevice::~SPCDevice()
 {
-	gSPC = NULL;
 }
 
 bool
-SPCDevice::Init()
+SPCDevice::Create()
 {
-	// 実行中に別インスタンスを頻繁に参照しないようにここで vmtype を覚えておく
-	vmtype = gMainApp.GetVMType();
-
-	// 本体の SCSI ID は SPC(MB89352) の BDID レジスタに ID が書き込まれる
-	// のを待たずに、最初から知っていることにする。本来 (少なくとも真っ当な
-	// システムでは) イニシエータを含むすべての SCSI 機器の SCSI ID は
-	// 電源を入れる前に、人間が一々手作業で重複なく設定しているはずなので。
-	// X68030 は SRAM $ED0070 下位3ビット。LUNA は 7 固定のようだ。
-	uint myid0;
-	if (vmtype == VMTYPE_X68030) {
-		myid0 = gSRAM->Peek8(0xed0070) & 7;
-	} else {
-		myid0 = 7;
+	if (inherited::Create() == false) {
+		return false;
 	}
 
-	// モニタには、接続している SCSI デバイスだけを表示する。
-	// そのため ID ではなく前から何番目のデバイスかのリストが欲しい。
-	for (int id = 0; id < 8; id++) {
-		if (id == myid0) {
-			connected_devices.push_back(this);
-		} else if (target[id].get()) {
-			connected_devices.push_back(target[id].get());
-		}
+	scsi.reset(new SCSIHostDevice(this, "spc0"));
+	if ((bool)scsi == false) {
+		return false;
 	}
+	scsi->SetCallback(
+		ToBusFreeCallback(&SPCDevice::BusFreeCallback),
+		ToSCSIHostCallback(&SPCDevice::SelectionAckCallback),
+		ToSCSIHostCallback(&SPCDevice::TransferReqCallback));
+
+	return true;
+}
 
-	// この親の Init() は connected_devices 作成後に呼ぶ
+// 初期化
+bool
+SPCDevice::Init()
+{
 	if (inherited::Init() == false) {
 		return false;
 	}
 
+	// 実行中に別インスタンスを頻繁に参照しないようにここで vmtype を覚えておく
+	vmtype = gMainApp.GetVMType();
+
 	// 入力クロックは機種によって異なる。
 	// X68k は 5MHz が入っている。
-	// LUNA-I はおそらく 6.144MHz だと思われる。LUNA88K は未確認。
+	// LUNA-I はおそらく 6.144MHz だと思われる。LUNA-88K は未確認。
 	const ConfigItem& item = gConfig->Find("!spc-tCLF");
 	t_CLF = item.AsInt();
 
 	// アクセスウェイト
-	if (vmtype == VMTYPE_X68030) {
+	if (vmtype == VMType::X68030) {
 		// InsideOut p.135
-		read_wait  = 18;
-		write_wait = 19;
+		read_wait  = busdata::Wait(18 * 40_nsec);
+		write_wait = busdata::Wait(19 * 40_nsec);
 	} else {
 		// 他機種は未調査だけど、判明するまでの間とりあえず
 		// ノーウェイトでない適当な値を入れておく。
-		read_wait  = 18;
-		write_wait = 19;
+		read_wait  = busdata::Wait(18 * 40_nsec);
+		write_wait = busdata::Wait(19 * 40_nsec);
 	}
 
 	// 割り込み信号線の接続先
-	if (vmtype == VMTYPE_X68030) {
+	if (vmtype == VMType::X68030) {
 		// X68030 は PEDEC (I/O コントローラ) にカスケード
-		interrupt = gPEDEC;
+		interrupt = GetPEDECDevice();
 	} else {
-		interrupt = gInterrupt;
+		interrupt = GetInterruptDevice();
 	}
 
-	monitor.SetSize(70, 15 + connected_devices.size());
+	// 汎用イベント
+	// 名前とコールバック関数は都度セットする。
+	phase_event.SetName("SPC Phase");
+	scheduler->RegistEvent(phase_event);
+
+	// TC カウンタイベント
+	timer_event.code = 0;
+	timer_event.SetName("SPC Timer");
+	scheduler->RegistEvent(timer_event);
+
+	// モニタに表示するアドレス
+	if (vmtype == VMType::X68030) {
+		baseaddr = 0xe96001;
+		regoffset = 2;
+	} else {
+		baseaddr = 0xe1000000;
+		regoffset = 4;
+	}
+	// 「イニシエータ以外のターゲット数」はここでは確定している。
+	monitor.SetSize(80, 15 + scsi->GetOwnedBus()->GetTargetCount() + 1);
 
 	return true;
 }
@@ -128,7 +124,7 @@ SPCDevice::Init()
 void
 SPCDevice::ResetHard(bool poweron)
 {
-	putlog(1, "リセット");
+	putlog(1, "Reset");
 
 	// 初期値 (p.61)
 	spc.sctl = SPC::SCTL_RESET;
@@ -146,190 +142,204 @@ SPCDevice::ResetHard(bool poweron)
 
 	// 内部状態
 	MakeIntsMask();
+	ChangeInterrupt();
 	prev_ints = (uint32)-1;
 	prev_ssts = (uint32)-1;
 	set_atn_in_selection = false;
 	transfer_command_running = false;
-	gScheduler->StopEvent(event);
-	gScheduler->StopEvent(event_tc);
+	scheduler->StopEvent(phase_event);
+	scheduler->StopEvent(timer_event);
 
 	last_busfree_time = 0;
 }
 
-uint64
-SPCDevice::Read(uint32 offset)
+busdata
+SPCDevice::ReadPort(uint32 offset)
 {
-	uint8 data;
-
-	gMPU->AddCycle(read_wait);
+	busdata data;
 
 	switch (offset) {
 	 case SPC::BDID:
 		data = 1 << myid;
-		putlog(3, "BDID -> $%02x", data);
-		return data;
+		putlog(3, "BDID -> $%02x", data.Data());
+		break;
 
 	 case SPC::SCTL:
 		data = spc.sctl;
-		putlog(3, "SCTL -> $%02x", data);
-		return data;
+		putlog(3, "SCTL -> $%02x", data.Data());
+		break;
 
 	 case SPC::SCMD:
 		data = spc.scmd;
-		putlog(3, "SCMD -> $%02x", data);
-		return data;
+		putlog(3, "SCMD -> $%02x", data.Data());
+		break;
 
 	 case SPC::TMOD:
 		// MB89352 には存在しないレジスタ。
 		// 下位2ビットは %0 らしいので意味はないだろうけど真似ておく。
-		return 0xfc;
+		data = 0xfc;
+		break;
 
 	 case SPC::INTS:
 		data = GetINTS();
 		if (loglevel >= 3) {
-			if (event_tc.IsRunning()) {
+			uint32 new_ints = data.Data();
+			if (timer_event.IsRunning()) {
 				// イベント中は値が変わった時だけ表示
-				if (prev_ints != data) {
-					putlogn("INTS -> $%02x", data);
-					prev_ints = data;
+				if (prev_ints != new_ints) {
+					putlogn("INTS -> $%02x", new_ints);
+					prev_ints = new_ints;
 				}
 			} else {
-				putlogn("INTS -> $%02x", data);
+				putlogn("INTS -> $%02x", new_ints);
 				prev_ints = -1U;
 			}
 		}
-		return data;
+		break;
 
 	 case SPC::PSNS:
 		data = GetPSNS();
-		putlog(3, "PSNS -> $%02x", data);
-		return data;
+		putlog(3, "PSNS -> $%02x", data.Data());
+		break;
 
 	 case SPC::SSTS:
 		data = GetSSTS();
 		if (loglevel >= 3) {
-			if (event_tc.IsRunning()) {
+			uint32 new_ssts = data.Data();
+			if (timer_event.IsRunning()) {
 				// イベント中は値が変わった時だけ表示
-				if (prev_ssts != data) {
-					putlogn("SSTS -> $%02x", data);
-					prev_ssts = data;
+				if (prev_ssts != new_ssts) {
+					putlogn("SSTS -> $%02x", new_ssts);
+					prev_ssts = new_ssts;
 				}
 			} else {
-				putlogn("SSTS -> $%02x", data);
+				putlogn("SSTS -> $%02x", new_ssts);
 				prev_ssts = -1U;
 			}
 		}
-		return data;
+		break;
 
 	 case SPC::SERR:
 		data = GetSERR();
-		putlog(3, "SERR -> $%02x", data);
-		return data;
+		putlog(3, "SERR -> $%02x", data.Data());
+		break;
 
 	 case SPC::PCTL:
 		data = GetPCTL();
-		putlog(3, "PCTL -> $%02x", data);
-		return data;
+		putlog(3, "PCTL -> $%02x", data.Data());
+		break;
 
 	 case SPC::MBC:
-		putlog(0, "MBC 未実装読み込み");
-		return 0xff;
+		putlog(0, "Read MBC (NOT IMPLEMENTED)");
+		data = 0x0f;
+		break;
 
 	 case SPC::DREG:
-		return ReadDREG();
+		data = ReadDREG();
+		break;
 
 	 case SPC::TEMP:
 		data = spc.temp_in;
-		putlog(3, "TEMP -> $%02x", data);
-		return data;
+		putlog(3, "TEMP -> $%02x", data.Data());
+		break;
 
 	 case SPC::TCH:
 		data = spc.tc >> 16;
-		putlog(3, "TCH  -> $%02x", data);
-		return data;
+		putlog(3, "TCH  -> $%02x", data.Data());
+		break;
 
 	 case SPC::TCM:
 		data = (spc.tc >> 8) & 0xff;
-		putlog(3, "TCM  -> $%02x", data);
-		return data;
+		putlog(3, "TCM  -> $%02x", data.Data());
+		break;
 
 	 case SPC::TCL:
 		data = spc.tc & 0xff;
-		putlog(3, "TCL  -> $%02x", data);
-		return data;
+		putlog(3, "TCL  -> $%02x", data.Data());
+		break;
 
 	 case SPC::EXBF:
 		// MB89352 には存在しないレジスタ
-		return 0xff;
+		data = 0xff;
+		break;
 
 	 default:
-		__unreachable();
+		VMPANIC("corrupted offset=%u", offset);
 	}
+
+	data |= read_wait;
+	data |= BusData::Size1;
+	return data;
 }
 
-uint64
-SPCDevice::Write(uint32 offset, uint32 data)
+busdata
+SPCDevice::WritePort(uint32 offset, uint32 data)
 {
-	gMPU->AddCycle(write_wait);
+	busdata r;
 
 	switch (offset) {
 	 case SPC::BDID:
+	 {
+		uint cur_myid = scsi->GetMyID();
 		putlog(3, "BDID <- $%02x", data);
-		if (myid == data) {
+		if (data == cur_myid) {
 			// 同じなら何もしない
-			return 0;
+			break;
 		}
-		if (myid == -1) {
-			// 初回なのでバスに参加
+		if ((int)cur_myid < 0) {
+			// 初回なのでバスに参加。myid はこちらでもコピーを覚えておく。
 			myid = data;
-			scsibus->Attach(this, myid);
+			if (scsi->GetOwnedBus()->AttachDevice(scsi.get(), myid) == false) {
+				putlog(0, "BDID %u conflicts with existing devices!", myid);
+			}
 		} else {
 			// そうでなければ移動?
-			putlog(0, "BDID 変更 %d to %d not implemented", myid, data);
-			return 0;
+			putlog(0, "BDID change %u to %u (NOT IMPLEMENTED)", cur_myid, data);
 		}
-		return 0;
+		break;
+	 }
 
 	 case SPC::SCTL:
 		WriteSCTL(data);
-		return 0;
+		break;
 
 	 case SPC::SCMD:
 		WriteSCMD(data);
-		return 0;
+		break;
 
 	 case SPC::TMOD:
 		// MB89352 には存在しないレジスタ
-		return 0;
+		break;
 
 	 case SPC::INTS:
 		WriteINTS(data);
-		return 0;
+		break;
 
 	 case SPC::SDGC:
 		spc.sdgc = data;
 		putlog(3, "SDGC <- $%02x", spc.sdgc);
-		return 0;
+		break;
 
 	 case SPC::SSTS:
 		// XXX 書き込みはどうなる?
-		return 0;
+		break;
 
 	 case SPC::SERR:
 		// XXX 書き込みはどうなる?
-		return 0;
+		break;
 
 	 case SPC::PCTL:
 		WritePCTL(data);
-		return 0;
+		break;
 
 	 case SPC::MBC:
 		// XXX 書き込みはどうなる?
-		return 0;
+		break;
 
 	 case SPC::DREG:
 		putlog(3, "DREG <- $%02x", data);
-		return WriteDREG(data);
+		r = WriteDREG(data);
+		break;
 
 	 case SPC::TEMP:
 		spc.temp_out = data;
@@ -343,39 +353,43 @@ SPCDevice::Write(uint32 offset, uint32 d
 		// SPC はセレクションタイムアウトしたからと言って、データバスに出した
 		// ID を自動的に取り下げたりはしないので、TEMP に $00 を書き込むことで
 		// データバスをクリアする、だったんでは。
-		if (0 && GetPhase() == SCSI::Phase::Selection) {
-			NegateSEL();
+		if (0 && scsi->GetPhase() == SCSI::Phase::Selection) {
+			scsi->NegateSEL();
 		}
 		// Out 方向ならバスに出す
-		if (GetIO() == false) {
-			SetData(spc.temp_out);
+		if (scsi->GetIO() == false) {
+			scsi->SetData(spc.temp_out);
 		}
-		return 0;
+		break;
 
 	 case SPC::TCH:
 		spc.tc = (spc.tc & 0x00ffff) | (data << 16);
 		putlog(3, "TCH  <- $%02x (TC=$%06x)", data, spc.tc);
-		return 0;
+		break;
 	 case SPC::TCM:
 		spc.tc = (spc.tc & 0xff00ff) | (data << 8);
 		putlog(3, "TCM  <- $%02x (TC=$%06x)", data, spc.tc);
-		return 0;
+		break;
 	 case SPC::TCL:
 		spc.tc = (spc.tc & 0xffff00) | data;
 		putlog(3, "TCL  <- $%02x (TC=$%06x)", data, spc.tc);
-		return 0;
+		break;
 
 	 case SPC::EXBF:
 		// MB89352 には存在しないレジスタ
-		return 0;
+		break;
 
 	 default:
-		__unreachable();
+		VMPANIC("corrupted offset=%u", offset);
 	}
+
+	r |= write_wait;
+	r |= BusData::Size1;
+	return r;
 }
 
-uint64
-SPCDevice::Peek(uint32 offset)
+busdata
+SPCDevice::PeekPort(uint32 offset)
 {
 	switch (offset) {
 	 case SPC::BDID:
@@ -408,15 +422,15 @@ SPCDevice::Peek(uint32 offset)
 		return GetPCTL();
 
 	 case SPC::MBC:
-		return 0xff;
+		return 0x0f;
 
 	 case SPC::DREG:
 		// X68k は SPC の DREQ 信号から DTACK 信号を作っているため、
 		// DREQ 信号を出さないプログラム転送モードでは DREG へのアクセスが
 		// すべてバスエラーになってしまう。
-		if (vmtype == VMTYPE_X68030) {
+		if (vmtype == VMType::X68030) {
 			if ((spc.scmd & SPC::SCMD_PROGRAM)) {
-				return (uint64)-1;
+				return BusData::BusErr;
 			}
 		}
 		return spc.dreg.Peek(0);
@@ -438,15 +452,22 @@ SPCDevice::Peek(uint32 offset)
 		return 0xff;
 
 	 default:
-		__unreachable();
+		return 0xff;
 	}
 }
 
+bool
+SPCDevice::PokePort(uint32 offset, uint32 data)
+{
+	return false;
+}
+
 void
 SPCDevice::MonitorUpdate(Monitor *, TextScreen& screen)
 {
 	int x;
 	int y;
+	SCSIBus *bus;
 	uint32 sctl;
 	uint32 scmd;
 	uint32 ints;
@@ -459,6 +480,7 @@ SPCDevice::MonitorUpdate(Monitor *, Text
 
 	screen.Clear();
 
+	bus  = scsi->GetOwnedBus();
 	sctl = spc.sctl;
 	scmd = spc.scmd;
 	ints = GetINTS();
@@ -469,33 +491,35 @@ SPCDevice::MonitorUpdate(Monitor *, Text
 	pctl = GetPCTL();
 	tc   = spc.tc;
 
+#define A(n)	(baseaddr + (n) * regoffset)
+
 	y = 0;
-	screen.Print(0, y++, "BDID: $%02x (ID=%d)", (1 << myid), myid);
-	screen.Print(0, y++, "SCTL: $%02x", sctl);
-	screen.Print(0, y++, "SCMD: $%02x", scmd);
-	screen.Print(0, y++, "INTS: $%02x", ints);
-	screen.Print(0, y++, "PSNS: $%02x", psns);
-	screen.Print(0, y++, "SDGC: $%02x", sdgc);
-	screen.Print(0, y++, "SSTS: $%02x", ssts);
-	screen.Print(0, y++, "SERR: $%02x", serr);
-	screen.Print(0, y++, "PCTL: $%02x", pctl);
-	screen.Print(0, y++, "MBC : $XX");
-	screen.Print(0, y,   "DREG: ");
+	screen.Print(0, y++, "$%08x BDID: $%02x (ID=%u)", A(0), (1 << myid), myid);
+	screen.Print(0, y++, "$%08x SCTL: $%02x", A(1), sctl);
+	screen.Print(0, y++, "$%08x SCMD: $%02x", A(2), scmd);
+	screen.Print(0, y++, "$%08x INTS: $%02x", A(4), ints);
+	screen.Print(0, y++, "$%08x PSNS: $%02x", A(5), psns);
+	screen.Print(0, y++, "$%08x SDGC: $%02x", A(5), sdgc);
+	screen.Print(0, y++, "$%08x SSTS: $%02x", A(6), ssts);
+	screen.Print(0, y++, "$%08x SERR: $%02x", A(7), serr);
+	screen.Print(0, y++, "$%08x PCTL: $%02x", A(8), pctl);
+	screen.Print(0, y++, "$%08x MBC : $XX", A(9));
+	screen.Print(0, y,   "$%08x DREG: ", A(10));
 	{
 		uint i;
 		for (i = 0; i < spc.dreg.Length(); i++) {
-			screen.Print(6 + i * 4, y, "$%02x", spc.dreg.Peek(i));
+			screen.Print(16 + i * 4, y, "$%02x", spc.dreg.Peek(i));
 		}
 		for (; i < 8; i++) {
-			screen.Puts(6 + i * 4, y, "---");
+			screen.Puts(16 + i * 4, y, "---");
 		}
 	}
 	y++;
-	screen.Print(0, y++, "TEMP: $%02x(IN) $%02x(OUT)",
+	screen.Print(0, y++, "$%08x TEMP: $%02x(IN) $%02x(OUT)", A(11),
 		spc.temp_in, spc.temp_out);
-	screen.Print(0, y++, "TC  : $%06x", tc);
+	screen.Print(0, y++, "$%08x TC  : $%06x", A(12), tc);
 
-	x = 10;
+	x = 20;
 	y = 1;
 
 	// SCTL
@@ -575,32 +599,20 @@ SPCDevice::MonitorUpdate(Monitor *, Text
 	y++;
 
 	// バス状態
-	x = 52;
+	x = 62;
 	y = 0;
-	SCSI::Phase ph {};
-	uint8 rst = 0;
-	uint8 bsy = 0;
-	uint8 sel = 0;
-	uint8 msg = 0;
-	uint8 cd = 0;
-	uint8 io = 0;
-	uint8 atn = 0;
-	uint8 req = 0;
-	uint8 ack = 0;
-	uint8 data = 0;
-	if (bus) {
-		ph  = GetPhase();
-		rst = GetRST();
-		bsy = GetBSY();
-		sel = GetSEL();
-		msg = GetMSG();
-		cd  = GetCD();
-		io  = GetIO();
-		atn = GetATN();
-		req = GetREQ();
-		ack = GetACK();
-		data = GetData();
-	}
+	SCSI::Phase ph = bus->GetPhase();
+	uint8 rst = bus->GetRST();
+	uint8 bsy = bus->GetBSY();
+	uint8 sel = bus->GetSEL();
+	uint8 msg = bus->GetMSG();
+	uint8 cd  = bus->GetCD();
+	uint8 io  = bus->GetIO();
+	uint8 atn = bus->GetATN();
+	uint8 req = bus->GetREQ();
+	uint8 ack = bus->GetACK();
+	uint8 data = bus->GetData();
+
 	screen.Print(x, y++, "Phase: %s", SCSI::GetPhaseName(ph));
 	screen.Puts(x, y++, TA::OnOff(rst), "RST");
 	screen.Puts(x, y++, TA::OnOff(bsy), "BSY");
@@ -624,7 +636,7 @@ SPCDevice::MonitorUpdate(Monitor *, Text
 	y = 2;
 	for (int i = 0; i < 8; i++) {
 		if ((bsy & (1 << i))) {
-			screen.Print(x + 4 + 8 - i, y, "%d", i);
+			screen.Print(x + 4 + 8 - i, y, "%u", i);
 		} else {
 			screen.Print(x + 4 + 8 - i, y, "_");
 		}
@@ -632,7 +644,7 @@ SPCDevice::MonitorUpdate(Monitor *, Text
 	y++;
 	for (int i = 0; i < 8; i++) {
 		if ((sel & (1 << i))) {
-			screen.Print(x + 4 + 8 - i, y, "%d", i);
+			screen.Print(x + 4 + 8 - i, y, "%u", i);
 		} else {
 			screen.Print(x + 4 + 8 - i, y, "_");
 		}
@@ -641,7 +653,7 @@ SPCDevice::MonitorUpdate(Monitor *, Text
 	// 転送フェーズ中なら現在の XferPhase
 	y += 2;
 	if (ph == SCSI::Phase::Transfer) {
-		screen.Puts(x + 5, y, SCSI::GetXferPhaseName(GetXfer()));
+		screen.Puts(x + 5, y, SCSI::GetXferPhaseName(scsi->GetXfer()));
 	}
 
 	// デバイス一覧
@@ -653,20 +665,20 @@ SPCDevice::MonitorUpdate(Monitor *, Text
 	x = 0;
 	y = 14;
 	screen.Print(x, y++, "ID Device     Command");
-	for (const auto d : connected_devices) {
+	for (const auto d : bus->GetConnectedDevices()) {
 		std::string desc = SCSI::GetDevTypeName(d->GetDevType());
 
-		SCSIDisk *disk = dynamic_cast<SCSIDisk*>(d);
+		const SCSIDisk *disk = dynamic_cast<const SCSIDisk*>(d);
 		if (disk && disk->IsMediumLoaded()) {
-			desc += string_format("(%dMB)",
-				(int)(disk->GetSize() / 1024 / 1024));
+			desc += string_format("(%uMB)",
+				(uint)(disk->GetSize() / 1024 / 1024));
 		}
 
 		// 本体 ID は BDID が設定される前でもエミュレータ的に知ってはいるが、
 		// BDID への書き込みがされなかったりおかしかったりしたら (上の BDID
 		// 欄でも分かるけど) ここも反映されるようにしておく。
 		char idc;
-		if (d->GetMyID() >= 0) {
+		if ((int)d->GetMyID() >= 0) {
 			idc = '0' + d->GetMyID();
 		} else {
 			idc = '-';
@@ -678,26 +690,23 @@ SPCDevice::MonitorUpdate(Monitor *, Text
 	// 実行中のコマンド
 	x = 14;
 	y = 15;
-	if (bus) {
-		SCSIDevice *d = bus->GetConnectedTarget();
-		SCSITarget *t = dynamic_cast<SCSITarget*>(d);
-		if (t) {
-			std::vector<uint8> cmds = t->GetCmdSeq();
-			if (cmds.size() > 0) {
-				// この ID のデバイス行に表示したい
-				// XXX もうちょっとマシにしたい
-				int i = 0;
-				for (; i < connected_devices.size(); i++) {
-					if (d == connected_devices[i]) {
-						break;
-					}
+	SCSITarget *t = bus->GetSelectedTarget();
+	if (t) {
+		std::vector<uint8> cmds = t->GetCmdSeq();
+		if (cmds.size() > 0) {
+			// この ID のデバイス行に表示したい
+			// XXX もうちょっとマシにしたい
+			const auto connected_devices = bus->GetConnectedDevices();
+			int i = 0;
+			for (; i < connected_devices.size(); i++) {
+				if (t == connected_devices[i]) {
+					break;
 				}
-				const char *cmdname = SCSI::GetCommandName(cmds[0]);
-				screen.Print(x, y + i, "$%02x %s", cmds[0], cmdname ?: "");
 			}
+			const char *cmdname = SCSI::GetCommandName(cmds[0]);
+			screen.Print(x, y + i, "$%02x %s", cmds[0], cmdname ?: "");
 		}
 	}
-
 }
 
 // レジスタをビットごとに表示する。MonitorUpdate の下請け。
@@ -734,7 +743,7 @@ SPCDevice::WriteSCTL(uint32 data)
 		putlog(2, "Control Reset");
 	}
 	if ((spc.sctl & SPC::SCTL_DIAG_MODE)) {
-		putlog(0, "SCTL 未実装 Diag Mode");
+		putlog(0, "SCTL Diag Mode (NOT IMPLEMENTED)");
 	}
 
 	// 割り込みマスクを更新して信号線の状態を変える
@@ -769,26 +778,26 @@ SPCDevice::WriteSCMD(uint32 data)
 		if ((spc.scmd & SPC::SCMD_RST_OUT)) {
 			// 0 -> 1
 			// すでに RST が立ってたらどうするか
-			if (GetRST() == false) {
+			if (scsi->GetRST() == false) {
 				putlog(1, "SCMD RST Out");
 				BusRelease();
-				AssertRST();
+				scsi->AssertRST();
 				SetINTS(SPC::INTS_RESET_COND);
 			}
 		} else {
 			// 1 -> 0
 			// 特にすることはない?
-			NegateRST();
+			scsi->NegateRST();
 		}
 	}
 
 	switch (spc.scmd >> 5) {
 	 case 0:
-		putlog(2, "SCMD Bus Release コマンド");
+		putlog(2, "SCMD Bus Release command");
 		if ((spc.scmd & SPC::SCMD_INTERCEPT)) {
 			// BusRelease コマンドの発行と共に Intercept ビットが立っていたら
 			// Service Required 割り込みをリセットする?
-			putlog(0, "SCMD Intercept with BusRelease not implemented");
+			putlog(0, "SCMD Intercept with BusRelease (NOT IMPLEMENTED)");
 		}
 		BusFree();
 		break;
@@ -796,7 +805,7 @@ SPCDevice::WriteSCMD(uint32 data)
 		SelectCommand();
 		break;
 	 case 2:
-		putlog(1, "SCMD Reset ATN コマンド (未実装)");
+		putlog(1, "SCMD Reset ATN command (NOT IMPLEMENTED)");
 		break;
 	 case 3:
 		SetATNCommand();
@@ -805,16 +814,16 @@ SPCDevice::WriteSCMD(uint32 data)
 		TransferCommand();
 		break;
 	 case 5:
-		putlog(2, "SCMD Transfer Pause コマンド (未実装)");
+		putlog(2, "SCMD Transfer Pause command (NOT IMPLEMENTED)");
 		break;
 	 case 6:
-		putlog(2, "SCMD Reset ACK/REQ コマンド");
+		putlog(2, "SCMD Reset ACK/REQ command");
 		// イニシエータなら ACK を、ターゲットなら REQ をリセットする。
 		// ターゲットモードはサポートしないので、操作するのは ACK のみ。
-		NegateACK();
+		scsi->NegateACK();
 		break;
 	 case 7:
-		putlog(2, "SCMD Set ACK/REQ コマンド");
+		putlog(2, "SCMD Set ACK/REQ command");
 		// イニシエータなら ACK を、ターゲットなら REQ をセットする。
 		// ターゲットモードはサポートしないので、操作するのは ACK のみ。
 		if ((spc.scmd & SPC::SCMD_INTERCEPT)) {
@@ -823,10 +832,10 @@ SPCDevice::WriteSCMD(uint32 data)
 			// これが影響するのは DREG にデータが滞留してる時だけのはず
 			// なので、その時だけログを出す。(X68030 IPLROM、NetBSD/x68k)
 			if (spc.dreg.Length() != 0) {
-				putlog(0, "SCMD Intercept with SetACK not implemented");
+				putlog(0, "SCMD Intercept with SetACK (NOT IMPLEMENTED)");
 			}
 		}
-		AssertACK();
+		scsi->AssertACK();
 		break;
 	}
 }
@@ -851,15 +860,16 @@ SPCDevice::WriteINTS(uint32 data)
 	// TIMEOUT ビットをクリアした時が
 	if ((ack & SPC::INTS_TIMEOUT)) {
 		// セレクションフェーズなら
-		if (GetPhase() == SCSI::Phase::Selection) {
+		if (scsi->GetPhase() == SCSI::Phase::Selection) {
 			// TC がゼロならバスフリー
 			// そうでなければリスタート
 			if (spc.tc == 0) {
 				BusFree();
 			} else {
-				event_tc.func = ToEventCallback(&SPCDevice::SelectionTimeout);
-				event_tc.time = spc.tc * t_CLF * 2;
-				gScheduler->RestartEvent(event_tc);
+				timer_event.func =
+					ToEventCallback(&SPCDevice::SelectionTimeout);
+				timer_event.time = spc.tc * t_CLF * 2;
+				scheduler->RestartEvent(timer_event);
 			}
 		}
 	}
@@ -878,8 +888,8 @@ SPCDevice::SetINTS(uint32 data)
 	spc.ints |= data;
 
 	// Disconnected 割り込みが起きたら ATN を下げる
-	if ((spc.ints & SPC::INTS_DISCONNECTED) && GetATN()) {
-		NegateATN();
+	if ((spc.ints & SPC::INTS_DISCONNECTED) && scsi->GetATN()) {
+		scsi->NegateATN();
 	}
 
 	ChangeInterrupt();
@@ -902,48 +912,48 @@ SPCDevice::ChangeInterrupt()
 	// ints_mask は現在の SCTL_INTR_EN 状態に応じたマスクが用意されているので
 	// 単純に AND するだけでよい。
 	if ((spc.ints & spc.ints_mask) != 0) {
-		putlog(3, "割り込み ints=$%02x", spc.ints);
+		putlog(3, "Interrupt (ints=$%02x)", spc.ints);
 		interrupt->AssertINT(this);
 	} else {
-		putlog(4, "割り込み解除");
+		putlog(4, "Negate Interrupt");
 		interrupt->NegateINT(this);
 	}
 }
 
 // PSNS レジスタ値の取得。
-// Read*() と Peek8() から呼ばれる。
+// ReadPort() と PeekPort() から呼ばれる。
 // scsibus 接続前にもモニタの更新などで呼ばれることに注意。
 uint32
 SPCDevice::GetPSNS() const
 {
 	uint32 data = 0;
 
-	if (__predict_false(bus == NULL)) {
+	if (__predict_false(scsi->GetBus() == NULL)) {
 		return 0;
 	}
 
-	if (GetREQ()) {
+	if (scsi->GetREQ()) {
 		data |= SPC::PSNS_REQ;
 	}
-	if (GetACK()) {
+	if (scsi->GetACK()) {
 		data |= SPC::PSNS_ACK;
 	}
-	if (GetATN()) {
+	if (scsi->GetATN()) {
 		data |= SPC::PSNS_ATN;
 	}
-	if (GetSEL()) {
+	if (scsi->GetSEL()) {
 		data |= SPC::PSNS_SEL;
 	}
-	if (GetBSY()) {
+	if (scsi->GetBSY()) {
 		data |= SPC::PSNS_BSY;
 	}
-	data |= (uint32)GetXfer();
+	data |= (uint32)scsi->GetXfer();
 
 	return data;
 }
 
 // SSTS レジスタ値の取得。
-// Read*() と Peek8() から呼ばれる。
+// ReadPort() と PeekPort() から呼ばれる。
 // scsibus 接続前にもモニタの更新などで呼ばれることに注意。
 uint32
 SPCDevice::GetSSTS() const
@@ -951,7 +961,7 @@ SPCDevice::GetSSTS() const
 	uint32 data;
 
 	data = spc.ssts & 0xf0;
-	if (__predict_true(bus != NULL) && GetRST()) {		// RST 信号
+	if (__predict_true(scsi->GetBus() != NULL) && scsi->GetRST()) {	// RST 信号
 		data |= SPC::SSTS_RST_IN;
 	}
 	if (spc.tc == 0) {	// TC がゼロか
@@ -998,23 +1008,25 @@ SPCDevice::GetPCTL() const
 
 // DREG レジスタの読み込み
 // (副作用があるので Peek 系からは呼ばないこと)
-uint64
+busdata
 SPCDevice::ReadDREG()
 {
-	uint32 data;
+	busdata data;
 
 	// X68k は SPC の DREQ 信号から DTACK 信号を作っているため、
 	// DREQ 信号を出さないプログラム転送モードでは DREG へのアクセスが
 	// すべてバスエラーになってしまう。
-	if (vmtype == VMTYPE_X68030) {
+	if (vmtype == VMType::X68030) {
 		if ((spc.scmd & SPC::SCMD_PROGRAM)) {
-			return (uint64)-1;
+			data.SetBusErr();
+			return data;
 		} else {
 			// DMA 転送で DREQ が出てないとき DTACK を出さないのを
-			// -2 を返すことで表現している
+			// Retry を返すことで表現している。
 			if (spc.dreg.Length() == 0) {
-				putlog(3, "DMA転送中の DREG empty");
-				return (uint64)-2;		// DTACK まだ出てないよ
+				putlog(3, "DREG empty during DMA transferring");
+				data.SetRetry();
+				return data;
 			}
 		}
 	}
@@ -1029,17 +1041,17 @@ SPCDevice::ReadDREG()
 		bool q_was_full = spc.dreg.IsFull();
 		data = spc.dreg.Dequeue();
 		if (loglevel >= 4) {
-			putlogn("DREG -> $%02x (q=%d TC=0x%x)",
-				data, spc.dreg.Length(), spc.tc);
+			putlogn("DREG -> $%02x (q=%u TC=0x%x)",
+				data.Data(), spc.dreg.Length(), spc.tc);
 		} else {
-			putlog(3, "DREG -> $%02x", data);
+			putlog(3, "DREG -> $%02x", data.Data());
 		}
 		// この読み出しが最後だったらこれで転送完了。
 		if (spc.tc == 0 && spc.dreg.Length() == 0) {
 			TransferComplete();
 		}
 		// この読み出しによってバッファに空きが出来たら転送再開。
-		if (transfer_command_running && GetREQ() && q_was_full) {
+		if (transfer_command_running && scsi->GetREQ() && q_was_full) {
 			// XXX 時間は適当
 			CallAfter(HardwareTransfer, "SPC ReadDREG to Transfer", t_CLF);
 		}
@@ -1048,7 +1060,7 @@ SPCDevice::ReadDREG()
 		if ((spc.scmd & SPC::SCMD_PROGRAM)) {
 			// XXX プログラム転送中ならどうなる?
 			data = 0xff;
-			putlog(3, "DREG -> $%02x (empty)", data);
+			putlog(3, "DREG -> $%02x (empty)", data.Data());
 		} else {
 			// DMA 転送
 			// DMA 側でリトライするのでここには来ないはず
@@ -1062,20 +1074,24 @@ SPCDevice::ReadDREG()
 }
 
 // DREG レジスタへの書き込み
-uint64
+busdata
 SPCDevice::WriteDREG(uint32 data)
 {
+	busdata r;
+
 	// X68k は SPC の DREQ 信号から DTACK 信号を作っているため、
 	// DREQ 信号を出さないプログラム転送モードでは DREG へのアクセスが
 	// すべてバスエラーになってしまう。
-	if (vmtype == VMTYPE_X68030) {
+	if (vmtype == VMType::X68030) {
 		if ((spc.scmd & SPC::SCMD_PROGRAM)) {
-			return (uint64)-1;
+			r.SetBusErr();
+			return data;
 		} else {
 			// DMA 転送で DREQ が出てないとき DTACK を出さないのを
-			// -2 を返すことで表現している
+			// Retry を返すことで表現している。
 			if (spc.dreg.IsFull()) {
-				return (uint64)-2;
+				r.SetRetry();
+				return r;
 			}
 		}
 	}
@@ -1086,14 +1102,21 @@ SPCDevice::WriteDREG(uint32 data)
 
 	// DREG キューに入れて..
 	if (spc.dreg.Enqueue(data) == false) {
-		VMPANIC("DREG キューオーバーフロー");
+		// XXX キューが一杯だったらどうなるのかは書いてないので分からない。
+		// a) キューを押し出して先頭を破棄する
+		// b) キューの末尾を破棄して新着データで上書きする
+		// c) キューの内容は変わらず新着データを破棄する
+		// どれで実装しても「この時何かしら失われるデータがある」こと自体は
+		// 再現出来ているはずなので、ここではひとまず a) の動作にしておく。
+		putlog(0, "DREG Queue overflow");
+		spc.dreg.EnqueueForce(data);
 	}
 	// ハードウェア転送中で REQ が来てれば送信。
-	if (transfer_command_running && GetREQ()) {
+	if (transfer_command_running && scsi->GetREQ()) {
 		// XXX 時間は適当
 		CallAfter(HardwareTransfer, "SPC WriteDREG to Transfer", t_CLF);
 	}
-	return 0;
+	return r;
 }
 
 // バスフリーフェーズに移行する
@@ -1104,7 +1127,7 @@ SPCDevice::BusFree()
 
 	// ここは BDID 書き込み前 (まだ scsibus に自身をつなげる前) にも呼ばれる。
 	// その場合は何もしない。バスフリーのはずでもあるし。
-	if (bus == NULL) {
+	if (scsi->GetBus() == NULL) {
 		return;
 	}
 
@@ -1119,15 +1142,15 @@ SPCDevice::BusRelease()
 	// BSY、SEL だけを落とす。それ以外の信号線は正しくはないけどとりあえず
 	// 一括して SCSIHostDevice::BusFreeCB() が落とすことにしてある。
 
-	switch (GetPhase()) {
+	switch (scsi->GetPhase()) {
 	 case SCSI::Phase::BusFree:
 	 case SCSI::Phase::Transfer:
 		break;
 	 case SCSI::Phase::Arbitration:
 	 case SCSI::Phase::Selection:
 	 case SCSI::Phase::Reselection:
-		NegateSEL();
-		NegateBSY();
+		scsi->NegateSEL();
+		scsi->NegateBSY();
 		break;
 	}
 }
@@ -1155,13 +1178,13 @@ SPCDevice::SelectCommand()
 
 		if (data == 0) {
 			// ターゲット ID がない
-			idstr = string_format("#%d", myid);
+			idstr = string_format("#%u", myid);
 		} else {
-			tgtid = DecodeID(data);
-			idstr = string_format("#%d -> #%d", myid, tgtid);
+			tgtid = SCSIBus::DecodeID(data);
+			idstr = string_format("#%u -> #%u", myid, tgtid);
 		}
 	}
-	putlog(2, "Select コマンド (%s)", idstr.c_str());
+	putlog(2, "Select command (%s)", idstr.c_str());
 
 	// アービトレーションありの場合
 	//
@@ -1203,7 +1226,7 @@ SPCDevice::SelectCommand()
 	// t_FRID/t_BFBL はたぶん今からではなくバスフリーになった時点から起算。
 	// t_FRID と t_BFBL は同じ式なのでここでは t_BFBL で統一。
 	uint64 t_BFBL = (6 + spc.GetTCL()) * t_CLF;	// Min
-	uint64 now = gScheduler->GetVirtTime();
+	uint64 now = scheduler->GetVirtTime();
 	if ((last_busfree_time + t_BFBL) > now) {
 		wait = (last_busfree_time + t_BFBL) - now;
 	} else {
@@ -1217,9 +1240,9 @@ SPCDevice::SelectCommand()
 	// N が 0 なら無限大なのでタイマーは動かさない。
 	uint32 N = (spc.tc >> 8) << 8;
 	if (N != 0) {
-		event_tc.func = ToEventCallback(&SPCDevice::SelectionTimeout);
-		event_tc.time = (N + 15) * t_CLF * 2;
-		gScheduler->RestartEvent(event_tc);
+		timer_event.func = ToEventCallback(&SPCDevice::SelectionTimeout);
+		timer_event.time = (N + 15) * t_CLF * 2;
+		scheduler->RestartEvent(timer_event);
 	}
 
 	if ((spc.sctl & SPC::SCTL_ARBIT_EN)) {
@@ -1256,13 +1279,13 @@ SPCDevice::Arbitration1(Event& ev)
 	// 区別するため PCTL に設定された IO をバスに出す(p.27)。
 	// MSG と CD は記述がないけどたぶん一緒に出すだろう。
 	// タイミングもどこでか分からんのでとりあえずここで。
-	SetData(spc.temp_out);
-	SetXfer(spc.pctl_out);
+	scsi->SetData(spc.temp_out);
+	scsi->SetXfer(spc.pctl_out);
 	if (set_atn_in_selection) {
-		AssertATN();
+		scsi->AssertATN();
 		set_atn_in_selection = false;
 	}
-	AssertSEL();
+	scsi->AssertSEL();
 
 	uint64 t_SIDA = 11 * t_CLF - 30;
 	uint64 t_IDBH = 2 * t_CLF - 80;
@@ -1276,7 +1299,7 @@ SPCDevice::Arbitration2(Event& ev)
 {
 	putlog(3, "Arbitration2");
 
-	NegateBSY();
+	scsi->NegateBSY();
 }
 
 // アービトレーションフェーズを使わない場合のセレクションフェーズで、
@@ -1292,9 +1315,9 @@ SPCDevice::Selection1(Event& ev)
 	// ここでは SEL がトリガーになるので SEL のアサートは最後に行う。
 	// アービトレーションなしなら PCTL に関係なくセレクションフェーズ (IO=0)
 	// を開始する(p.27)。MSG と CD は記述がないけどたぶん一緒に出すだろう。
-	SetData(spc.temp_out);
-	SetXfer(0);
-	AssertSEL();
+	scsi->SetData(spc.temp_out);
+	scsi->SetXfer(0);
+	scsi->AssertSEL();
 }
 
 // セレクションコマンドがタイムアウトした時に呼ばれるコールバック。
@@ -1303,13 +1326,13 @@ SPCDevice::SelectionTimeout(Event& ev)
 {
 	if (loglevel >= 3) {
 		int tgtid;
-		uint8 data = GetData();
+		uint8 data = scsi->GetData();
 		data &= ~(1 << myid);
-		tgtid = DecodeID(data);
+		tgtid = SCSIBus::DecodeID(data);
 		if (tgtid == -1) {
 			putlogn("Selection timeout");
 		} else {
-			putlogn("Selection timeout (#%d)", tgtid);
+			putlogn("Selection timeout (#%u)", tgtid);
 		}
 	}
 
@@ -1329,7 +1352,7 @@ SPCDevice::SetATNCommand()
 
 	// Select コマンド前なら、セレクションフェーズで ATN を立てる指示。
 	// 接続中(転送フェーズ中?) なら即座に ATN を立てる。
-	switch (GetPhase()) {
+	switch (scsi->GetPhase()) {
 	 case SCSI::Phase::BusFree:
 		set_atn_in_selection = true;
 		break;
@@ -1338,7 +1361,7 @@ SPCDevice::SetATNCommand()
 	 case SCSI::Phase::Reselection:
 		// Select Command 実行中なので実質ここで呼ばれることはないはずだが
 	 case SCSI::Phase::Transfer:
-		AssertATN();
+		scsi->AssertATN();
 		break;
 	}
 }
@@ -1353,7 +1376,7 @@ SPCDevice::TransferCommand()
 	// SCMD レジスタ書き込みのうち Transfer コマンドに関わる未実装機能の
 	// ログ表示は SCMD レジスタ書き込み時ではなくここで表示。
 	if ((spc.scmd & SPC::SCMD_TERM_MODE)) {
-		putlog(0, "SCMD Padding mode not implemented");
+		putlog(0, "SCMD Padding mode (NOT IMPLEMENTED)");
 	}
 
 	transfer_command_running = true;
@@ -1361,7 +1384,7 @@ SPCDevice::TransferCommand()
 
 	// PCTL レジスタに設定してあるフェーズをバスに出す。
 	// XXX どこでかよく分からんけど、とりあえずこの辺で。
-	SetXfer(spc.pctl_out);
+	scsi->SetXfer(spc.pctl_out);
 
 	// 転送開始前に DREG キューを初期化する。
 	// MPU 側が DREG を全部読まずにフェーズを中断とかはたぶん出来るので。
@@ -1370,22 +1393,21 @@ SPCDevice::TransferCommand()
 
 	// Transfer コマンド発行時に REQ がアサートされていなければ
 	// アサートされるまで待つ。この場合、ここでは何もすることはなく、
-	// TransferReqCB() が呼ばれてそこから転送フェーズが動き始める。
-	if (GetREQ()) {
+	// OnTransferReq() が呼ばれてそこから転送フェーズが動き始める。
+	if (scsi->GetREQ()) {
 		// コマンドから転送開始までちょっとだけかかる
 		CallAfter(HardwareTransfer, "SPC SCMD to Start", 4 * t_CLF);
 	}
 }
 
-// バスフリーコールバック (SCSIBus から呼ばれる)
+// バスフリーコールバック (SCSIHostDevice から呼ばれる)。
 // id はこのバスフリーを行ったデバイス (0..7)
 void
-SPCDevice::BusFreeCB(int id)
+SPCDevice::BusFreeCallback(uint id)
 {
 	putlog(4, "BusFree Callback");
 
-	inherited::BusFreeCB(id);
-	last_busfree_time = gScheduler->GetVirtTime();
+	last_busfree_time = scheduler->GetVirtTime();
 
 	transfer_command_running = false;
 	spc.ssts &= ~(SPC::SSTS_CONN_INIT | SPC::SSTS_CONN_TARG);
@@ -1401,19 +1423,15 @@ SPCDevice::BusFreeCB(int id)
 	}
 }
 
-// ターゲットがセレクションに応答したコールバック (SCSIBus から呼ばれる)
+// ターゲットがセレクションに応答したコールバック
+// (SCSIHostDevice から呼ばれる)
 void
-SPCDevice::SelectionAckCB()
+SPCDevice::SelectionAckCallback()
 {
-	// ターゲットが BSY を立てて応答したので、
-	// こちらはデータバスをクリアして SEL を下げる。
-	// これによりセレクションフェーズは成功で完了する。
-	putlog(4, "SelectionAck Calback");
-	SetData(0);
-	NegateSEL();
+	putlog(4, "SelectionAck Callback");
 
 	// タイムアウトイベントを停止
-	gScheduler->StopEvent(event_tc);
+	scheduler->StopEvent(timer_event);
 
 	spc.ssts &= ~SPC::SSTS_SPC_BUSY;
 	spc.ssts |= SPC::SSTS_CONN_INIT;
@@ -1422,9 +1440,10 @@ SPCDevice::SelectionAckCB()
 	SetINTS(SPC::INTS_COMPLETE);
 }
 
-// 転送フェーズでターゲットが REQ を立てたコールバック (SCSIBus から呼ばれる)
+// 転送フェーズでターゲットが REQ を立てたコールバック
+// (SCSIHostDevice から呼ばれる)
 void
-SPCDevice::TransferReqCB()
+SPCDevice::TransferReqCallback()
 {
 	// REQ が立てば Transfer Command 発行前でも Xfer In Progress は立てる
 	spc.ssts |= SPC::SSTS_IN_PROGRESS;
@@ -1432,7 +1451,7 @@ SPCDevice::TransferReqCB()
 	if (transfer_command_running) {
 		// Transfer Command 実行中なら REQ 待ちだったので
 		// ここでハードウェア転送を開始。
-		HardwareTransfer(event);
+		HardwareTransfer(phase_event);
 	}
 
 	// データシートでは、マニュアル転送で IN 方向の時データバスの内容を
@@ -1440,8 +1459,8 @@ SPCDevice::TransferReqCB()
 	// HardwareTransfer によって状態が変更される可能性があるため
 	// もう一回チェック。
 	if (transfer_command_running == false) {
-		if (GetIO()) {
-			spc.temp_in = GetData();
+		if (scsi->GetIO()) {
+			spc.temp_in = scsi->GetData();
 		}
 	}
 }
@@ -1450,7 +1469,7 @@ SPCDevice::TransferReqCB()
 // イベントコールバックとして呼び出すこと。
 // REQ がアサートされているのを確認してから呼び出すこと。
 // (1バイト送信するとターゲットが再び REQ を上げるなどしてくるので、
-// TransferReqCB() か DREG 書き込みから再びここが呼ばれる)
+// OnTransferReq() か DREG 書き込みから再びここが呼ばれる)
 void
 SPCDevice::HardwareTransfer(Event& ev)
 {
@@ -1460,7 +1479,7 @@ SPCDevice::HardwareTransfer(Event& ev)
 
 	// PCTL のフェーズと SCSI バスのフェーズが一致していれば転送(開始|継続)。
 	// そうでなければ Service Required 割り込み。
-	if (spc.pctl_out != GetXfer()) {
+	if (spc.pctl_out != scsi->GetXfer()) {
 		transfer_command_running = false;
 		putlog(3, "Hardware Transfer terminated (phase mismatch)");
 		SetINTS(SPC::INTS_SERV_REQ);
@@ -1469,9 +1488,9 @@ SPCDevice::HardwareTransfer(Event& ev)
 
 	// DREG が送受信不可能ならここでは何もせずに帰るだけでよい。
 	// その後 DREG アクセスによって送受信可能になればまたこれが呼ばれる。
-	if (GetIO()) {
+	if (scsi->GetIO()) {
 		// Input (ターゲット → イニシエータ)
-		bool queued = spc.dreg.Enqueue(GetData());
+		bool queued = spc.dreg.Enqueue(scsi->GetData());
 		// SDGC_XFER_ENABLE なら読み込めるデータがある場合に割り込み。
 		if ((spc.sdgc & SPC::SDGC_XFER_ENABLE)) {
 			putlog(3, "SDGC XFER_OUT Interrupt");
@@ -1484,7 +1503,7 @@ SPCDevice::HardwareTransfer(Event& ev)
 	} else {
 		// Output (イニシエータ → ターゲット)
 		if (spc.dreg.Length() != 0) {
-			SetData(spc.dreg.Dequeue());
+			scsi->SetData(spc.dreg.Dequeue());
 		} else {
 			putlog(4, "Hardware Transfer (DREG empty)");
 			// SDGC_XFER_ENABLE ならキューに書き込む必要がある場合に割り込み。
@@ -1496,17 +1515,17 @@ SPCDevice::HardwareTransfer(Event& ev)
 		}
 	}
 
-	AssertACK();
+	scsi->AssertACK();
 
 	// 転送完了
 	spc.tc--;
-	putlog(4, "Hardware Transfer: %s q=%d tc=%d",
-		(GetIO() ? "IN" : "OUT"), spc.dreg.Length(), spc.tc);
+	putlog(4, "Hardware Transfer: %s q=%u tc=%u",
+		(scsi->GetIO() ? "IN" : "OUT"), spc.dreg.Length(), spc.tc);
 	if (spc.tc == 0) {
 		// どちらでも転送自体はここで完了(?)
 		transfer_command_running = false;
 
-		if (GetIO()) {
+		if (scsi->GetIO()) {
 			// Input 時は TC がゼロになって DREG キューを全部読み出した
 			// ところで完了なので、ここでは何もしない。
 		} else {
@@ -1516,14 +1535,14 @@ SPCDevice::HardwareTransfer(Event& ev)
 	}
 
 	// メッセージアウトフェーズの最終バイト送信中(?)に ATN を下げる
-	if (GetXfer() == SCSI::XferPhase::MsgOut && spc.tc == 0) {
-		NegateATN();
+	if (scsi->GetXfer() == SCSI::XferPhase::MsgOut && spc.tc == 0) {
+		scsi->NegateATN();
 	}
 
-	if (GetXfer() == SCSI::XferPhase::MsgIn && spc.tc == 0) {
+	if (scsi->GetXfer() == SCSI::XferPhase::MsgIn && spc.tc == 0) {
 		// ただしメッセージインフェーズの最終バイトなら ACK を下げない。
 	} else {
-		NegateACK();
+		scsi->NegateACK();
 	}
 }