2021-12-12 03:17:23

by Jason Wang

Subject: [PATCH] greybus: es2: fix typo in a comment

The double `for' in the comment in line 81 is repeated. Remove one
of them from the comment.

Signed-off-by: Jason Wang <[email protected]>
---
drivers/greybus/es2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/greybus/es2.c b/drivers/greybus/es2.c
index 15661c7f3633..e89cca015095 100644
--- a/drivers/greybus/es2.c
+++ b/drivers/greybus/es2.c
@@ -78,7 +78,7 @@ struct es2_cport_in {
* @hd: pointer to our gb_host_device structure
*
* @cport_in: endpoint, urbs and buffer for cport in messages
- * @cport_out_endpoint: endpoint for for cport out messages
+ * @cport_out_endpoint: endpoint for cport out messages
* @cport_out_urb: array of urbs for the CPort out messages
* @cport_out_urb_busy: array of flags to see if the @cport_out_urb is busy or
* not.
--
2.34.1

2021-12-13 20:02:02

by Alex Elder

Subject: Re: [PATCH] greybus: es2: fix typo in a comment

On 12/11/21 9:16 PM, Jason Wang wrote:
> The double `for' in the comment in line 81 is repeated. Remove one
> of them from the comment.

This looks fine. But it's so trivial... Are you aware
of *any* other similar trivial problems in comments that
could be fixed together with this? If so, I would prefer
that.

If you've looked, and there are no others:

Reviewed-by: Alex Elder <[email protected]>


> Signed-off-by: Jason Wang <[email protected]>
> ---
> drivers/greybus/es2.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/greybus/es2.c b/drivers/greybus/es2.c
> index 15661c7f3633..e89cca015095 100644
> --- a/drivers/greybus/es2.c
> +++ b/drivers/greybus/es2.c
> @@ -78,7 +78,7 @@ struct es2_cport_in {
> * @hd: pointer to our gb_host_device structure
> *
> * @cport_in: endpoint, urbs and buffer for cport in messages
> - * @cport_out_endpoint: endpoint for for cport out messages
> + * @cport_out_endpoint: endpoint for cport out messages
> * @cport_out_urb: array of urbs for the CPort out messages
> * @cport_out_urb_busy: array of flags to see if the @cport_out_urb is busy or
> * not.
>