2021-12-05 16:17:21

by Xiang wangx

Subject: [PATCH] tty/ldsem: Fix syntax errors in comments

Delete the redundant word 'are'

Signed-off-by: Xiang wangx <[email protected]>
---
drivers/tty/tty_ldsem.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/tty/tty_ldsem.c b/drivers/tty/tty_ldsem.c
index ce8291053af3..3be428c16260 100644
--- a/drivers/tty/tty_ldsem.c
+++ b/drivers/tty/tty_ldsem.c
@@ -163,7 +163,7 @@ down_read_failed(struct ld_semaphore *sem, long count, long timeout)

/*
* Try to reverse the lock attempt but if the count has changed
- * so that reversing fails, check if there are are no waiters,
+ * so that reversing fails, check if there are no waiters,
* and early-out if not
*/
do {
--
2.20.1

2021-12-08 06:21:23

by Jiri Slaby

Subject: Re: [PATCH] tty/ldsem: Fix syntax errors in comments

On 05. 12. 21, 17:15, Xiang wangx wrote:
> Delete the redundant word 'are'
>
> Signed-off-by: Xiang wangx <[email protected]>

Reviewed-by: Jiri Slaby <[email protected]>

Maybe you can add also a period to the end of the sentence.

> ---
> drivers/tty/tty_ldsem.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/tty/tty_ldsem.c b/drivers/tty/tty_ldsem.c
> index ce8291053af3..3be428c16260 100644
> --- a/drivers/tty/tty_ldsem.c
> +++ b/drivers/tty/tty_ldsem.c
> @@ -163,7 +163,7 @@ down_read_failed(struct ld_semaphore *sem, long count, long timeout)
>
> /*
> * Try to reverse the lock attempt but if the count has changed
> - * so that reversing fails, check if there are are no waiters,
> + * so that reversing fails, check if there are no waiters,
> * and early-out if not
> */
> do {
>


--
js