On 10/10/11 02:29, Linus Walleij wrote: > 2011/10/9 Zoltan Devai <zoss@xxxxxxxxx>: > >> Signed-off-by: Zoltan Devai <zoss@xxxxxxxxx> >> --- >> arch/arm/mach-spmp8000/adc.c | 465 +++++++++++++++++++++ >> arch/arm/mach-spmp8000/include/mach/spmp8000adc.h | 29 ++ > > I think we stopped stuffing misc drivers under arch/arm/* > > And stuffing them under drivers/misc/* won't be popular either. > > IIO has some ADCs in drivers/staging/iio/adc > these seem all to be intended to be used from userspace. Yeah, we've had various discussions about adding support for in kernel users. Its always fallen on the fact that no one has had the time to write the code to allow it all to be linked up. Right now it's a case of drivers providing their own interfaces if they want to allow this. Mark Brown has raised this issue before so I've cc'd him. For reference of those not following the original thread (based on a quick scan of the driver code in question). Driver has two callback sets: 1) Touch panel reads (I guess going straight to an input driver) (1 of these only). This is annoyingly tied up with the more general adc registers. 2) General purpose adc callbacks. Both are driven off an interrupt, but data ready appears to manually driven (so software triggered sampling then data ready signal when sampling is done for a single channel). So what are these callbacks used for? Note they all run as interrupt top halves... Options at current time are: mfd supporting input device and either hwmon or iio device. A driver that hosts both a hwmon / iio and an input device. Be brave (+ have a lot of time) and propose patches adding the relevant tying together code to allow iio drivers to be queried in kernel. It shouldn't be that bad actually if you solve the how to work out which device / channel you want in a consistent way. It really depends on what those general adc callbacks get used for. > > There are a few in-kernel ADCs all over the kernel tree > for various specific purposes. > > I don't know where we should go with this kind of stuff, > really :-/ > > Yours, > Linus Walleij > -- > 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 > -- 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
