Commit 414ba022 authored by Takashi Sakamoto's avatar Takashi Sakamoto Committed by Takashi Iwai

ALSA: firewire-lib: add support arbitrary value for fmt/fdf fields in CIP header

Some vendor specific protocol uses its own value for fmt/fdf fields in
CIP header.

This commit support to set arbitrary values for the fields.

In IEC 61883-6, NO-DATA code is defined for FDF field. A packet with this
code includes no data even if it includes some data blocks. This commit
still leaves a condition to handle this special packet.
Signed-off-by: default avatarTakashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 20e44577
...@@ -59,8 +59,8 @@ ...@@ -59,8 +59,8 @@
* Audio and Music transfer protocol specific parameters * Audio and Music transfer protocol specific parameters
* only "Clock-based rate control mode" is supported * only "Clock-based rate control mode" is supported
*/ */
#define CIP_FMT_AM (0x10 << CIP_FMT_SHIFT) #define CIP_FMT_AM 0x10
#define AMDTP_FDF_AM824 (0 << (CIP_FDF_SHIFT + 3)) #define AMDTP_FDF_AM824 0x00
#define AMDTP_FDF_NO_DATA 0xff #define AMDTP_FDF_NO_DATA 0xff
/* TODO: make these configurable */ /* TODO: make these configurable */
...@@ -94,6 +94,8 @@ int amdtp_stream_init(struct amdtp_stream *s, struct fw_unit *unit, ...@@ -94,6 +94,8 @@ int amdtp_stream_init(struct amdtp_stream *s, struct fw_unit *unit,
s->callbacked = false; s->callbacked = false;
s->sync_slave = NULL; s->sync_slave = NULL;
s->fmt = CIP_FMT_AM;
return 0; return 0;
} }
EXPORT_SYMBOL(amdtp_stream_init); EXPORT_SYMBOL(amdtp_stream_init);
...@@ -225,6 +227,8 @@ int amdtp_stream_set_parameters(struct amdtp_stream *s, ...@@ -225,6 +227,8 @@ int amdtp_stream_set_parameters(struct amdtp_stream *s,
s->data_block_quadlets = s->pcm_channels + midi_channels; s->data_block_quadlets = s->pcm_channels + midi_channels;
s->midi_ports = midi_ports; s->midi_ports = midi_ports;
s->fdf = AMDTP_FDF_AM824 | s->sfc;
/* /*
* In IEC 61883-6, one data block represents one event. In ALSA, one * In IEC 61883-6, one data block represents one event. In ALSA, one
* event equals to one PCM frame. But Dice has a quirk at higher * event equals to one PCM frame. But Dice has a quirk at higher
...@@ -691,8 +695,10 @@ static int handle_out_packet(struct amdtp_stream *s, unsigned int data_blocks, ...@@ -691,8 +695,10 @@ static int handle_out_packet(struct amdtp_stream *s, unsigned int data_blocks,
buffer[0] = cpu_to_be32(ACCESS_ONCE(s->source_node_id_field) | buffer[0] = cpu_to_be32(ACCESS_ONCE(s->source_node_id_field) |
(s->data_block_quadlets << CIP_DBS_SHIFT) | (s->data_block_quadlets << CIP_DBS_SHIFT) |
s->data_block_counter); s->data_block_counter);
buffer[1] = cpu_to_be32(CIP_EOH | CIP_FMT_AM | AMDTP_FDF_AM824 | buffer[1] = cpu_to_be32(CIP_EOH |
(s->sfc << CIP_FDF_SHIFT) | syt); ((s->fmt << CIP_FMT_SHIFT) & CIP_FMT_MASK) |
((s->fdf << CIP_FDF_SHIFT) & CIP_FDF_MASK) |
(syt & CIP_SYT_MASK));
s->data_block_counter = (s->data_block_counter + data_blocks) & 0xff; s->data_block_counter = (s->data_block_counter + data_blocks) & 0xff;
...@@ -732,6 +738,7 @@ static int handle_in_packet(struct amdtp_stream *s, ...@@ -732,6 +738,7 @@ static int handle_in_packet(struct amdtp_stream *s,
unsigned int *data_blocks, unsigned int syt) unsigned int *data_blocks, unsigned int syt)
{ {
u32 cip_header[2]; u32 cip_header[2];
unsigned int fmt, fdf;
unsigned int data_block_quadlets, data_block_counter, dbc_interval; unsigned int data_block_quadlets, data_block_counter, dbc_interval;
struct snd_pcm_substream *pcm; struct snd_pcm_substream *pcm;
unsigned int pcm_frames; unsigned int pcm_frames;
...@@ -745,8 +752,7 @@ static int handle_in_packet(struct amdtp_stream *s, ...@@ -745,8 +752,7 @@ static int handle_in_packet(struct amdtp_stream *s,
* For convenience, also check FMT field is AM824 or not. * For convenience, also check FMT field is AM824 or not.
*/ */
if (((cip_header[0] & CIP_EOH_MASK) == CIP_EOH) || if (((cip_header[0] & CIP_EOH_MASK) == CIP_EOH) ||
((cip_header[1] & CIP_EOH_MASK) != CIP_EOH) || ((cip_header[1] & CIP_EOH_MASK) != CIP_EOH)) {
((cip_header[1] & CIP_FMT_MASK) != CIP_FMT_AM)) {
dev_info_ratelimited(&s->unit->device, dev_info_ratelimited(&s->unit->device,
"Invalid CIP header for AMDTP: %08X:%08X\n", "Invalid CIP header for AMDTP: %08X:%08X\n",
cip_header[0], cip_header[1]); cip_header[0], cip_header[1]);
...@@ -755,10 +761,19 @@ static int handle_in_packet(struct amdtp_stream *s, ...@@ -755,10 +761,19 @@ static int handle_in_packet(struct amdtp_stream *s,
goto end; goto end;
} }
/* Check valid protocol or not. */
fmt = (cip_header[1] & CIP_FMT_MASK) >> CIP_FMT_SHIFT;
if (fmt != s->fmt) {
dev_err(&s->unit->device,
"Detect unexpected protocol: %08x %08x\n",
cip_header[0], cip_header[1]);
return -EIO;
}
/* Calculate data blocks */ /* Calculate data blocks */
fdf = (cip_header[1] & CIP_FDF_MASK) >> CIP_FDF_SHIFT;
if (payload_quadlets < 3 || if (payload_quadlets < 3 ||
((cip_header[1] & CIP_FDF_MASK) == (fmt == CIP_FMT_AM && fdf == AMDTP_FDF_NO_DATA)) {
(AMDTP_FDF_NO_DATA << CIP_FDF_SHIFT))) {
*data_blocks = 0; *data_blocks = 0;
} else { } else {
data_block_quadlets = data_block_quadlets =
......
...@@ -128,6 +128,8 @@ struct amdtp_stream { ...@@ -128,6 +128,8 @@ struct amdtp_stream {
unsigned int source_node_id_field; unsigned int source_node_id_field;
unsigned int data_block_quadlets; unsigned int data_block_quadlets;
unsigned int data_block_counter; unsigned int data_block_counter;
unsigned int fmt;
unsigned int fdf;
/* quirk: fixed interval of dbc between previos/current packets. */ /* quirk: fixed interval of dbc between previos/current packets. */
unsigned int tx_dbc_interval; unsigned int tx_dbc_interval;
/* quirk: indicate the value of dbc field in a first packet. */ /* quirk: indicate the value of dbc field in a first packet. */
......
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