Subject: Re: [RESEND net-next 2/4] net: bonding: remove repeated word

From: Jay Vosburgh <[email-masked]>
Date: Tue, 30 Mar 2021 09:05:59 -0700

Huazhong Tan <[email-masked]> wrote:

>From: Peng Li <[email-masked]>
>
>Remove repeated word "that".
>
>Signed-off-by: Peng Li <[email-masked]>
>Signed-off-by: Huazhong Tan <[email-masked]>
>---
> drivers/net/bonding/bond_alb.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/net/bonding/bond_alb.c b/drivers/net/bonding/bond_alb.c
>index c3091e0..3455f2c 100644
>--- a/drivers/net/bonding/bond_alb.c
>+++ b/drivers/net/bonding/bond_alb.c
>@@ -1098,7 +1098,7 @@ static void alb_fasten_mac_swap(struct bonding *bond, struct slave *slave1,
>  * If @slave's permanent hw address is different both from its current
>  * address and from @bond's address, then somewhere in the bond there's
>  * a slave that has @slave's permanet address as its current address.
>- * We'll make sure that that slave no longer uses @slave's permanent address.
>+ * We'll make sure that slave no longer uses @slave's permanent address.

	This is actually correct as written, but I can see that it's a
bit confusing.  Rather than removing the second that, I'd suggest
changing it to "... make sure the other slave no longer uses ..." to be
clearer.

	-J

---
	-Jay Vosburgh, [email-masked]