WS: Fixed regression for SDMA Control port

According to sound_dma, bit 7 can be read as set, but WSHSTest expects bit 7 to be clear after setting it. WSHSTest attempts to start the sound dma while length is 0, which might immediately clear the enable flag.

Author: SourMesen

Core/Shared/RecordedRomTest.cpp

@@ -137,6 +137,8 @@ void RecordedRomTest::Record(string filename, bool reset)
 		settings->GetGbaConfig().DisableFrameSkipping = true;
 
 		settings->GetGbaConfig().SkipBootScreen = false;
+		settings->GetWsConfig().UseBootRom = true;
+		settings->GetWsConfig().LcdShowIcons = true;
 
 		//Start recording movie alongside with screenshots
 		RecordMovieOptions options;
@@ -225,6 +227,8 @@ RomTestResult RecordedRomTest::Run(string filename)
 		settings->GetGbaConfig().DisableFrameSkipping = true;
 
 		settings->GetGbaConfig().SkipBootScreen = false;
+		settings->GetWsConfig().UseBootRom = true;
+		settings->GetWsConfig().LcdShowIcons = true;
 
 		_emu->Lock();
 		//Start playing movie

Core/WS/WsDmaController.cpp

@@ -45,8 +45,7 @@ void WsDmaController::RunGeneralDma()
 
 void WsDmaController::ProcessSoundDma()
 {
-	if(!_state.SdmaEnabled || _state.SdmaLength == 0) {
-		//TODOWS what does enabling when length = 0 do?
+	if(!_state.SdmaEnabled) {
 		return;
 	}
 
@@ -170,7 +169,12 @@ void WsDmaController::WritePort(uint16_t port, uint8_t value)
 			break;
 
 		case 0x52:
-			_state.SdmaControl = value & 0x5F;
+			if(_state.SdmaLength == 0) {
+				//Clear enable flag if length is 0 (based on sound_dma + WSHSTest results)
+				value &= ~0x80;
+			}
+
+			_state.SdmaControl = value & 0xDF;
 
 			switch(value & 0x03) {
 				default: