Hello!

On 01/11/2017 06:09 PM, Andrew Lunn wrote:

+static int marvell_hwmon_probe(struct phy_device *phydev,
+                              const struct hwmon_chip_info *chip)
+{
+       struct marvell_priv *priv = phydev->priv;
+       struct device *dev = &phydev->mdio.dev;
+       int err;
+
+       err = marvell_hwmon_name(phydev);
+       if (err)
+               return err;
+
+       priv->hwmon_dev = devm_hwmon_device_register_with_info(
+               dev, priv->hwmon_name, phydev, chip, NULL);
+
+       if (IS_ERR(priv->hwmon_dev))
+               return PTR_ERR(priv->hwmon_dev);
+
+       return 0;
+}

0-day has pointed out this can be simplified to just

        return PTR_ERR(priv->hwmon_dev);

   PTR_ERR_OR_ZERO() maybe?

[...]

MBR, Sergei

Reply via email to