diff --git a/engines/sci/sound/drivers/midi.cpp b/engines/sci/sound/drivers/midi.cpp
index 70e1cf21330..ccb1c87d027 100644
--- a/engines/sci/sound/drivers/midi.cpp
+++ b/engines/sci/sound/drivers/midi.cpp
@@ -378,10 +378,10 @@ int MidiPlayer_Midi::getVolume() {
 }
 
 void MidiPlayer_Midi::setReverb(byte reverb) {
-	assert(reverb < kReverbConfigNr || reverb == 127);
+	assert(reverb < kReverbConfigNr);
 	_reverb = reverb;
 
-	if (_hasReverb && _reverb != 127)	// 127: SCI invalid, don't send to sound card
+	if (_hasReverb)
 		sendMt32SysEx(0x100001, _reverbConfig[_reverb], 3, true);
 }
 
diff --git a/engines/sci/sound/midiparser_sci.cpp b/engines/sci/sound/midiparser_sci.cpp
index abd3894d9a1..398917764ca 100644
--- a/engines/sci/sound/midiparser_sci.cpp
+++ b/engines/sci/sound/midiparser_sci.cpp
@@ -501,7 +501,8 @@ void MidiParser_SCI::parseNextEvent(EventInfo &info) {
 			// Also, sci/sound/iterator/iterator.cpp, function BaseSongIterator::parseMidiCommand()
 			switch (info.basic.param1) {
 			case kSetReverb:
-				((MidiPlayer *)_driver)->setReverb(info.basic.param2);
+				if (info.basic.param2 != 127)	// 127: SCI invalid, ignore
+					((MidiPlayer *)_driver)->setReverb(info.basic.param2);
 				break;
 			case kMidiHold:
 				// Check if the hold ID marker is the same as the hold ID
diff --git a/engines/sci/sound/music.cpp b/engines/sci/sound/music.cpp
index 010f161a7a0..6628c175c63 100644
--- a/engines/sci/sound/music.cpp
+++ b/engines/sci/sound/music.cpp
@@ -227,7 +227,12 @@ MusicEntry *SciMusic::getActiveSci0MusicSlot() {
 
 void SciMusic::setReverb(byte reverb) {
 	Common::StackLock lock(_mutex);
-	_pMidiDrv->setReverb(reverb);
+	if (reverb != 127)	// 127: SCI invalid, ignore
+		_pMidiDrv->setReverb(reverb);
+
+	// SSCI stored a separate reverb value per song
+	// We don't, currently, as the current functionality
+	// works without an additional variable
 }
 
 byte SciMusic::getReverb() {