ALSA: firewire-motu: add model-specific table of chunk count
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Tue, 19 May 2020 11:16:33 +0000 (20:16 +0900)
committerTakashi Iwai <tiwai@suse.de>
Fri, 22 May 2020 14:52:14 +0000 (16:52 +0200)
In MOTU protocol, data block consists of SPH and 24-bit chunks
aligned to quadlet. The number of chunks per data block is specific
to model. For models with optical interface, the number differs
depending on I/O settings for the interface (ADAT, TOSLINK).

Currently the number is calculated from flags in model-specific
data. However this is weak in the case that the model has quirks.
Actually, for quirks of some models, flags are used against their
original meanings.

This commit adds model-specific table of chunk count. For future
integration, this table is based on the calculation.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Link: https://lore.kernel.org/r/20200519111641.123211-7-o-takashi@sakamocchi.jp
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/firewire/motu/motu-protocol-v2.c
sound/firewire/motu/motu-protocol-v3.c
sound/firewire/motu/motu.h

index e2913cb54da89a6b363fbd13af4452700e77a75b..6cd7a5f1f46cf0765cfa0b07b2018495b9b23fff 100644 (file)
@@ -306,7 +306,8 @@ const struct snd_motu_spec snd_motu_spec_828mk2 = {
                 SND_MOTU_SPEC_HAS_OPT_IFACE_A |
                 SND_MOTU_SPEC_RX_MIDI_2ND_Q |
                 SND_MOTU_SPEC_TX_MIDI_2ND_Q,
-
+       .tx_fixed_pcm_chunks = {14, 14, 0},
+       .rx_fixed_pcm_chunks = {14, 14, 0},
        .analog_in_ports = 8,
        .analog_out_ports = 8,
 };
@@ -321,7 +322,8 @@ const struct snd_motu_spec snd_motu_spec_traveler = {
                 SND_MOTU_SPEC_HAS_OPT_IFACE_A |
                 SND_MOTU_SPEC_RX_MIDI_2ND_Q |
                 SND_MOTU_SPEC_TX_MIDI_2ND_Q,
-
+       .tx_fixed_pcm_chunks = {14, 14, 8},
+       .rx_fixed_pcm_chunks = {14, 14, 8},
        .analog_in_ports = 8,
        .analog_out_ports = 8,
 };
@@ -335,6 +337,8 @@ const struct snd_motu_spec snd_motu_spec_ultralite = {
                 SND_MOTU_SPEC_RX_MIDI_2ND_Q |
                 SND_MOTU_SPEC_TX_MIDI_2ND_Q |
                 SND_MOTU_SPEC_RX_SEPARATED_MAIN,
+       .tx_fixed_pcm_chunks = {14, 14, 0},
+       .rx_fixed_pcm_chunks = {14, 14, 0},
        .analog_in_ports = 8,
        .analog_out_ports = 8,
 };
@@ -349,6 +353,8 @@ const struct snd_motu_spec snd_motu_spec_8pre = {
                 SND_MOTU_SPEC_HAS_OPT_IFACE_B |
                 SND_MOTU_SPEC_RX_MIDI_2ND_Q |
                 SND_MOTU_SPEC_TX_MIDI_2ND_Q,
+       .tx_fixed_pcm_chunks = {10, 6, 0},
+       .rx_fixed_pcm_chunks = {10, 6, 0},
        .analog_in_ports = 8,
        .analog_out_ports = 2,
 };
index 8192bcdd1b3720e2cc91371f83c917aa5817d790..370d22da276ecf31fed1503764cba9d1b6e5ed3e 100644 (file)
@@ -324,7 +324,8 @@ const struct snd_motu_spec snd_motu_spec_828mk3 = {
                 SND_MOTU_SPEC_HAS_OPT_IFACE_B |
                 SND_MOTU_SPEC_RX_MIDI_3RD_Q |
                 SND_MOTU_SPEC_TX_MIDI_3RD_Q,
-
+       .tx_fixed_pcm_chunks = {18, 18, 14},
+       .rx_fixed_pcm_chunks = {14, 14, 10},
        .analog_in_ports = 8,
        .analog_out_ports = 8,
 };
@@ -338,6 +339,8 @@ const struct snd_motu_spec snd_motu_spec_audio_express = {
                 SND_MOTU_SPEC_RX_SEPARATED_MAIN |
                 SND_MOTU_SPEC_RX_MIDI_2ND_Q |
                 SND_MOTU_SPEC_TX_MIDI_3RD_Q,
+       .tx_fixed_pcm_chunks = {10, 10, 0},
+       .rx_fixed_pcm_chunks = {10, 10, 0},
        .analog_in_ports = 2,
        .analog_out_ports = 4,
 };
@@ -349,6 +352,8 @@ const struct snd_motu_spec snd_motu_spec_4pre = {
                 SND_MOTU_SPEC_TX_MICINST_CHUNK |
                 SND_MOTU_SPEC_TX_RETURN_CHUNK |
                 SND_MOTU_SPEC_RX_SEPARATED_MAIN,
+       .tx_fixed_pcm_chunks = {10, 10, 0},
+       .rx_fixed_pcm_chunks = {10, 10, 0},
        .analog_in_ports = 2,
        .analog_out_ports = 2,
 };
index f8274ed94766cd268d5d71070b85454638bca489..790aa34d396fd8652dcbf1ecc67577cc7de04e33 100644 (file)
@@ -118,6 +118,9 @@ struct snd_motu_spec {
        enum snd_motu_protocol_version protocol_version;
        enum snd_motu_spec_flags flags;
 
+       unsigned char tx_fixed_pcm_chunks[3];
+       unsigned char rx_fixed_pcm_chunks[3];
+
        unsigned char analog_in_ports;
        unsigned char analog_out_ports;
 };