2022-06-06 05:40:04

by Xiang wangx

Subject: [PATCH] RDMA/hw/hfi1/pio_copy: Fix syntax errors in comments

Delete the redundant word 'and'.

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

diff --git a/drivers/infiniband/hw/hfi1/pio_copy.c b/drivers/infiniband/hw/hfi1/pio_copy.c
index 136f9a99e1e0..7690f996d5e3 100644
--- a/drivers/infiniband/hw/hfi1/pio_copy.c
+++ b/drivers/infiniband/hw/hfi1/pio_copy.c
@@ -172,7 +172,7 @@ static inline void jcopy(u8 *dest, const u8 *src, u32 n)
}

/*
- * Read nbytes from "from" and and place them in the low bytes
+ * Read nbytes from "from" and place them in the low bytes
* of pbuf->carry. Other bytes are left as-is. Any previous
* value in pbuf->carry is lost.
*
--
2.36.1

2022-06-06 12:04:04

by Dennis Dalessandro

Subject: Re: [PATCH] RDMA/hw/hfi1/pio_copy: Fix syntax errors in comments

On 6/5/22 5:05 AM, Xiang wangx wrote:
> Delete the redundant word 'and'.
>
> Signed-off-by: Xiang wangx <[email protected]>
> ---
> drivers/infiniband/hw/hfi1/pio_copy.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/infiniband/hw/hfi1/pio_copy.c b/drivers/infiniband/hw/hfi1/pio_copy.c
> index 136f9a99e1e0..7690f996d5e3 100644
> --- a/drivers/infiniband/hw/hfi1/pio_copy.c
> +++ b/drivers/infiniband/hw/hfi1/pio_copy.c
> @@ -172,7 +172,7 @@ static inline void jcopy(u8 *dest, const u8 *src, u32 n)
> }
>
> /*
> - * Read nbytes from "from" and and place them in the low bytes
> + * Read nbytes from "from" and place them in the low bytes
> * of pbuf->carry. Other bytes are left as-is. Any previous
> * value in pbuf->carry is lost.
> *

Don't know that I would call this a "syntax" error. Perhaps change subject to
say "Fix typo in comment"

-Denny