Commit eb4a378f authored by Takashi Sakamoto's avatar Takashi Sakamoto Committed by Takashi Iwai

ALSA: fireworks: drop reuse of incoming packet parameter for ougoing packet parameter

On Fireworks board module of Echo Audio, TSB43Cx43A (IceLynx Micro, iCEM)
is used to process payload of isochronous packets. There's an public
document of this chip[1]. This document is for firmware programmers to
transfer/receive AMDTP with IEC60958 data format, however in clause 2.5,
2.6 and 2.7, we can see system design to utilize the sequence of value in
SYT field of CIP header. In clause 2.3, we can see the specification of
Audio Master Clock (MCLK) from iCEM.

Well, this clock is actually not used for sampling clock. This can be
confirmed when corresponding driver transfer random value as the sequence
of SYT field. Even if in this case, the unit generates proper sound.

Additionally, in unique command set for this board module, the format
of CIP is changed; for IEC 61883-6 mode which we use, and for Windows
Operating System. In the latter mode, the whole 32 bit field in second CIP
header from Windows driver is used to represent counter of packets (NO-DATA
code is still used for packets without data blocks). If the master clock
was physically used by DSP on the board module, the Windows driver must
have transferred correct sequence of SYT field.

Furthermore, as long as seeing capacities of AudioFire2, AudioFire4,
AudioFire8, AudioFirePre8 and AudioFire12, these models don't support
SYT-Match clock source.

Summary, we have no need to relate incoming/outgoing packets. This commit
drops reusing SYT sequence of incoming packets for outgoing packets.

