* [PATCH] greybus: es2: fix typo in a comment
@ 2021-12-12 3:16 Jason Wang
2021-12-13 20:01 ` Alex Elder
0 siblings, 1 reply; 2+ messages in thread
From: Jason Wang @ 2021-12-12 3:16 UTC (permalink / raw)
To: elder; +Cc: johan, gregkh, greybus-dev, linux-kernel, Jason Wang
The double `for' in the comment in line 81 is repeated. Remove one
of them from the comment.
Signed-off-by: Jason Wang <wangborong@cdjrlc.com>
---
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
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] greybus: es2: fix typo in a comment
2021-12-12 3:16 [PATCH] greybus: es2: fix typo in a comment Jason Wang
@ 2021-12-13 20:01 ` Alex Elder
0 siblings, 0 replies; 2+ messages in thread
From: Alex Elder @ 2021-12-13 20:01 UTC (permalink / raw)
To: Jason Wang, elder; +Cc: johan, gregkh, greybus-dev, linux-kernel
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 <elder@linaro.org>
> Signed-off-by: Jason Wang <wangborong@cdjrlc.com>
> ---
> 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.
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-12-13 20:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-12 3:16 [PATCH] greybus: es2: fix typo in a comment Jason Wang
2021-12-13 20:01 ` Alex Elder
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).