Commit f25f17dc authored by Takashi Iwai's avatar Takashi Iwai

ALSA: seq: ump: Fix conversion from MIDI2 to MIDI1 UMP messages

The conversion from MIDI2 to MIDI1 UMP messages had a leftover
artifact (superfluous bit shift), and this resulted in the bogus type
check, leading to empty outputs.  Let's fix it.

Fixes: e9e02819 ("ALSA: seq: Automatic conversion of UMP events")
Cc: <stable@vger.kernel.org>
Link: https://github.com/alsa-project/alsa-utils/issues/262
Message-ID: <20240419100442.14806-1-tiwai@suse.de>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 7ee5faad
...@@ -428,7 +428,7 @@ static int cvt_ump_midi2_to_midi1(struct snd_seq_client *dest, ...@@ -428,7 +428,7 @@ static int cvt_ump_midi2_to_midi1(struct snd_seq_client *dest,
midi1->note.group = midi2->note.group; midi1->note.group = midi2->note.group;
midi1->note.status = midi2->note.status; midi1->note.status = midi2->note.status;
midi1->note.channel = midi2->note.channel; midi1->note.channel = midi2->note.channel;
switch (midi2->note.status << 4) { switch (midi2->note.status) {
case UMP_MSG_STATUS_NOTE_ON: case UMP_MSG_STATUS_NOTE_ON:
case UMP_MSG_STATUS_NOTE_OFF: case UMP_MSG_STATUS_NOTE_OFF:
midi1->note.note = midi2->note.note; midi1->note.note = midi2->note.note;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment