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;198757673;13503038;p?http://java.sun.com/javaone _______________________________________________ Freevo-devel mailing list Freevo-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freevo-devel
