On 12/23/18 11:58 AM, Andrew Lunn wrote: >> I can also use a simple conditional, since I don't expect the number of >> HWMON properties to grow, eg. >> >> if (type == hwmon_in && attr == hwmon_in_lcrit_alarm) {...} >> if (type == hwmon_temp && attr == hwmon_temp_crit_alarm) {...} > > Yes, that is fine. > > Please make sure you Cc: the HWMON maintainer on this patch.
Fine -- Best regards, Marek Vasut