[1] Using TSB43Cx43A: S/PDIF over 1394 (2003, Texus Instruments
Incorporated)
http://www.ti.com/analog/docs/litabsmultiplefilelist.tsp?literatureNumber=slla148&docCategoryId=1&familyId=361Signed-off-by: default avatarTakashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent c71283cb
...@@ -84,7 +84,6 @@ struct snd_efw { ...@@ -84,7 +84,6 @@ struct snd_efw {
unsigned int pcm_capture_channels[SND_EFW_MULTIPLIER_MODES]; unsigned int pcm_capture_channels[SND_EFW_MULTIPLIER_MODES];
unsigned int pcm_playback_channels[SND_EFW_MULTIPLIER_MODES]; unsigned int pcm_playback_channels[SND_EFW_MULTIPLIER_MODES];
struct amdtp_stream *master;
struct amdtp_stream tx_stream; struct amdtp_stream tx_stream;
struct amdtp_stream rx_stream; struct amdtp_stream rx_stream;
struct cmp_connection out_conn; struct cmp_connection out_conn;
......
...@@ -120,23 +120,6 @@ destroy_stream(struct snd_efw *efw, struct amdtp_stream *stream) ...@@ -120,23 +120,6 @@ destroy_stream(struct snd_efw *efw, struct amdtp_stream *stream)
cmp_connection_destroy(&efw->out_conn); cmp_connection_destroy(&efw->out_conn);
} }
static int
get_sync_mode(struct snd_efw *efw, enum cip_flags *sync_mode)
{
enum snd_efw_clock_source clock_source;
int err;
err = snd_efw_command_get_clock_source(efw, &clock_source);
if (err < 0)
return err;
if (clock_source == SND_EFW_CLOCK_SOURCE_SYTMATCH)
return -ENOSYS;
*sync_mode = CIP_SYNC_TO_DEVICE;
return 0;
}
static int static int
check_connection_used_by_others(struct snd_efw *efw, struct amdtp_stream *s) check_connection_used_by_others(struct snd_efw *efw, struct amdtp_stream *s)
{ {
...@@ -208,9 +191,6 @@ int snd_efw_stream_init_duplex(struct snd_efw *efw) ...@@ -208,9 +191,6 @@ int snd_efw_stream_init_duplex(struct snd_efw *efw)
int snd_efw_stream_start_duplex(struct snd_efw *efw, unsigned int rate) int snd_efw_stream_start_duplex(struct snd_efw *efw, unsigned int rate)
{ {
struct amdtp_stream *master, *slave;
unsigned int slave_substreams;
enum cip_flags sync_mode;
unsigned int curr_rate; unsigned int curr_rate;
int err = 0; int err = 0;
...@@ -218,32 +198,19 @@ int snd_efw_stream_start_duplex(struct snd_efw *efw, unsigned int rate) ...@@ -218,32 +198,19 @@ int snd_efw_stream_start_duplex(struct snd_efw *efw, unsigned int rate)
if (efw->playback_substreams == 0 && efw->capture_substreams == 0) if (efw->playback_substreams == 0 && efw->capture_substreams == 0)
goto end; goto end;
err = get_sync_mode(efw, &sync_mode);
if (err < 0)
goto end;
if (sync_mode == CIP_SYNC_TO_DEVICE) {
master = &efw->tx_stream;
slave = &efw->rx_stream;
slave_substreams = efw->playback_substreams;
} else {
master = &efw->rx_stream;
slave = &efw->tx_stream;
slave_substreams = efw->capture_substreams;
}
/* /*
* Considering JACK/FFADO streaming: * Considering JACK/FFADO streaming:
* TODO: This can be removed hwdep functionality becomes popular. * TODO: This can be removed hwdep functionality becomes popular.
*/ */
err = check_connection_used_by_others(efw, master); err = check_connection_used_by_others(efw, &efw->rx_stream);
if (err < 0) if (err < 0)
goto end; goto end;
/* packet queueing error */ /* packet queueing error */
if (amdtp_streaming_error(slave)) if (amdtp_streaming_error(&efw->tx_stream))
stop_stream(efw, slave); stop_stream(efw, &efw->tx_stream);
if (amdtp_streaming_error(master)) if (amdtp_streaming_error(&efw->rx_stream))
stop_stream(efw, master); stop_stream(efw, &efw->rx_stream);
/* stop streams if rate is different */ /* stop streams if rate is different */
err = snd_efw_command_get_sampling_rate(efw, &curr_rate); err = snd_efw_command_get_sampling_rate(efw, &curr_rate);
...@@ -252,20 +219,17 @@ int snd_efw_stream_start_duplex(struct snd_efw *efw, unsigned int rate) ...@@ -252,20 +219,17 @@ int snd_efw_stream_start_duplex(struct snd_efw *efw, unsigned int rate)
if (rate == 0) if (rate == 0)
rate = curr_rate; rate = curr_rate;
if (rate != curr_rate) { if (rate != curr_rate) {
stop_stream(efw, slave); stop_stream(efw, &efw->tx_stream);
stop_stream(efw, master); stop_stream(efw, &efw->rx_stream);
} }
/* master should be always running */ /* master should be always running */
if (!amdtp_stream_running(master)) { if (!amdtp_stream_running(&efw->rx_stream)) {
amdtp_stream_set_sync(sync_mode, master, slave);
efw->master = master;
err = snd_efw_command_set_sampling_rate(efw, rate); err = snd_efw_command_set_sampling_rate(efw, rate);
if (err < 0) if (err < 0)
goto end; goto end;
err = start_stream(efw, master, rate); err = start_stream(efw, &efw->rx_stream, rate);
if (err < 0) { if (err < 0) {
dev_err(&efw->unit->device, dev_err(&efw->unit->device,
"fail to start AMDTP master stream:%d\n", err); "fail to start AMDTP master stream:%d\n", err);
...@@ -274,12 +238,13 @@ int snd_efw_stream_start_duplex(struct snd_efw *efw, unsigned int rate) ...@@ -274,12 +238,13 @@ int snd_efw_stream_start_duplex(struct snd_efw *efw, unsigned int rate)
} }
/* start slave if needed */ /* start slave if needed */
if (slave_substreams > 0 && !amdtp_stream_running(slave)) { if (efw->capture_substreams > 0 &&
err = start_stream(efw, slave, rate); !amdtp_stream_running(&efw->tx_stream)) {
err = start_stream(efw, &efw->tx_stream, rate);
if (err < 0) { if (err < 0) {
dev_err(&efw->unit->device, dev_err(&efw->unit->device,
"fail to start AMDTP slave stream:%d\n", err); "fail to start AMDTP slave stream:%d\n", err);
stop_stream(efw, master); stop_stream(efw, &efw->rx_stream);
} }
} }
end: end:
...@@ -288,26 +253,11 @@ int snd_efw_stream_start_duplex(struct snd_efw *efw, unsigned int rate) ...@@ -288,26 +253,11 @@ int snd_efw_stream_start_duplex(struct snd_efw *efw, unsigned int rate)
void snd_efw_stream_stop_duplex(struct snd_efw *efw) void snd_efw_stream_stop_duplex(struct snd_efw *efw)
{ {
struct amdtp_stream *master, *slave; if (efw->capture_substreams == 0) {
unsigned int master_substreams, slave_substreams; stop_stream(efw, &efw->tx_stream);
if (efw->master == &efw->rx_stream) {
slave = &efw->tx_stream;
master = &efw->rx_stream;
slave_substreams = efw->capture_substreams;
master_substreams = efw->playback_substreams;
} else {
slave = &efw->rx_stream;
master = &efw->tx_stream;
slave_substreams = efw->playback_substreams;
master_substreams = efw->capture_substreams;
}
if (slave_substreams == 0) {
stop_stream(efw, slave);
if (master_substreams == 0) if (efw->playback_substreams == 0)
stop_stream(efw, master); stop_stream(efw, &efw->rx_stream);
} }
} }
......
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