* [PATCH] net/mlx4_en: fix spelling mistake: "quiting" -> "quitting" @ 2019-02-17 23:03 Colin King 2019-02-18 9:37 ` Tariq Toukan 2019-02-18 20:08 ` David Miller 0 siblings, 2 replies; 6+ messages in thread From: Colin King @ 2019-02-17 23:03 UTC (permalink / raw) To: Tariq Toukan, David S . Miller, netdev, linux-rdma Cc: kernel-janitors, linux-kernel From: Colin Ian King <colin.king@canonical.com> There is a spelling mistake in a en_err error message. Fix it. Signed-off-by: Colin Ian King <colin.king@canonical.com> --- drivers/net/ethernet/mellanox/mlx4/en_netdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c index 6b88881b8e35..c1438ae52a11 100644 --- a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c +++ b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c @@ -3360,7 +3360,7 @@ int mlx4_en_init_netdev(struct mlx4_en_dev *mdev, int port, dev->addr_len = ETH_ALEN; mlx4_en_u64_to_mac(dev->dev_addr, mdev->dev->caps.def_mac[priv->port]); if (!is_valid_ether_addr(dev->dev_addr)) { - en_err(priv, "Port: %d, invalid mac burned: %pM, quiting\n", + en_err(priv, "Port: %d, invalid mac burned: %pM, quitting\n", priv->port, dev->dev_addr); err = -EINVAL; goto out; -- 2.20.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] net/mlx4_en: fix spelling mistake: "quiting" -> "quitting" 2019-02-17 23:03 [PATCH] net/mlx4_en: fix spelling mistake: "quiting" -> "quitting" Colin King @ 2019-02-18 9:37 ` Tariq Toukan 2019-02-18 10:25 ` Dan Carpenter 2019-02-18 20:08 ` David Miller 1 sibling, 1 reply; 6+ messages in thread From: Tariq Toukan @ 2019-02-18 9:37 UTC (permalink / raw) To: Colin King, David S . Miller, netdev@vger.kernel.org, linux-rdma@vger.kernel.org Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org On 2/18/2019 1:03 AM, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > There is a spelling mistake in a en_err error message. Fix it. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > drivers/net/ethernet/mellanox/mlx4/en_netdev.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c > index 6b88881b8e35..c1438ae52a11 100644 > --- a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c > +++ b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c > @@ -3360,7 +3360,7 @@ int mlx4_en_init_netdev(struct mlx4_en_dev *mdev, int port, > dev->addr_len = ETH_ALEN; > mlx4_en_u64_to_mac(dev->dev_addr, mdev->dev->caps.def_mac[priv->port]); > if (!is_valid_ether_addr(dev->dev_addr)) { > - en_err(priv, "Port: %d, invalid mac burned: %pM, quiting\n", > + en_err(priv, "Port: %d, invalid mac burned: %pM, quitting\n", > priv->port, dev->dev_addr); > err = -EINVAL; > goto out; > Hi Colin, thanks for your patch. Reviewed-by: Tariq Toukan <tariqt@mellanox.com> I would suggest adding a Fixes line, but looking into the history of the typo, it went through many patches that modified this line but preserved the typo. Actually, it dates back to the very first commit that introduces mlx4 driver: Patches history: 2b3ddf27f48c net/mlx4_core: Replace VF zero mac with random mac in mlx4_core ef96f7d46ad8 net/mlx4_en: Handle unassigned VF MAC address correctly 6bbb6d99f3d2 net/mlx4_en: Optimize Rx fast path filter checks 453a60827735 mlx4_en: Giving interface name in debug messages c27a02cd94d6 mlx4_en: Add driver for Mellanox ConnectX 10GbE NIC I'm not sure what the "Fixes:" policy is in these cases. Thanks, Tariq ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] net/mlx4_en: fix spelling mistake: "quiting" -> "quitting" 2019-02-18 9:37 ` Tariq Toukan @ 2019-02-18 10:25 ` Dan Carpenter 2019-02-18 14:54 ` Tariq Toukan 0 siblings, 1 reply; 6+ messages in thread From: Dan Carpenter @ 2019-02-18 10:25 UTC (permalink / raw) To: Tariq Toukan Cc: Colin King, David S . Miller, netdev@vger.kernel.org, linux-rdma@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org On Mon, Feb 18, 2019 at 09:37:22AM +0000, Tariq Toukan wrote: > > > On 2/18/2019 1:03 AM, Colin King wrote: > > From: Colin Ian King <colin.king@canonical.com> > > > > There is a spelling mistake in a en_err error message. Fix it. > > > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > > --- > > drivers/net/ethernet/mellanox/mlx4/en_netdev.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c > > index 6b88881b8e35..c1438ae52a11 100644 > > --- a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c > > +++ b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c > > @@ -3360,7 +3360,7 @@ int mlx4_en_init_netdev(struct mlx4_en_dev *mdev, int port, > > dev->addr_len = ETH_ALEN; > > mlx4_en_u64_to_mac(dev->dev_addr, mdev->dev->caps.def_mac[priv->port]); > > if (!is_valid_ether_addr(dev->dev_addr)) { > > - en_err(priv, "Port: %d, invalid mac burned: %pM, quiting\n", > > + en_err(priv, "Port: %d, invalid mac burned: %pM, quitting\n", > > priv->port, dev->dev_addr); > > err = -EINVAL; > > goto out; > > > > Hi Colin, thanks for your patch. > > Reviewed-by: Tariq Toukan <tariqt@mellanox.com> > > I would suggest adding a Fixes line, but looking into the history of the > typo, it went through many patches that modified this line but preserved > the typo. > Actually, it dates back to the very first commit that introduces mlx4 > driver: > > Patches history: > 2b3ddf27f48c net/mlx4_core: Replace VF zero mac with random mac in mlx4_core > ef96f7d46ad8 net/mlx4_en: Handle unassigned VF MAC address correctly > 6bbb6d99f3d2 net/mlx4_en: Optimize Rx fast path filter checks > 453a60827735 mlx4_en: Giving interface name in debug messages > c27a02cd94d6 mlx4_en: Add driver for Mellanox ConnectX 10GbE NIC > > I'm not sure what the "Fixes:" policy is in these cases. I wouldn't necessarily put a Fixes tag on this, because does fixing the spelling really count as a bugfix? It's borderline whether it's a fix or a cleanup. regards, daan carpenter ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] net/mlx4_en: fix spelling mistake: "quiting" -> "quitting" 2019-02-18 10:25 ` Dan Carpenter @ 2019-02-18 14:54 ` Tariq Toukan 0 siblings, 0 replies; 6+ messages in thread From: Tariq Toukan @ 2019-02-18 14:54 UTC (permalink / raw) To: Dan Carpenter, Tariq Toukan Cc: Colin King, David S . Miller, netdev@vger.kernel.org, linux-rdma@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org On 2/18/2019 12:25 PM, Dan Carpenter wrote: > On Mon, Feb 18, 2019 at 09:37:22AM +0000, Tariq Toukan wrote: >> >> >> On 2/18/2019 1:03 AM, Colin King wrote: >>> From: Colin Ian King <colin.king@canonical.com> >>> >>> There is a spelling mistake in a en_err error message. Fix it. >>> >>> Signed-off-by: Colin Ian King <colin.king@canonical.com> >>> --- >>> drivers/net/ethernet/mellanox/mlx4/en_netdev.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c >>> index 6b88881b8e35..c1438ae52a11 100644 >>> --- a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c >>> +++ b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c >>> @@ -3360,7 +3360,7 @@ int mlx4_en_init_netdev(struct mlx4_en_dev *mdev, int port, >>> dev->addr_len = ETH_ALEN; >>> mlx4_en_u64_to_mac(dev->dev_addr, mdev->dev->caps.def_mac[priv->port]); >>> if (!is_valid_ether_addr(dev->dev_addr)) { >>> - en_err(priv, "Port: %d, invalid mac burned: %pM, quiting\n", >>> + en_err(priv, "Port: %d, invalid mac burned: %pM, quitting\n", >>> priv->port, dev->dev_addr); >>> err = -EINVAL; >>> goto out; >>> >> >> Hi Colin, thanks for your patch. >> >> Reviewed-by: Tariq Toukan <tariqt@mellanox.com> >> >> I would suggest adding a Fixes line, but looking into the history of the >> typo, it went through many patches that modified this line but preserved >> the typo. >> Actually, it dates back to the very first commit that introduces mlx4 >> driver: >> >> Patches history: >> 2b3ddf27f48c net/mlx4_core: Replace VF zero mac with random mac in mlx4_core >> ef96f7d46ad8 net/mlx4_en: Handle unassigned VF MAC address correctly >> 6bbb6d99f3d2 net/mlx4_en: Optimize Rx fast path filter checks >> 453a60827735 mlx4_en: Giving interface name in debug messages >> c27a02cd94d6 mlx4_en: Add driver for Mellanox ConnectX 10GbE NIC >> >> I'm not sure what the "Fixes:" policy is in these cases. > > I wouldn't necessarily put a Fixes tag on this, because does fixing the > spelling really count as a bugfix? It's borderline whether it's a fix > or a cleanup. > > regards, > daan carpenter > Thanks Dan, I'm fine with that. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] net/mlx4_en: fix spelling mistake: "quiting" -> "quitting" 2019-02-17 23:03 [PATCH] net/mlx4_en: fix spelling mistake: "quiting" -> "quitting" Colin King 2019-02-18 9:37 ` Tariq Toukan @ 2019-02-18 20:08 ` David Miller 2019-02-19 8:20 ` Tariq Toukan 1 sibling, 1 reply; 6+ messages in thread From: David Miller @ 2019-02-18 20:08 UTC (permalink / raw) To: colin.king; +Cc: tariqt, netdev, linux-rdma, kernel-janitors, linux-kernel From: Colin King <colin.king@canonical.com> Date: Sun, 17 Feb 2019 23:03:31 +0000 > From: Colin Ian King <colin.king@canonical.com> > > There is a spelling mistake in a en_err error message. Fix it. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> Applied, thanks Colin. And I agree that this doesn't really deserve a Fixes: tag. Fixes: tags should really be for changes that introduce truly functional bugs. And that could even be applied in this case _iff_ the string was essential in some way for userland tools which parse the output or similar. But that is not the case here. Anyways, thanks. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] net/mlx4_en: fix spelling mistake: "quiting" -> "quitting" 2019-02-18 20:08 ` David Miller @ 2019-02-19 8:20 ` Tariq Toukan 0 siblings, 0 replies; 6+ messages in thread From: Tariq Toukan @ 2019-02-19 8:20 UTC (permalink / raw) To: David Miller, colin.king@canonical.com Cc: Tariq Toukan, netdev@vger.kernel.org, linux-rdma@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org On 2/18/2019 10:08 PM, David Miller wrote: > From: Colin King <colin.king@canonical.com> > Date: Sun, 17 Feb 2019 23:03:31 +0000 > >> From: Colin Ian King <colin.king@canonical.com> >> >> There is a spelling mistake in a en_err error message. Fix it. >> >> Signed-off-by: Colin Ian King <colin.king@canonical.com> > > Applied, thanks Colin. > > And I agree that this doesn't really deserve a Fixes: tag. > > Fixes: tags should really be for changes that introduce truly > functional bugs. > > And that could even be applied in this case _iff_ the string > was essential in some way for userland tools which parse the > output or similar. But that is not the case here. > Thanks for the clarification. > Anyways, thanks. > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-02-19 8:20 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-02-17 23:03 [PATCH] net/mlx4_en: fix spelling mistake: "quiting" -> "quitting" Colin King 2019-02-18 9:37 ` Tariq Toukan 2019-02-18 10:25 ` Dan Carpenter 2019-02-18 14:54 ` Tariq Toukan 2019-02-18 20:08 ` David Miller 2019-02-19 8:20 ` Tariq Toukan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).