Torsten Kurbad wrote:
> On Friday, April 25, 2008 at 22:22 Duncan Webb wrote:
>
>>>> What about using the path:
/sys/class/hwmon/hwmon0/device/
>>>>
>>> that looks much too "dynamic" in my
eyes. What, if, e.g., the coretemp
>>> module accidentely becomes "hwmon0"?
Also, the plugin as it is, assumes
>>> the tempX_Y files in a _subdirectory_ of
PLATFORM_PATH, therefore we'd
>>> have to provide
"/sys/class/hwmon/hwmon0", and the
"/device" subdir gets
>>> merely chosen by accident.
>>>
>> IIRC, it's described in lm_sensors docs that
/sys/class/hwmon/hwmon0 is
>> the correct place.
>>
>
> in either case, the patch I supplied would ensure that
the correcz
> subdir is returned (here: "device"). ;o)
>
Sorry I didn't mean that you patch was in any way invalid.
I was just trying to say where the correct path now is. As
of 2.6.24
there does not seem to be any sensors in the
/sys/devices/platform/ path.
I think that the module is now invalid as it is possible to
have more
than one path with sensors. On my AMD64 there are two sensor
paths one
for the sensors and one for the processor. I suspect that it
would be
best to allow each sensor's path to be specified.
Duncan
------------------------------------------------------------
-------------
This SF.net email is sponsored by the 2008 JavaOne(SM)
Conference
Don't miss this year's exciting event. There's still time to
save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;1987
57673;13503038;p?http://java.sun.com/javaone
_______________________________________________
Freevo-devel mailing list
Freevo-devel lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-dev
el
|
