[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b38433a9-191f-9ba8-3230-edb962867bad@infradead.org>
Date: Sun, 18 Sep 2022 08:08:34 -0700
From: Randy Dunlap <rdunlap@...radead.org>
To: Jilin Yuan <yuanjilin@...rlc.com>, namit@...are.com,
pv-drivers@...are.com, arnd@...db.de, gregkh@...uxfoundation.org
Cc: linux-kernel@...r.kernel.org
Subject: Re: [PATCH] mm: fix repeated words in comments
On 9/18/22 03:01, Jilin Yuan wrote:
> Delete the redundant word 'on'.
>
> Signed-off-by: Jilin Yuan <yuanjilin@...rlc.com>
> ---
> drivers/misc/vmw_balloon.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/misc/vmw_balloon.c b/drivers/misc/vmw_balloon.c
> index 61a2be712bf7..c2e774f68133 100644
> --- a/drivers/misc/vmw_balloon.c
> +++ b/drivers/misc/vmw_balloon.c
> @@ -736,7 +736,7 @@ static int vmballoon_handle_one_result(struct vmballoon *b, struct page *page,
> * @p: pointer to where the page struct is returned.
> *
> * Following a lock or unlock operation, returns the status of the operation for
> - * an individual page. Provides the page that the operation was performed on on
> + * an individual page. Provides the page that the operation was performed on
This would make more sense to me: (s/on on/on in/)
> + * an individual page. Provides the page that the operation was performed on in
> * the @page argument.
> *
> * Returns: The status of a lock or unlock operation for an individual page.
--
~Randy
Powered by blists - more mailing lists