Re: [PATCH v2] fpga: fix typo in s10_ops_write() comment

From: Xu Yilun

Date: Wed Sep 09 2026 - 02:59:42 EST


On Mon, Sep 07, 2026 at 10:13:44AM +0530, Hemanth Selam wrote:
> The comment reads "reuse it to send more data or free if if all data has
> been sent". The first "if" is not a repetition, it should be "it", to
> match the "reuse it" earlier in the same sentence.
>
> v1 deleted the second "if" instead, which left the sentence without an
> object. Xu Yilun pointed out the correct reading.

These change history should not belong to the changelog. Put them under
--- so they won't appear when applied.

>
> Only touches a comment, no code changes.
>
> Assisted-by: Cursor:claude-opus-5
> Signed-off-by: Hemanth Selam <hemanth.selam@xxxxxxxxx>
> ---
> drivers/fpga/stratix10-soc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/fpga/stratix10-soc.c b/drivers/fpga/stratix10-soc.c
> index b8ec2e6f615f..5aed560050c2 100644
> --- a/drivers/fpga/stratix10-soc.c
> +++ b/drivers/fpga/stratix10-soc.c
> @@ -286,7 +286,7 @@ static int s10_ops_write(struct fpga_manager *mgr, const char *buf,
>
> /*
> * Loop waiting for buffers to be returned. When a buffer is returned,
> - * reuse it to send more data or free if if all data has been sent.
> + * reuse it to send more data or free it if all data has been sent.
> */
> while (count > 0 || s10_free_buffer_count(mgr) != NUM_SVC_BUFS) {
> reinit_completion(&priv->status_return_completion);
>