2011-02-25 06:11:38

by Justin P. Mattock

Subject: [PATCH 06/21]drivers:media:cx23418.h remove one to many l's in the word.

The patch below removes an extra "l" in the word.

Signed-off-by: Justin P. Mattock <[email protected]>

---
drivers/media/video/cx18/cx23418.h | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/media/video/cx18/cx23418.h b/drivers/media/video/cx18/cx23418.h
index 7e40035..935f557 100644
--- a/drivers/media/video/cx18/cx23418.h
+++ b/drivers/media/video/cx18/cx23418.h
@@ -477,7 +477,7 @@
/* The are no buffers ready. Try again soon! */
#define CXERR_NODATA_AGAIN 0x00001E

-/* The stream is stopping. Function not alllowed now! */
+/* The stream is stopping. Function not allowed now! */
#define CXERR_STOPPING_STATUS 0x00001F

/* Trying to access hardware when the power is turned OFF */
--
1.7.4.1

2011-02-26 00:54:50

by Andy Walls

Subject: Re: [PATCH 06/21]drivers:media:cx23418.h remove one to many l's in the word.

On Thu, 2011-02-24 at 22:11 -0800, Justin P. Mattock wrote:
> The patch below removes an extra "l" in the word.
>
> Signed-off-by: Justin P. Mattock <[email protected]>
>
> ---
> drivers/media/video/cx18/cx23418.h | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/media/video/cx18/cx23418.h b/drivers/media/video/cx18/cx23418.h
> index 7e40035..935f557 100644
> --- a/drivers/media/video/cx18/cx23418.h
> +++ b/drivers/media/video/cx18/cx23418.h
> @@ -477,7 +477,7 @@
> /* The are no buffers ready. Try again soon! */
> #define CXERR_NODATA_AGAIN 0x00001E
>
> -/* The stream is stopping. Function not alllowed now! */
> +/* The stream is stopping. Function not allowed now! */

If the spelling mistake is worthy of a fix, why isn't the egregious
grammar mistake also worth fixing?

Also, those aren't the only spelling and grammar mistakes in comments in
that file.

-Andy

> #define CXERR_STOPPING_STATUS 0x00001F
>
> /* Trying to access hardware when the power is turned OFF */