2021-05-28 16:11:37

by navin

[permalink] [raw]
Subject: [PATCH] fpga: fpga-bridge: removed repeated word

Removed repeated word and.
Reported by checkpatch.

Signed-off-by: Navin Sankar Velliangiri <[email protected]>
---
drivers/fpga/fpga-bridge.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/fpga/fpga-bridge.c b/drivers/fpga/fpga-bridge.c
index e9266b2a357f..9ada1126faf7 100644
--- a/drivers/fpga/fpga-bridge.c
+++ b/drivers/fpga/fpga-bridge.c
@@ -228,7 +228,7 @@ EXPORT_SYMBOL_GPL(fpga_bridges_put);
* @info: fpga image specific information
* @bridge_list: list of FPGA bridges
*
- * Get an exclusive reference to the bridge and and it to the list.
+ * Get an exclusive reference to the bridge and it to the list.
*
* Return 0 for success, error code from of_fpga_bridge_get() othewise.
*/
@@ -258,7 +258,7 @@ EXPORT_SYMBOL_GPL(of_fpga_bridge_get_to_list);
* @info: fpga image specific information
* @bridge_list: list of FPGA bridges
*
- * Get an exclusive reference to the bridge and and it to the list.
+ * Get an exclusive reference to the bridge and it to the list.
*
* Return 0 for success, error code from fpga_bridge_get() othewise.
*/
--
2.31.1


2021-05-28 16:21:28

by Tom Rix

[permalink] [raw]
Subject: Re: [PATCH] fpga: fpga-bridge: removed repeated word


On 5/28/21 8:05 AM, Navin Sankar Velliangiri wrote:
> Removed repeated word and.
> Reported by checkpatch.
>
> Signed-off-by: Navin Sankar Velliangiri <[email protected]>
> ---
> drivers/fpga/fpga-bridge.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/fpga/fpga-bridge.c b/drivers/fpga/fpga-bridge.c
> index e9266b2a357f..9ada1126faf7 100644
> --- a/drivers/fpga/fpga-bridge.c
> +++ b/drivers/fpga/fpga-bridge.c
> @@ -228,7 +228,7 @@ EXPORT_SYMBOL_GPL(fpga_bridges_put);
> * @info: fpga image specific information
> * @bridge_list: list of FPGA bridges
> *
> - * Get an exclusive reference to the bridge and and it to the list.
> + * Get an exclusive reference to the bridge and it to the list.
> *
> * Return 0 for success, error code from of_fpga_bridge_get() othewise.
> */
> @@ -258,7 +258,7 @@ EXPORT_SYMBOL_GPL(of_fpga_bridge_get_to_list);
> * @info: fpga image specific information
> * @bridge_list: list of FPGA bridges
> *
> - * Get an exclusive reference to the bridge and and it to the list.
> + * Get an exclusive reference to the bridge and it to the list.

Looks good.

Reviewed-by: Tom Rix <[email protected]>

Moritz,

This patch and my spelling cleanup are low risk, can we get these into
fpga-next ?

https://lore.kernel.org/linux-fpga/[email protected]/

Tom

> *
> * Return 0 for success, error code from fpga_bridge_get() othewise.
> */

2021-05-28 22:16:15

by Moritz Fischer

[permalink] [raw]
Subject: Re: [PATCH] fpga: fpga-bridge: removed repeated word

On Fri, May 28, 2021 at 08:35:57PM +0530, Navin Sankar Velliangiri wrote:
> Removed repeated word and.
> Reported by checkpatch.
>
> Signed-off-by: Navin Sankar Velliangiri <[email protected]>
> ---
> drivers/fpga/fpga-bridge.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/fpga/fpga-bridge.c b/drivers/fpga/fpga-bridge.c
> index e9266b2a357f..9ada1126faf7 100644
> --- a/drivers/fpga/fpga-bridge.c
> +++ b/drivers/fpga/fpga-bridge.c
> @@ -228,7 +228,7 @@ EXPORT_SYMBOL_GPL(fpga_bridges_put);
> * @info: fpga image specific information
> * @bridge_list: list of FPGA bridges
> *
> - * Get an exclusive reference to the bridge and and it to the list.
> + * Get an exclusive reference to the bridge and it to the list.
> *
> * Return 0 for success, error code from of_fpga_bridge_get() othewise.
> */
> @@ -258,7 +258,7 @@ EXPORT_SYMBOL_GPL(of_fpga_bridge_get_to_list);
> * @info: fpga image specific information
> * @bridge_list: list of FPGA bridges
> *
> - * Get an exclusive reference to the bridge and and it to the list.
> + * Get an exclusive reference to the bridge and it to the list.
> *
> * Return 0 for success, error code from fpga_bridge_get() othewise.
> */
> --
> 2.31.1
>

Didn't apply on top of Tom's change, so I fixed it up. Queued.

- Moritz