[lm-sensors] [PATCH 2/2] hwmon: (adm1031) allow setting update rate

Jonathan Cameron jic23 at cam.ac.uk
Sun May 23 12:28:29 CEST 2010


...
> @@ -75,6 +79,7 @@ struct adm1031_data {
>  	int chip_type;
>  	char valid;		/* !=0 if following fields are valid */
>  	unsigned long last_updated;	/* In jiffies */
> +	unsigned int update_rate;	/* In milliseconds */
Hi,

Just a quick comment.  Aren't the unit of a rate typically things per time period?
(i.e. same as frequency)  If you want to go with milli seconds, then surely
it should be called update_period.

Jonathan



More information about the lm-sensors mailing list