Re: [PATCH 12/14] staging: iio: adc: new driver for ADT7408 temperature sensors

[Guenter Roeck <guenter.roeck@ericsson.com>]

On Mon, Oct 25, 2010 at 06:32:50AM -0400, Jonathan Cameron wrote:
> > 
> > I'd love to see some reasoning why hardware monitoring drivers are
> > moved to or directly written in iio.
> > 
> > Also, I seem to be missing your point re "high performance devices".
> > Are you saying that hwmon is not suitable for high performance
> > hardware monitoring devices ?
> Yes.  Point me at someone doing 1MSps or higher via pretty printing through
> a sysfs interface.  Admittedly none of the controversial drivers in this

Ok.

> set do that currently either, but that's why I have asked Analog to confirm
> what they are using them for.  The point is that these devices are only
> hardware monitoring to you because that is what you think they are for.
> Some of them (not the one I forwarded initially) are general purpose ADC's
> that have a temp sensor because the temperature can effect the calibration
> of the outputs.
> 
I did not refer to the chips with generic ADC sensors.
The chips I referred to are AD7414/15, ADT75, ADT7310, ADT7408, and ADT7410,
though I may have missed some.

> We went through this in a lot of depth back when IIO first came about.
> There is a boundary. We just need to pin down where it is.

For the ambient temperature sensors on the other chips - did you consider
adding hwmon device entries for those ? There may of course be reasons against
doing that, but it may be an option. There are other drivers outside the hwmon 
directory which call hwmon_device_register(), so it is not a new concept.

Thanks,
Guenter