Skip to content

Commit

Permalink
ALSA: firewire-digi00x: add another rawmidi character device for MIDI…
Browse files Browse the repository at this point in the history
… control ports

Digi 002/003 family uses two ways to transfer MIDI messages. They're
different mechanisms, while it's better to handle the ways in different
ALSA rawmidi character devices because one character device has just a
set of operations.

This commit adds another rawmidi character device for control MIDI port. As
a result, first rawmidi character device is just for MIDI messages
transferred by isochronous packets.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
  • Loading branch information
Takashi Sakamoto authored and Takashi Iwai committed Oct 30, 2015
1 parent c5fcee0 commit d1482fb
Showing 1 changed file with 50 additions and 54 deletions.
104 changes: 50 additions & 54 deletions sound/firewire/digi00x/digi00x-midi.c
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,6 @@ static int midi_phys_open(struct snd_rawmidi_substream *substream)
struct snd_dg00x *dg00x = substream->rmidi->private_data;
int err;

/* This port is for asynchronous transaction. */
if (substream->number == 0)
return 0;

err = snd_dg00x_stream_lock_try(dg00x);
if (err < 0)
return err;
Expand All @@ -35,10 +31,6 @@ static int midi_phys_close(struct snd_rawmidi_substream *substream)
{
struct snd_dg00x *dg00x = substream->rmidi->private_data;

/* This port is for asynchronous transaction. */
if (substream->number == 0)
return 0;

mutex_lock(&dg00x->mutex);
dg00x->substreams_counter--;
snd_dg00x_stream_stop_duplex(dg00x);
Expand All @@ -56,21 +48,12 @@ static void midi_phys_capture_trigger(struct snd_rawmidi_substream *substream,

spin_lock_irqsave(&dg00x->lock, flags);

/* This port is for asynchronous transaction. */
if (substream->number == 0) {
if (up)
dg00x->in_control = substream;
else
dg00x->in_control = NULL;
} else {
if (up)
amdtp_dot_midi_trigger(&dg00x->tx_stream,
substream->number - 1,
substream);
else
amdtp_dot_midi_trigger(&dg00x->tx_stream,
substream->number - 1, NULL);
}
if (up)
amdtp_dot_midi_trigger(&dg00x->tx_stream, substream->number,
substream);
else
amdtp_dot_midi_trigger(&dg00x->tx_stream, substream->number,
NULL);

spin_unlock_irqrestore(&dg00x->lock, flags);
}
Expand All @@ -83,20 +66,12 @@ static void midi_phys_playback_trigger(struct snd_rawmidi_substream *substream,

spin_lock_irqsave(&dg00x->lock, flags);

/* This port is for asynchronous transaction. */
if (substream->number == 0) {
if (up)
snd_fw_async_midi_port_run(&dg00x->out_control,
substream);
} else {
if (up)
amdtp_dot_midi_trigger(&dg00x->rx_stream,
substream->number - 1,
substream);
else
amdtp_dot_midi_trigger(&dg00x->rx_stream,
substream->number - 1, NULL);
}
if (up)
amdtp_dot_midi_trigger(&dg00x->rx_stream, substream->number,
substream);
else
amdtp_dot_midi_trigger(&dg00x->rx_stream, substream->number,
NULL);

spin_unlock_irqrestore(&dg00x->lock, flags);
}
Expand Down Expand Up @@ -177,15 +152,16 @@ static struct snd_rawmidi_ops midi_ctl_playback_ops = {
};

static void set_midi_substream_names(struct snd_dg00x *dg00x,
struct snd_rawmidi_str *str)
struct snd_rawmidi_str *str,
bool is_ctl)
{
struct snd_rawmidi_substream *subs;

list_for_each_entry(subs, &str->substreams, list) {
if (subs->number > 0)
if (!is_ctl)
snprintf(subs->name, sizeof(subs->name),
"%s MIDI %d",
dg00x->card->shortname, subs->number);
dg00x->card->shortname, subs->number + 1);
else
/* This port is for asynchronous transaction. */
snprintf(subs->name, sizeof(subs->name),
Expand All @@ -196,32 +172,52 @@ static void set_midi_substream_names(struct snd_dg00x *dg00x,

int snd_dg00x_create_midi_devices(struct snd_dg00x *dg00x)
{
struct snd_rawmidi *rmidi;
struct snd_rawmidi *rmidi[2];
struct snd_rawmidi_str *str;
unsigned int i;
int err;

/* Add physical midi ports. */
err = snd_rawmidi_new(dg00x->card, dg00x->card->driver, 0,
DOT_MIDI_OUT_PORTS + 1, DOT_MIDI_IN_PORTS + 1, &rmidi);
DOT_MIDI_OUT_PORTS, DOT_MIDI_IN_PORTS, &rmidi[0]);
if (err < 0)
return err;

snprintf(rmidi->name, sizeof(rmidi->name),
snprintf(rmidi[0]->name, sizeof(rmidi[0]->name),
"%s MIDI", dg00x->card->shortname);
rmidi->private_data = dg00x;

rmidi->info_flags |= SNDRV_RAWMIDI_INFO_INPUT;
snd_rawmidi_set_ops(rmidi, SNDRV_RAWMIDI_STREAM_INPUT,
snd_rawmidi_set_ops(rmidi[0], SNDRV_RAWMIDI_STREAM_INPUT,
&midi_phys_capture_ops);
str = &rmidi->streams[SNDRV_RAWMIDI_STREAM_INPUT];
set_midi_substream_names(dg00x, str);

rmidi->info_flags |= SNDRV_RAWMIDI_INFO_OUTPUT;
snd_rawmidi_set_ops(rmidi, SNDRV_RAWMIDI_STREAM_OUTPUT,
snd_rawmidi_set_ops(rmidi[0], SNDRV_RAWMIDI_STREAM_OUTPUT,
&midi_phys_playback_ops);
str = &rmidi->streams[SNDRV_RAWMIDI_STREAM_OUTPUT];
set_midi_substream_names(dg00x, str);

rmidi->info_flags |= SNDRV_RAWMIDI_INFO_DUPLEX;
/* Add a pair of control midi ports. */
err = snd_rawmidi_new(dg00x->card, dg00x->card->driver, 1,
1, 1, &rmidi[1]);
if (err < 0)
return err;

snprintf(rmidi[1]->name, sizeof(rmidi[1]->name),
"%s control", dg00x->card->shortname);

snd_rawmidi_set_ops(rmidi[1], SNDRV_RAWMIDI_STREAM_INPUT,
&midi_ctl_capture_ops);
snd_rawmidi_set_ops(rmidi[1], SNDRV_RAWMIDI_STREAM_OUTPUT,
&midi_ctl_playback_ops);

for (i = 0; i < ARRAY_SIZE(rmidi); i++) {
rmidi[i]->private_data = dg00x;

rmidi[i]->info_flags |= SNDRV_RAWMIDI_INFO_INPUT;
str = &rmidi[i]->streams[SNDRV_RAWMIDI_STREAM_INPUT];
set_midi_substream_names(dg00x, str, i);

rmidi[i]->info_flags |= SNDRV_RAWMIDI_INFO_OUTPUT;
str = &rmidi[i]->streams[SNDRV_RAWMIDI_STREAM_OUTPUT];
set_midi_substream_names(dg00x, str, i);

rmidi[i]->info_flags |= SNDRV_RAWMIDI_INFO_DUPLEX;
}

return 0;
}

0 comments on commit d1482fb

Please sign in to comment.