From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 25 Oct 2010 04:19:43 -0700 From: Guenter Roeck To: Jonathan Cameron CC: Mike Frysinger , "linux-iio@vger.kernel.org" , "device-drivers-devel@blackfin.uclinux.org" , Sonic Zhang Subject: Re: [PATCH 12/14] staging: iio: adc: new driver for ADT7408 temperature sensors Message-ID: <20101025111943.GA16863@ericsson.com> References: <1287865757-1031-1-git-send-email-vapier@gentoo.org> <1287865757-1031-12-git-send-email-vapier@gentoo.org> <4CC4B8F8.7010005@cam.ac.uk> <20101025004650.GA15907@ericsson.com> <4CC55CD2.1020501@cam.ac.uk> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" In-Reply-To: <4CC55CD2.1020501@cam.ac.uk> List-ID: 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