Re: [PATCH v4] nvme/host: fixup some typos

[Wilfred Mallawa]

On Wed, 2026-01-14 at 16:37 +1000, Wilfred Mallawa wrote:

Gentle ping, in-case this got lost in the threads :)

Wilfred

> From: Wilfred Mallawa <wilfred.mallawa@xxxxxxx>
> 
> Fix up some minor typos/grammar in the nvme host tcp driver and a
> comment
> style to conform to the standard kernel style.
> 
> Signed-off-by: Wilfred Mallawa <wilfred.mallawa@xxxxxxx>
> Reviewed-by: Damien Le Moal <dlemoal@xxxxxxxxxx>
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> ---
> V4:
> 	- Minor grammar fixup
> ---
>  drivers/nvme/host/tcp.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/nvme/host/tcp.c b/drivers/nvme/host/tcp.c
> index 69cb04406b47..6658a5806dd9 100644
> --- a/drivers/nvme/host/tcp.c
> +++ b/drivers/nvme/host/tcp.c
> @@ -25,7 +25,8 @@
>  
>  struct nvme_tcp_queue;
>  
> -/* Define the socket priority to use for connections were it is
> desirable
> +/*
> + * Define the socket priority to use for connections where it is
> desirable
>   * that the NIC consider performing optimized packet processing or
> filtering.
>   * A non-zero value being sufficient to indicate general
> consideration of any
>   * possible optimization.  Making it a module param allows for
> alternative
> @@ -926,8 +927,9 @@ static int nvme_tcp_recv_data(struct
> nvme_tcp_queue *queue, struct sk_buff *skb,
>  			req->curr_bio = req->curr_bio->bi_next;
>  
>  			/*
> -			 * If we don`t have any bios it means that
> controller
> -			 * sent more data than we requested, hence
> error
> +			 * If we do not have any bios, it means that
> the
> +			 * controller sent more data than we
> requested,
> +			 * hence error
>  			 */
>  			if (!req->curr_bio) {
>  				dev_err(queue->ctrl->ctrl.device,