Merged change from http://forums.ngemu.com/pcsx-discussion/116599-developers-wanted-pcsx-revival-9.html#post1902678 with minor amendments.

GPU DMA chain timing (FF7 opening CG and maybe other games too - not tested with rebel assault 2 though).


git-svn-id: https://pcsxr.svn.codeplex.com/svn/pcsxr@57139 e17a0e51-4ae3-4d35-97c3-1a29b211df97

1 files changed, 159 insertions, 141 deletions

diff --git a/libpcsxcore/cdrom.c b/libpcsxcore/cdrom.c
index dfd14656..ab8a76c1 100644
--- a/libpcsxcore/cdrom.c
+++ b/libpcsxcore/cdrom.c
@@ -184,6 +184,7 @@ void cdrInterrupt() {
 			SetResultSize(1);
 			cdr.Result[0] = cdr.StatP;
 			cdr.Stat = Acknowledge;
+			cdr.LidCheck++;
 			break;
 
 		case CdlSetloc:
@@ -235,6 +236,7 @@ void cdrInterrupt() {
 			cdr.Result[0] = cdr.StatP;
 			cdr.Stat = Complete;
 //			cdr.Stat = Acknowledge;
+			cdr.LidCheck++;
 			break;
 
 		case CdlPause:
@@ -323,7 +325,7 @@ void cdrInterrupt() {
 			SetResultSize(8);
 			subq = (struct SubQ *)CDR_getBufferSub();
 
-			// DATA + AUDIO has subchannel data

+			// DATA + AUDIO has subchannel data
 			if (subq != NULL) {
 				cdr.Result[0] = subq->TrackNumber;
 				cdr.Result[1] = subq->IndexNumber;
@@ -337,7 +339,7 @@ void cdrInterrupt() {
 			} else {
 				cdr.Result[0] = 1;
 				cdr.Result[1] = 1;
-

+
 				// NOTE: This only works for TRACK 01
 				cdr.Result[2] = btoi(cdr.Prev[0]);
 				cdr.Result[3] = btoi(cdr.Prev[1]) - 2;
@@ -359,9 +361,9 @@ void cdrInterrupt() {
 			break;
 
     	case CdlGetTN:
-			StopReading();

-			StopCdda();

-

+			StopReading();
+			StopCdda();
+
 			cdr.CmdProcess = 0;
 			SetResultSize(3);
 			cdr.StatP |= 0x2;
@@ -547,12 +549,12 @@ void cdrInterrupt() {
 				cdr.Result[5] = 0;
 				cdr.Result[6] = 0;
 				cdr.Result[7] = 0;
-			}

-			else

-				// FIXME!!

+			}
+			else
+				// FIXME!!
 				//memcpy(cdr.Result, &stat.Track, 8);
-				0;

-

+				0;
+
 			cdr.Stat = 1;
 			SetResultSize(8);
 			AddIrqQueue(REPPLAY_ACK, cdReadTime);
@@ -566,35 +568,55 @@ void cdrInterrupt() {
 			break;
 	}
 
-	// check case open/close

-	i = stat.Status;

-	if (CDR_getStatus(&stat) != -1) {

-		if (stat.Type == 0xff) cdr.Stat = DiskError;

-

-		// case now open

-		if (stat.Status & 0x10) {

-			if( Irq != CdlNop )

-			{

-				cdr.Stat = DiskError;

-				cdr.Result[0] |= 0x01;

-			}

-

-			// GameShark Lite: Wants -exactly- $10

-			cdr.Result[0] |= 0x10;

-			cdr.Result[0] &= ~0x02;

-		}

-		// case now closed

-		else if (i & 0x10) {

-			cdr.StatP &= ~0x11;

-			cdr.Result[0] |= 0x2;

-

-			// GameShark Lite: Wants -exactly- $42, then $02

-			cdr.Result[0] |= 0x40;

-

-			CheckCdrom();

-		}

-	}

-

+	// check case open/close
+	if (cdr.LidCheck > 0) {
+		cdr.LidCheck--;
+
+		i = stat.Status;
+		if (CDR_getStatus(&stat) != -1) {
+			if (stat.Type == 0xff) cdr.Stat = DiskError;
+
+			// case now open
+			if (stat.Status & 0x10) {
+				// GameShark Lite: Death if DiskError happens
+				//
+				// Vib Ribbon: Needs DiskError for CD swap
+
+				if (Irq != CdlNop) {
+					cdr.Stat = DiskError;
+
+					cdr.StatP |= 0x01;
+					cdr.Result[0] |= 0x01;
+				}
+
+				cdr.StatP |= 0x10;
+				cdr.StatP &= ~0x02;
+
+				// GameShark Lite: Wants -exactly- $10
+				cdr.Result[0] |= 0x10;
+				cdr.Result[0] &= ~0x02;
+			}
+
+			// case now closed
+			else if (i & 0x10) {
+				cdr.StatP &= ~0x11;
+				cdr.StatP |= 0x2;
+				cdr.StatP |= 0x40;
+
+				// GameShark Lite: Wants -exactly- $42, then $02
+				cdr.Result[0] |= 0x2;
+				cdr.Result[0] |= 0x40;
+
+				CheckCdrom();
+			}
+
+			else if (cdr.LidCheck == 0) {
+				// GameShark Lite: Seek detection done
+				cdr.StatP &= ~0x40;
+			}
+		}
+	}
+
 	if (cdr.Stat != NoIntr && cdr.Reg2 != 0x18) {
 		psxHu32ref(0x1070) |= SWAP32((u32)0x4);
 	}
@@ -1081,75 +1103,75 @@ void cdrWrite3(unsigned char rt) {
 				break;
 		}
 	}
-

-	/*

-	GameShark CDX 3.3

-

-	TODO: Can we relax the rules?

-	*/

-	if (rt == 0 && cdr.Ctrl == 0xd8 && cdr.Readed == 1) {

-		u32 i;

-

-

-		// low-level status access

-		SetResultSize(15);

-

-

-		i = stat.Status;

-		if (CDR_getStatus(&stat) != -1) {

-			// Hack: Fake CDROM seek / spin time

-			static int seek_time = 0;

-

-			if (stat.Type == 0xff) {

-			}

-			else if (stat.Status & 0x10) {

-				// unknown #1 - status?

-				// unknown #2 - case open?

-

-				cdr.Result[0] = 0x11;

-				cdr.Result[1] = 0x80;

-			}

-			else if ( i & 0x10 ) {

-				cdr.Result[0] = 0x13;

-				cdr.Result[1] = 0x80;

-

-				// minimum 50 tries for GS CDX 3.3

-				seek_time = 50;

-			}

-			else if ( seek_time ) {

-				cdr.Result[0] = 0x13;

-				cdr.Result[1] = 0x80;

-

-				seek_time--;

-			}

-			else

-			{

-				cdr.Result[0] = 0x01;

-				cdr.Result[1] = 0x00;

-			}

-

-			// unknown #3

-			cdr.Result[2] = 0;

-

-			// unknown #4 - error?

-			cdr.Result[3] = 0;

-

-			// unknown #5-14

-			cdr.Result[4] = 0;

-			cdr.Result[5] = 0;

-			cdr.Result[6] = 0;

-			cdr.Result[7] = 0;

-			cdr.Result[8] = 0;

-			cdr.Result[9] = 0;

-			cdr.Result[10] = 0;

-			cdr.Result[11] = 0;

-			cdr.Result[12] = 0;

-			cdr.Result[13] = 0;

-

-			// unknown #15 - error?

-			cdr.Result[14] = 0;

-		}

-	}

+
+	/*
+	GameShark CDX 3.3
+
+	TODO: Can we relax the rules?
+	*/
+	if (rt == 0 && cdr.Ctrl == 0xd8 && cdr.Readed == 1) {
+		u32 i;
+
+
+		// low-level status access
+		SetResultSize(15);
+
+
+		i = stat.Status;
+		if (CDR_getStatus(&stat) != -1) {
+			// Hack: Fake CDROM seek / spin time
+			static int seek_time = 0;
+
+			if (stat.Type == 0xff) {
+			}
+			else if (stat.Status & 0x10) {
+				// unknown #1 - status?
+				// unknown #2 - case open?
+
+				cdr.Result[0] = 0x11;
+				cdr.Result[1] = 0x80;
+			}
+			else if ( i & 0x10 ) {
+				cdr.Result[0] = 0x13;
+				cdr.Result[1] = 0x80;
+
+				// minimum 50 tries for GS CDX 3.3
+				seek_time = 50;
+			}
+			else if ( seek_time ) {
+				cdr.Result[0] = 0x13;
+				cdr.Result[1] = 0x80;
+
+				seek_time--;
+			}
+			else
+			{
+				cdr.Result[0] = 0x01;
+				cdr.Result[1] = 0x00;
+			}
+
+			// unknown #3
+			cdr.Result[2] = 0;
+
+			// unknown #4 - error?
+			cdr.Result[3] = 0;
+
+			// unknown #5-14
+			cdr.Result[4] = 0;
+			cdr.Result[5] = 0;
+			cdr.Result[6] = 0;
+			cdr.Result[7] = 0;
+			cdr.Result[8] = 0;
+			cdr.Result[9] = 0;
+			cdr.Result[10] = 0;
+			cdr.Result[11] = 0;
+			cdr.Result[12] = 0;
+			cdr.Result[13] = 0;
+
+			// unknown #15 - error?
+			cdr.Result[14] = 0;
+		}
+	}
 }
 
 void psxDma3(u32 madr, u32 bcr, u32 chcr) {
@@ -1179,24 +1201,24 @@ void psxDma3(u32 madr, u32 bcr, u32 chcr) {
 #endif
 				break;
 			}
-

-			/*

-			GS CDX: Enhancement CD crash

-			- Setloc 0:0:0

-			- CdlPlay

-			- Spams DMA3 and gets buffer overrun

-			*/

-

-			if( (cdr.pTransfer-cdr.Transfer) + cdsize > 2352 )

-			{

-				// avoid crash - probably should wrap here

-				//memcpy(ptr, cdr.pTransfer, cdsize);

-			}

-			else

-			{

-				memcpy(ptr, cdr.pTransfer, cdsize);

-			}

-

+
+			/*
+			GS CDX: Enhancement CD crash
+			- Setloc 0:0:0
+			- CdlPlay
+			- Spams DMA3 and gets buffer overrun
+			*/
+
+			if( (cdr.pTransfer-cdr.Transfer) + cdsize > 2352 )
+			{
+				// avoid crash - probably should wrap here
+				//memcpy(ptr, cdr.pTransfer, cdsize);
+			}
+			else
+			{
+				memcpy(ptr, cdr.pTransfer, cdsize);
+			}
+
 			psxCpu->Clear(madr, cdsize / 4);
 			cdr.pTransfer += cdsize;
 			break;
@@ -1233,18 +1255,14 @@ int cdrFreeze(gzFile f, int Mode) {
 
 	return 0;
 }
-

-

-

-

-

-void LidInterrupt()

-{

-	// generate interrupt if none active - open or close

-	if( cdr.Irq == 0 || cdr.Irq == 0xff )

-	{

-		cdr.Ctrl |= 0x80;

-		cdr.Stat = NoIntr; 

-		AddIrqQueue(CdlNop, 0x800);

-	}

-}

+
+void LidInterrupt() {
+	cdr.LidCheck = 3; // this should be enough
+
+	// generate interrupt if none active - open or close
+	if (cdr.Irq == 0 || cdr.Irq == 0xff) {
+		cdr.Ctrl |= 0x80;
+		cdr.Stat = NoIntr; 
+		AddIrqQueue(CdlNop, 0x800);
+	}
+}