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
