dann frazier wrote:
On Wed, Dec 06, 2006 at 09:05:50PM -0500, Ralph Siemsen wrote:
dann frazier wrote:

The manpage for set_therm says:
"The program requires a char device node /dev/temperature with major=10, minor=131 to interface with the ds1620 device driver"

However, on my freshly-installed-with-etch-rc1 system, I have a /dev/temp, not a /dev/temperature. /dev/temp has the right major/minor, so it seems like /dev/temp should be used instead.
At one point in time, there was objection to using the standard /dev
entry because the netwinder's output format when you read from this
device does not match the standard (celsius vs farenheit, if memory
serves).  This may be a moot point, as /proc/therm gives it in
human-readable form anyways.

I really don't know enough about it to have an opinion on /proc
interface vs. /dev interface. I'm just going on what I read in the man
page which details using /dev/temperature and not /dev/temp - since
they have the same major/minor, their behavior should be identical.

I don't really think the LSB comes into play here as this isn't a
standardized interface. However, I think a reasonable method is to
attempt to read/use /dev/temp and if that fails try /dev/temperature.

I would rather think that the order should be reversed... First /dev/temperature, and if that fails - /dev/temp.
That is the way it is implemented already in fan_ctrl.c and set_therm.c
It is currently missing in fand.c

However, much better would be to create a symlink from /dev/temp to /dev/temperature.

Reason: set_therm and fan_ctrl programs use /dev/temperature for setting the thermostat configuration. fand is taking advantage of 0.5 degree temperature resolution - using propriatory ioctl call. All of them need a non-standard /dev/temp interface. As such it should not be attempted if the correct Netwinder driver is not installed. Hence since it is hardware-specific, it may be better to keep it as a specific /dev/ interface... (rename to nwtemp????)

So the quick fix will be to add /dev/temp to the man page. The proper - to add /dev/temperature symlink to udev rules...

Woody



--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to