Huazhong Tan <tanhuazh...@huawei.com> wrote:

>From: Peng Li <lipeng...@huawei.com>
>
>Remove repeated word "that".
>
>Signed-off-by: Peng Li <lipeng...@huawei.com>
>Signed-off-by: Huazhong Tan <tanhuazh...@huawei.com>
>---
> 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, jay.vosbu...@canonical.com

Reply via email to