Re: [PATCH 0/3] hwmon: More LM63 enhancements

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

 



Hi Guenter,

On Tue, 22 Nov 2011 12:39:57 -0800, Guenter Roeck wrote:
> Add support for unsigned temperatures on LM96163
> Add support for writing remote T_CRIT limit
> Add support for writing conversion rate register
> 
> Question:
>   Calculations are currently along the line of "/ 125 * 32", causing loss of accuracy.
>   Ok to change to "* 32 / 125" ?

I think this is on purpose. Multiplying last guarantees that we don't
try to set bits which do not exist to 1. As long as proper rounding is
done before dividing, I don't quite see how we could be losing
accuracy. Do you have an actual example?

-- 
Jean Delvare

_______________________________________________
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