linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] sound/firewire: Fix trivial typos in comments
@ 2015-03-09 21:13 Yannick Guerrini
  2015-03-10 11:40 ` Takashi Iwai
  0 siblings, 1 reply; 2+ messages in thread
From: Yannick Guerrini @ 2015-03-09 21:13 UTC (permalink / raw)
  To: clemens; +Cc: perex, tiwai, trivial, alsa-devel, linux-kernel, Yannick Guerrini

Change 'propper' to 'proper'
Change 'paramters' to 'parameters'
Change 'SYT_INTEVAL' to 'SYT_INTERVAL'
Change 'aligh'/'alighed' to 'align'/'aligned'

Signed-off-by: Yannick Guerrini <yguerrini@tomshardware.fr>
---
 sound/firewire/amdtp.c                           | 8 ++++----
 sound/firewire/fireworks/fireworks_transaction.c | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/sound/firewire/amdtp.c b/sound/firewire/amdtp.c
index 5cc356d..e061355 100644
--- a/sound/firewire/amdtp.c
+++ b/sound/firewire/amdtp.c
@@ -166,10 +166,10 @@ int amdtp_stream_add_pcm_hw_constraints(struct amdtp_stream *s,
 	 * One AMDTP packet can include some frames. In blocking mode, the
 	 * number equals to SYT_INTERVAL. So the number is 8, 16 or 32,
 	 * depending on its sampling rate. For accurate period interrupt, it's
-	 * preferrable to aligh period/buffer sizes to current SYT_INTERVAL.
+	 * preferrable to align period/buffer sizes to current SYT_INTERVAL.
 	 *
-	 * TODO: These constraints can be improved with propper rules.
-	 * Currently apply LCM of SYT_INTEVALs.
+	 * TODO: These constraints can be improved with proper rules.
+	 * Currently apply LCM of SYT_INTERVALs.
 	 */
 	err = snd_pcm_hw_constraint_step(runtime, 0,
 					 SNDRV_PCM_HW_PARAM_PERIOD_SIZE, 32);
@@ -270,7 +270,7 @@ static void amdtp_read_s32(struct amdtp_stream *s,
  * @s: the AMDTP stream to configure
  * @format: the format of the ALSA PCM device
  *
- * The sample format must be set after the other paramters (rate/PCM channels/
+ * The sample format must be set after the other parameters (rate/PCM channels/
  * MIDI) and before the stream is started, and must not be changed while the
  * stream is running.
  */
diff --git a/sound/firewire/fireworks/fireworks_transaction.c b/sound/firewire/fireworks/fireworks_transaction.c
index 2a85e42..f550808 100644
--- a/sound/firewire/fireworks/fireworks_transaction.c
+++ b/sound/firewire/fireworks/fireworks_transaction.c
@@ -13,7 +13,7 @@
  *
  * Transaction substance:
  *  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.
+ *  exist. All of the parameters are 32 bit aligned to big endian.
  *   data[0]:	Length of transaction substance
  *   data[1]:	Transaction version
  *   data[2]:	Sequence number. This is incremented by the device
-- 
1.9.5.msysgit.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] sound/firewire: Fix trivial typos in comments
  2015-03-09 21:13 [PATCH] sound/firewire: Fix trivial typos in comments Yannick Guerrini
@ 2015-03-10 11:40 ` Takashi Iwai
  0 siblings, 0 replies; 2+ messages in thread
From: Takashi Iwai @ 2015-03-10 11:40 UTC (permalink / raw)
  To: Yannick Guerrini; +Cc: clemens, perex, trivial, alsa-devel, linux-kernel

At Mon, 9 Mar 2015 22:13:03 +0100,
Yannick Guerrini wrote:
> 
> Change 'propper' to 'proper'
> Change 'paramters' to 'parameters'
> Change 'SYT_INTEVAL' to 'SYT_INTERVAL'
> Change 'aligh'/'alighed' to 'align'/'aligned'
> 
> Signed-off-by: Yannick Guerrini <yguerrini@tomshardware.fr>

Applied, thanks.


Takashi

> ---
>  sound/firewire/amdtp.c                           | 8 ++++----
>  sound/firewire/fireworks/fireworks_transaction.c | 2 +-
>  2 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/sound/firewire/amdtp.c b/sound/firewire/amdtp.c
> index 5cc356d..e061355 100644
> --- a/sound/firewire/amdtp.c
> +++ b/sound/firewire/amdtp.c
> @@ -166,10 +166,10 @@ int amdtp_stream_add_pcm_hw_constraints(struct amdtp_stream *s,
>  	 * One AMDTP packet can include some frames. In blocking mode, the
>  	 * number equals to SYT_INTERVAL. So the number is 8, 16 or 32,
>  	 * depending on its sampling rate. For accurate period interrupt, it's
> -	 * preferrable to aligh period/buffer sizes to current SYT_INTERVAL.
> +	 * preferrable to align period/buffer sizes to current SYT_INTERVAL.
>  	 *
> -	 * TODO: These constraints can be improved with propper rules.
> -	 * Currently apply LCM of SYT_INTEVALs.
> +	 * TODO: These constraints can be improved with proper rules.
> +	 * Currently apply LCM of SYT_INTERVALs.
>  	 */
>  	err = snd_pcm_hw_constraint_step(runtime, 0,
>  					 SNDRV_PCM_HW_PARAM_PERIOD_SIZE, 32);
> @@ -270,7 +270,7 @@ static void amdtp_read_s32(struct amdtp_stream *s,
>   * @s: the AMDTP stream to configure
>   * @format: the format of the ALSA PCM device
>   *
> - * The sample format must be set after the other paramters (rate/PCM channels/
> + * The sample format must be set after the other parameters (rate/PCM channels/
>   * MIDI) and before the stream is started, and must not be changed while the
>   * stream is running.
>   */
> diff --git a/sound/firewire/fireworks/fireworks_transaction.c b/sound/firewire/fireworks/fireworks_transaction.c
> index 2a85e42..f550808 100644
> --- a/sound/firewire/fireworks/fireworks_transaction.c
> +++ b/sound/firewire/fireworks/fireworks_transaction.c
> @@ -13,7 +13,7 @@
>   *
>   * Transaction substance:
>   *  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.
> + *  exist. All of the parameters are 32 bit aligned to big endian.
>   *   data[0]:	Length of transaction substance
>   *   data[1]:	Transaction version
>   *   data[2]:	Sequence number. This is incremented by the device
> -- 
> 1.9.5.msysgit.0
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-03-10 11:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-09 21:13 [PATCH] sound/firewire: Fix trivial typos in comments Yannick Guerrini
2015-03-10 11:40 ` Takashi Iwai

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).