Re: lm sensors sysfs file structure

Pavel Machek (pavel@ucw.cz)
Tue, 1 Apr 2003 22:22:17 +0200


Hi!

> > > > > Floating point values XXX.X or XXX.XX in degrees Celcius.
> > > >
> > > > If we're restructuring it, I think we should also agree on _one_ common
> > > > denominator for all values ie. mVolt and milli-Degree Celsius, so that
> > > > no userspace program ever again has know how to convert them to
> > > > user-readable values and every user can just cat the values and doesn't
> > > > have to wonder if it's centi-Volt, milli-Volt, centi-Degree, dezi-Degree
> > > > or whatever.
> > >
> > > Um, that's what my proposal stated. Do you not agree with it? (You're
> > > quoting the existing document above, not my proposed changes.)
> >
> > Well, you had cV for PSU voltages and
> > mV for cpu core voltage. I guess mV
> > and mili-deg-C everywhere would be
> > nicer.
>
> I was trying to keep consistant with what the old /proc values were
> reported as. I'll go fix that up.
>
> As for why no floating point, it's a pain in the but to both output a
> fixed point number from the kernel into floating point, and to parse a
> floating point number from userspace within the kernel, turning it into
> a fixed point number. With the proposal I wrote up, none of that is
> needed, and all userspace has to do is divide by a factor of 10 to get
> the proper value.

Okay, I did not realize it is input, too. Outputing reals is really
easy; but parsing them. Okay, I guess I'm convinced. You should still
use mili-something, not centi-something...
Pavel

-- 
When do you have a heart between your knees?
[Johanka's followup: and *two* hearts?]
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/