openkylin
/
linux
mirror of https://gitee.com/openkylin/linux.git
9
0
Fork 0
Code Issues Projects Releases Wiki Activity

ALSA: firewire-lib: Add support for deferred transaction 

Some devices based on BeBoB use this type of AV/C transaction.

'Deferred Transaction' is defined in 'AV/C Digital Interface Command Set
General Specification' and is used by targets to make a response deferred
during processing it.

If a target may not be able to complete a command within 100msec since
receiving the command, then the target shall return INTERIM response,
to which final response will follow later. CONTROL/NOTIFY commands are
allowed for deferred transaction.

In the specification, devices allow to send INTERIM response just one time.
But this commit allows to handle several INTERIM response with two reasons.
One reason is to simplify codes, and another reason is to prepare for
devices which is out of specification.

There is an issue. In the specification, the interval between INTERIM
response and final response is 'Unspecified interval'. The specification
depends on each subunit specification for this interval.

But we promise to finish this function for caller. In this reason, I use
FCP_TIMEOUT_MS for this interval. Currently it's 125msec. When we find
devices which needs more time for this interval, then let us add some codes
to apply more interval for 'Unspecified interval'.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
Takashi Sakamoto 2014-04-25 22:44:58 +09:00 committed by Takashi Iwai
parent b04479fb85
commit 00a7bb81c2
1 changed files with 28 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
sound/firewire/fcp.c
View File

@ -30,6 +30,7 @@ enum fcp_state {
STATE_PENDING, STATE_PENDING,
STATE_BUS_RESET, STATE_BUS_RESET,
STATE_COMPLETE, STATE_COMPLETE,
STATE_DEFERRED,
}; };
struct fcp_transaction { struct fcp_transaction {
@ -40,6 +41,7 @@ struct fcp_transaction {
unsigned int response_match_bytes; unsigned int response_match_bytes;
enum fcp_state state; enum fcp_state state;
wait_queue_head_t wait; wait_queue_head_t wait;
bool deferrable;
}; };
/** /**
@ -81,6 +83,9 @@ int fcp_avc_transaction(struct fw_unit *unit,
t.state = STATE_PENDING; t.state = STATE_PENDING;
init_waitqueue_head(&t.wait); init_waitqueue_head(&t.wait);
if (*(const u8 *)command == 0x00 || *(const u8 *)command == 0x03)
t.deferrable = true;
spin_lock_irq(&transactions_lock); spin_lock_irq(&transactions_lock);
list_add_tail(&t.list, &transactions); list_add_tail(&t.list, &transactions);
spin_unlock_irq(&transactions_lock); spin_unlock_irq(&transactions_lock);
@ -93,11 +98,21 @@ int fcp_avc_transaction(struct fw_unit *unit,
(void *)command, command_size, 0); (void *)command, command_size, 0);
if (ret < 0) if (ret < 0)
break; break;
deferred:
wait_event_timeout(t.wait, t.state != STATE_PENDING, wait_event_timeout(t.wait, t.state != STATE_PENDING,
msecs_to_jiffies(FCP_TIMEOUT_MS)); msecs_to_jiffies(FCP_TIMEOUT_MS));
if (t.state == STATE_COMPLETE) { if (t.state == STATE_DEFERRED) {
/*
* 'AV/C General Specification' define no time limit
* on command completion once an INTERIM response has
* been sent. but we promise to finish this function
* for a caller. Here we use FCP_TIMEOUT_MS for next
* interval. This is not in the specification.
*/
t.state = STATE_PENDING;
goto deferred;
} else if (t.state == STATE_COMPLETE) {
ret = t.response_size; ret = t.response_size;
break; break;
} else if (t.state == STATE_BUS_RESET) { } else if (t.state == STATE_BUS_RESET) {
@ -132,7 +147,8 @@ void fcp_bus_reset(struct fw_unit *unit)
spin_lock_irq(&transactions_lock); spin_lock_irq(&transactions_lock);
list_for_each_entry(t, &transactions, list) { list_for_each_entry(t, &transactions, list) {
if (t->unit == unit && if (t->unit == unit &&
t->state == STATE_PENDING) { (t->state == STATE_PENDING ||
t->state == STATE_DEFERRED)) {
t->state = STATE_BUS_RESET; t->state = STATE_BUS_RESET;
wake_up(&t->wait); wake_up(&t->wait);
} }
@ -186,10 +202,15 @@ static void fcp_response(struct fw_card *card, struct fw_request *request,
if (t->state == STATE_PENDING && if (t->state == STATE_PENDING &&
is_matching_response(t, data, length)) { is_matching_response(t, data, length)) {
t->state = STATE_COMPLETE; if (t->deferrable && *(const u8 *)data == 0x0f) {
t->response_size = min((unsigned int)length, t->state = STATE_DEFERRED;
t->response_size); } else {
memcpy(t->response_buffer, data, t->response_size); t->state = STATE_COMPLETE;
t->response_size = min_t(unsigned int, length,
t->response_size);
memcpy(t->response_buffer, data,
t->response_size);
}
wake_up(&t->wait); wake_up(&t->wait);
} }
} }