> 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=n Are 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