[alsa-devel] [PATCH] ALSA: firewire-lib: fix kerneldoc errors

Takashi Iwai tiwai at suse.de
Tue Nov 18 18:10:52 CET 2014


At Tue, 18 Nov 2014 23:59:40 +0900,
Takashi Sakamoto wrote:
> 
> Complete missing parameters, correct wrong reference, and add an explaination
> about the differences between the latest specification and our implementation.
> 
> Signed-off-by: Takashi Sakamoto <o-takashi at sakamocchi.jp>

Thanks, applied.


Takashi

> ---
>  sound/firewire/amdtp.h | 24 ++++++++++++++++++++++--
>  sound/firewire/cmp.c   |  2 ++
>  2 files changed, 24 insertions(+), 2 deletions(-)
> 
> diff --git a/sound/firewire/amdtp.h b/sound/firewire/amdtp.h
> index 4823c08..e6e8926 100644
> --- a/sound/firewire/amdtp.h
> +++ b/sound/firewire/amdtp.h
> @@ -23,7 +23,7 @@
>   *	corresponds to the end of event in the packet. Out of IEC 61883.
>   * @CIP_WRONG_DBS: Only for in-stream. The value of dbs is wrong in in-packets.
>   *	The value of data_block_quadlets is used instead of reported value.
> - * @SKIP_DBC_ZERO_CHECK: Only for in-stream.  Packets with zero in dbc is
> + * @CIP_SKIP_DBC_ZERO_CHECK: Only for in-stream.  Packets with zero in dbc is
>   *	skipped for detecting discontinuity.
>   * @CIP_SKIP_INIT_DBC_CHECK: Only for in-stream. The value of dbc in first
>   *	packet is not continuous from an initial value.
> @@ -43,7 +43,27 @@ enum cip_flags {
>  };
>  
>  /**
> - * enum cip_sfc - a stream's sample rate
> + * enum cip_sfc - supported Sampling Frequency Codes (SFCs)
> + * @CIP_SFC_32000:   32,000 data blocks
> + * @CIP_SFC_44100:   44,100 data blocks
> + * @CIP_SFC_48000:   48,000 data blocks
> + * @CIP_SFC_88200:   88,200 data blocks
> + * @CIP_SFC_96000:   96,000 data blocks
> + * @CIP_SFC_176400: 176,400 data blocks
> + * @CIP_SFC_192000: 192,000 data blocks
> + * @CIP_SFC_COUNT: the number of supported SFCs
> + *
> + * These values are used to show nominal Sampling Frequency Code in
> + * Format Dependent Field (FDF) of AMDTP packet header. In IEC 61883-6:2002,
> + * this code means the number of events per second. Actually the code
> + * represents the number of data blocks transferred per second in an AMDTP
> + * stream.
> + *
> + * In IEC 61883-6:2005, some extensions were added to support more types of
> + * data such as 'One Bit LInear Audio', therefore the meaning of SFC became
> + * different depending on the types.
> + *
> + * Currently our implementation is compatible with IEC 61883-6:2002.
>   */
>  enum cip_sfc {
>  	CIP_SFC_32000  = 0,
> diff --git a/sound/firewire/cmp.c b/sound/firewire/cmp.c
> index ba8df5a..ae3bc19 100644
> --- a/sound/firewire/cmp.c
> +++ b/sound/firewire/cmp.c
> @@ -114,6 +114,7 @@ static int pcr_modify(struct cmp_connection *c,
>   * cmp_connection_init - initializes a connection manager
>   * @c: the connection manager to initialize
>   * @unit: a unit of the target device
> + * @direction: input or output
>   * @pcr_index: the index of the iPCR/oPCR on the target device
>   */
>  int cmp_connection_init(struct cmp_connection *c,
> @@ -154,6 +155,7 @@ EXPORT_SYMBOL(cmp_connection_init);
>  /**
>   * cmp_connection_check_used - check connection is already esablished or not
>   * @c: the connection manager to be checked
> + * @used: the pointer to store the result of checking the connection
>   */
>  int cmp_connection_check_used(struct cmp_connection *c, bool *used)
>  {
> -- 
> 2.1.0
> 


More information about the Alsa-devel mailing list