diff mbox series

net: ethernet/sfc: Fix comment typo

Message ID 20220715045914.23629-1-wangborong@cdjrlc.com (mailing list archive)
State Rejected
Delegated to: Netdev Maintainers
Headers show
Series net: ethernet/sfc: Fix comment typo | expand

Commit Message

Jason Wang July 15, 2022, 4:59 a.m. UTC
The double `that' is duplicated in line 2438, remove one.

Signed-off-by: Jason Wang <wangborong@cdjrlc.com>
---
 drivers/net/ethernet/sfc/falcon/falcon.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Edward Cree July 18, 2022, 2:19 p.m. UTC | #1
On 15/07/2022 05:59, Jason Wang wrote:
> The double `that' is duplicated in line 2438, remove one.
> 
> Signed-off-by: Jason Wang <wangborong@cdjrlc.com>
> ---
>  drivers/net/ethernet/sfc/falcon/falcon.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/sfc/falcon/falcon.c b/drivers/net/ethernet/sfc/falcon/falcon.c
> index 3324a6219a09..9fcd28500939 100644
> --- a/drivers/net/ethernet/sfc/falcon/falcon.c
> +++ b/drivers/net/ethernet/sfc/falcon/falcon.c
> @@ -2435,7 +2435,7 @@ static void falcon_init_rx_cfg(struct ef4_nic *efx)
>  		 * supports scattering for user-mode queues, but will
>  		 * split DMA writes at intervals of RX_USR_BUF_SIZE
>  		 * (32-byte units) even for kernel-mode queues.  We
> -		 * set it to be so large that that never happens.
> +		 * set it to be so large that never happens.
>  		 */
>  		EF4_SET_OWORD_FIELD(reg, FRF_AA_RX_DESC_PUSH_EN, 0);
>  		EF4_SET_OWORD_FIELD(reg, FRF_AA_RX_USR_BUF_SIZE,
> 

Nack.  The first 'that' is a subordinating conjunction, the second is
 a demonstrative pronoun, so this is not a mere duplication.
(Admittedly, some writers would omit the conjunction, but I think the
 sentence reads more clearly with it present.)
One possible alternative would be to change the second 'that' to
 'this' or 'the above', or even 'such splitting', if you really find
 the original confusing.

-ed
diff mbox series

Patch

diff --git a/drivers/net/ethernet/sfc/falcon/falcon.c b/drivers/net/ethernet/sfc/falcon/falcon.c
index 3324a6219a09..9fcd28500939 100644
--- a/drivers/net/ethernet/sfc/falcon/falcon.c
+++ b/drivers/net/ethernet/sfc/falcon/falcon.c
@@ -2435,7 +2435,7 @@  static void falcon_init_rx_cfg(struct ef4_nic *efx)
 		 * supports scattering for user-mode queues, but will
 		 * split DMA writes at intervals of RX_USR_BUF_SIZE
 		 * (32-byte units) even for kernel-mode queues.  We
-		 * set it to be so large that that never happens.
+		 * set it to be so large that never happens.
 		 */
 		EF4_SET_OWORD_FIELD(reg, FRF_AA_RX_DESC_PUSH_EN, 0);
 		EF4_SET_OWORD_FIELD(reg, FRF_AA_RX_USR_BUF_SIZE,