> That isn't the point. A 2-second hot loop is just as bad. > There should be a usleep_range() or similar between loop iterations.I'll replace schedule() with usleep_range() Alvaro
> Even better would be a separate CONFIG_SNET_VDPA_HWMON Kconfig option.I prefer to wrap everything with a single Kconfig option.> depends on HWMON || HWMON=nAre you referring here to CONFIG_SNET_VDPA, or to the CONFIG_SNET_VDPA_HWMON you suggested? Because if this refers to CONFIG_SNET_VDPA, HWMON=m will block the driver. Alvaro