All of lore.kernel.org
 help / color / mirror / Atom feed
* [net PATCH] net: sfp: Fix typo in state machine debug string
@ 2021-10-04 21:50 Sean Anderson
  2021-10-05 12:10 ` patchwork-bot+netdevbpf
  0 siblings, 1 reply; 2+ messages in thread
From: Sean Anderson @ 2021-10-04 21:50 UTC (permalink / raw)
  To: Russell King, netdev
  Cc: Heiner Kallweit, Florian Fainelli, Jakub Kicinski,
	David S . Miller, linux-kernel, Andrew Lunn, Sean Anderson

The string should be "tx_disable" to match the state enum.

Fixes: 4005a7cb4f55 ("net: phy: sftp: print debug message with text, not numbers")
Signed-off-by: Sean Anderson <sean.anderson@seco.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
---
This was first submitted as [1], but has been submitted separately, per
request.

[1] https://lore.kernel.org/netdev/YVtypfZJfivfDnu7@lunn.ch/T/#m0b62cbe7804eb8df8dc69944ec3824274ae1b0c1

 drivers/net/phy/sfp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/phy/sfp.c b/drivers/net/phy/sfp.c
index 34e90216bd2c..ab77a9f439ef 100644
--- a/drivers/net/phy/sfp.c
+++ b/drivers/net/phy/sfp.c
@@ -134,7 +134,7 @@ static const char * const sm_state_strings[] = {
 	[SFP_S_LINK_UP] = "link_up",
 	[SFP_S_TX_FAULT] = "tx_fault",
 	[SFP_S_REINIT] = "reinit",
-	[SFP_S_TX_DISABLE] = "rx_disable",
+	[SFP_S_TX_DISABLE] = "tx_disable",
 };
 
 static const char *sm_state_to_str(unsigned short sm_state)
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [net PATCH] net: sfp: Fix typo in state machine debug string
  2021-10-04 21:50 [net PATCH] net: sfp: Fix typo in state machine debug string Sean Anderson
@ 2021-10-05 12:10 ` patchwork-bot+netdevbpf
  0 siblings, 0 replies; 2+ messages in thread
From: patchwork-bot+netdevbpf @ 2021-10-05 12:10 UTC (permalink / raw)
  To: Sean Anderson
  Cc: linux, netdev, hkallweit1, f.fainelli, kuba, davem, linux-kernel, andrew

Hello:

This patch was applied to netdev/net.git (refs/heads/master):

On Mon,  4 Oct 2021 17:50:02 -0400 you wrote:
> The string should be "tx_disable" to match the state enum.
> 
> Fixes: 4005a7cb4f55 ("net: phy: sftp: print debug message with text, not numbers")
> Signed-off-by: Sean Anderson <sean.anderson@seco.com>
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> ---
> This was first submitted as [1], but has been submitted separately, per
> request.
> 
> [...]

Here is the summary with links:
  - [net] net: sfp: Fix typo in state machine debug string
    https://git.kernel.org/netdev/net/c/25a9da6641f1

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-10-05 12:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-04 21:50 [net PATCH] net: sfp: Fix typo in state machine debug string Sean Anderson
2021-10-05 12:10 ` patchwork-bot+netdevbpf

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.