ALSA: oxfw: add stream format quirk for SCS.1 models
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Tue, 22 Dec 2015 00:15:46 +0000 (09:15 +0900)
committerTakashi Iwai <tiwai@suse.de>
Tue, 22 Dec 2015 10:51:32 +0000 (11:51 +0100)
As long as I investigate SCS.1m, this model reports to transfer/receive
PCM data channels/MIDI conformant data channels in tx/rx AMDTP packet.
There's a contradiction that this model actually has no analog/digital
capture port for PCM frames and no physical MIDI ports.

I guess that SCS.1d also has the contradiction. This model has no
analog/digital ports for PCM frames and no physical MIDI ports, thus it
requires no streaming functionality.

This commit adds some modification codes to handle the contradiction,
as much as possible. Unfortunately, this module adds one PCM playback
substream for SCS.1d so as SCS.1m.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/oxfw/oxfw.c

index e7f2698..abedc22 100644 (file)
@@ -164,8 +164,16 @@ static int detect_quirks(struct snd_oxfw *oxfw)
         * Stanton models supports asynchronous transactions for unique MIDI
         * messages.
         */
-       if (oxfw->entry->vendor_id == OUI_STANTON)
+       if (oxfw->entry->vendor_id == OUI_STANTON) {
+               /* No physical MIDI ports. */
+               oxfw->midi_input_ports = 0;
+               oxfw->midi_output_ports = 0;
+
+               /* Output stream exists but no data channels are useful. */
+               oxfw->has_output = false;
+
                return snd_oxfw_scs1x_add(oxfw);
+       }
 
        /*
         * TASCAM FireOne has physical control and requires a pair of additional