diff mbox

[3/6] fireworks: Improve comments about Fireworks transaction

Message ID 1401863137-15103-4-git-send-email-o-takashi@sakamocchi.jp (mailing list archive)
State Accepted
Commit 72f784f7d0da3072ee9ad9b31b88094bc0abc2df
Headers show

Commit Message

Takashi Sakamoto June 4, 2014, 6:25 a.m. UTC
It includes descriptions to cause misreading.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
---
 sound/firewire/fireworks/fireworks_transaction.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

Comments

Takashi Iwai June 4, 2014, 12:39 p.m. UTC | #1
At Wed,  4 Jun 2014 15:25:34 +0900,
Takashi Sakamoto wrote:
> 
> It includes descriptions to cause misreading.
> 
> Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>

Applied, thanks.


Takashi

> ---
>  sound/firewire/fireworks/fireworks_transaction.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/sound/firewire/fireworks/fireworks_transaction.c b/sound/firewire/fireworks/fireworks_transaction.c
> index a6a9e9f..255dabc 100644
> --- a/sound/firewire/fireworks/fireworks_transaction.c
> +++ b/sound/firewire/fireworks/fireworks_transaction.c
> @@ -8,19 +8,19 @@
>  
>  /*
>   * Fireworks have its own transaction. The transaction can be delivered by AV/C
> - * Vendor Specific command. But at least Windows driver and firmware version 5.5
> - * or later don't use it.
> + * Vendor Specific command frame or usual asynchronous transaction. At least,
> + * Windows driver and firmware version 5.5 or later don't use AV/C command.
>   *
>   * Transaction substance:
> - *  At first, 6 data exist. Following to the 6 data, parameters for each
> - *  commands exists. All of parameters are 32 bit alighed to big endian.
> + *  At first, 6 data exist. Following to the data, parameters for each command
> + *  exist. All of the parameters are 32 bit alighed to big endian.
>   *   data[0]:	Length of transaction substance
>   *   data[1]:	Transaction version
>   *   data[2]:	Sequence number. This is incremented by the device
> - *   data[3]:	transaction category
> - *   data[4]:	transaction command
> - *   data[5]:	return value in response.
> - *   data[6-]:	parameters
> + *   data[3]:	Transaction category
> + *   data[4]:	Transaction command
> + *   data[5]:	Return value in response.
> + *   data[6-]:	Parameters
>   *
>   * Transaction address:
>   *  command:	0xecc000000000
> -- 
> 1.8.3.2
>
diff mbox

Patch

diff --git a/sound/firewire/fireworks/fireworks_transaction.c b/sound/firewire/fireworks/fireworks_transaction.c
index a6a9e9f..255dabc 100644
--- a/sound/firewire/fireworks/fireworks_transaction.c
+++ b/sound/firewire/fireworks/fireworks_transaction.c
@@ -8,19 +8,19 @@ 
 
 /*
  * Fireworks have its own transaction. The transaction can be delivered by AV/C
- * Vendor Specific command. But at least Windows driver and firmware version 5.5
- * or later don't use it.
+ * Vendor Specific command frame or usual asynchronous transaction. At least,
+ * Windows driver and firmware version 5.5 or later don't use AV/C command.
  *
  * Transaction substance:
- *  At first, 6 data exist. Following to the 6 data, parameters for each
- *  commands exists. All of parameters are 32 bit alighed to big endian.
+ *  At first, 6 data exist. Following to the data, parameters for each command
+ *  exist. All of the parameters are 32 bit alighed to big endian.
  *   data[0]:	Length of transaction substance
  *   data[1]:	Transaction version
  *   data[2]:	Sequence number. This is incremented by the device
- *   data[3]:	transaction category
- *   data[4]:	transaction command
- *   data[5]:	return value in response.
- *   data[6-]:	parameters
+ *   data[3]:	Transaction category
+ *   data[4]:	Transaction command
+ *   data[5]:	Return value in response.
+ *   data[6-]:	Parameters
  *
  * Transaction address:
  *  command:	0xecc000000000