Hi,
sorry for the delay.
On 29.05.11 17:10, Henrik Brix Andersen wrote:
Hi,
On May 29, 2011, at 16:25, Andreas Tobler wrote:
Author: andreast
Date: Sun May 29 14:25:42 2011
New Revision: 222449
URL: http://svn.freebsd.org/changeset/base/222449
Log:
Add a new driver, the ad7417, to read temperatures and voltages on some
PowerMac's.
Approved by: nwhitehorn (mentor)
Added:
head/sys/dev/iicbus/ad7417.c (contents, props changed)
Modified:
head/sys/conf/files.powerpc
head/sys/powerpc/conf/GENERIC
head/sys/powerpc/conf/GENERIC64
head/sys/powerpc/conf/NOTES
Since this driver is powerpc-only (the chip is not powerpc-specific - but this
driver depends on OFW?) shouldn't it live somewhere under sys/powerpc/ instead
of the generic sys/dev/iicbus/?
Or perhaps be rewritten to a more generic form?
The chip itself is not PowerMac specific. It contains four times the
functionality of the ad7418 which is already in there. (dev/iicbus)
The ofw is only needed to detect the chip and to wire the sensor place
to the right fan property. I think this could be refactored that it is
PowerMac only and others could benefit from the driver too.
The same would apply for the ds1775.c and the max6690.c. Those chips are
not PowerMac only, they are simple i2c temp sensors.
To get there, it would really be helpful if there are other people with
non PowerMac hardware who would have an interest in these drivers.
Then we could sit together, refactor and test. W/o the corresponding
hardware it seems a bit difficult for me to test.
A few months ago I asked on hackers@ how I could combine those two
drivers. I got no feedback.
Now we're heading towards 9.0 and we (nwhitehorn@) wanted to get this
driver in. We need these drivers to run certain G5 PowerMac's in a
silent mode.
W/o we have a swarm of bees which makes it very hard to concentrate ;)
Gruss,
Andreas
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
