net: sfp: fix hwmon

The referenced commit below allowed more than one hwmon device to be
created per SFP, which is definitely not what we want. Avoid this by
only creating the hwmon device just as we transition to WAITDEV state.

Fixes: 139d3a212a1f ("net: sfp: allow modules with slow diagnostics to probe")
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/phy/sfp.c b/drivers/net/phy/sfp.c
index c118d9f0195b9a4c635d43fed4cbdacf042379c3..c0b9a8e4e65a9ba5f4d6f90a036efbb15d8aa21b 100644 (file)
--- a/drivers/net/phy/sfp.c
+++ b/drivers/net/phy/sfp.c
@@ -1754,6 +1754,10 @@ static void sfp_sm_module(struct sfp *sfp, unsigned int event)
                        break;
                }
 
+               err = sfp_hwmon_insert(sfp);
+               if (err)
+                       dev_warn(sfp->dev, "hwmon probe failed: %d\n", err);
+
                sfp_sm_mod_next(sfp, SFP_MOD_WAITDEV, 0);
                /* fall through */
        case SFP_MOD_WAITDEV:
@@ -1803,15 +1807,6 @@ static void sfp_sm_module(struct sfp *sfp, unsigned int event)
        case SFP_MOD_ERROR:
                break;
        }
-
-#if IS_ENABLED(CONFIG_HWMON)
-       if (sfp->sm_mod_state >= SFP_MOD_WAITDEV &&
-           IS_ERR_OR_NULL(sfp->hwmon_dev)) {
-               err = sfp_hwmon_insert(sfp);
-               if (err)
-                       dev_warn(sfp->dev, "hwmon probe failed: %d\n", err);
-       }
-#endif
 }
 
 static void sfp_sm_main(struct sfp *sfp, unsigned int event)