Re: [PATCH 2/2] hwmon: (adm1031) allow setting update rate

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



...
> @@ -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

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux