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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux