From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tariq Toukan Subject: Re: [PATCH] net/mlx4: fix spelling mistake: "Inrerface" -> "Interface" Date: Tue, 22 May 2018 18:25:35 +0300 Message-ID: <6e1c71f4-f2f5-7a84-7d86-13e660073f44@mellanox.com> References: <20180522083728.5874-1-colin.king@canonical.com> <77704a43-2b78-e88b-04b9-9a2623e8b5fb@mellanox.com> <297abbee-1f5c-6159-e559-44d83ddfb7ff@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org To: Colin Ian King , Tariq Toukan , "David S . Miller" , netdev@vger.kernel.org, linux-rdma@vger.kernel.org Return-path: In-Reply-To: <297abbee-1f5c-6159-e559-44d83ddfb7ff@canonical.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On 22/05/2018 6:23 PM, Colin Ian King wrote: > On 22/05/18 16:21, Tariq Toukan wrote: >> >> >> On 22/05/2018 11:37 AM, Colin King wrote: >>> From: Colin Ian King >>> >>> Trivial fix to spelling mistake in mlx4_dbg debug message >>> >>> Signed-off-by: Colin Ian King >>> --- >>>   drivers/net/ethernet/mellanox/mlx4/intf.c | 2 +- >>>   1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/net/ethernet/mellanox/mlx4/intf.c >>> b/drivers/net/ethernet/mellanox/mlx4/intf.c >>> index 2edcce98ab2d..6bd4103265d2 100644 >>> --- a/drivers/net/ethernet/mellanox/mlx4/intf.c >>> +++ b/drivers/net/ethernet/mellanox/mlx4/intf.c >>> @@ -172,7 +172,7 @@ int mlx4_do_bond(struct mlx4_dev *dev, bool enable) >>>           list_add_tail(&dev_ctx->list, &priv->ctx_list); >>>           spin_unlock_irqrestore(&priv->ctx_lock, flags); >>>   -        mlx4_dbg(dev, "Inrerface for protocol %d restarted with >>> when bonded mode is %s\n", >>> +        mlx4_dbg(dev, "Interface for protocol %d restarted with when >>> bonded mode is %s\n", >> >> Thanks Colin. >> I think there's one more thing to fix here. >> It is redundant to say "with when", it was probably done by mistake. >> Let's rephrase, maybe this way? >> >> restarted with bonded mode %s > > Sounds like a good idea, do you want me to send V2 of the patch with > this fix? > Yes please. >> >>>                dev_ctx->intf->protocol, enable ? >>>                "enabled" : "disabled"); >>>       } >>> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe >> kernel-janitors" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at  http://vger.kernel.org/majordomo-info.html >