Re: [PATCH resend] sensors: Add support for additional attributes to the sensors command

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

 



On Tue, 15 Mar 2011 15:54:15 -0700, Guenter Roeck wrote:
> On Tue, Mar 15, 2011 at 05:50:46PM -0400, Jean Delvare wrote:
> > Apparently our unit selection algorithm picks the smallest unit if
> > value is 0? Wouldn't it make more sense to pick the base unit instead
> > (W in this case)?
> > 
> I'll add
> 	if (abs_value == 0) {
>                 *prefixstr = "";
>                 return;
>         }
> to the beginning of scale_value(). Seems to be the easiest fix.

Agreed, that's what I had in mind too.

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