> +static int mv88q2xxx_hwmon_probe(struct phy_device *phydev) > +{ > + struct device *dev = &phydev->mdio.dev; > + struct device *hwmon; > + char *hwmon_name; > + int ret; > + > + /* Enable temperature sensor interrupt */ > + ret = phy_set_bits_mmd(phydev, MDIO_MMD_PCS, > + MDIO_MMD_PCS_MV_TEMP_SENSOR1, > + MDIO_MMD_PCS_MV_TEMP_SENSOR1_INT_EN); You enable an interrupt, but i don't see any changes to the interrupt handler to handle any interrupts which are generated? Andrew