Re: [PATCH v3] staging:iio: trigger: Add hrtimer trigger

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

 



On 4/18/2012 12:58 PM, Getz, Robin wrote:
On Mon 16 Apr 2012 12:55, Lars-Peter Clausen pondered:
On 04/16/2012 06:17 PM, Jonathan Cameron wrote:
Lars-Peter Clausen<lars@xxxxxxxxxx>  wrote:
From: Marten Svanfeldt<marten@xxxxxxxxxxxxxxxxxxx>

This patch adds a IIO trigger driver which uses a highres timer to
provide a
frequency based trigger.
Fine as it stands but same issue arises as we had with userspace trigger
still.  What are we doing registering a pure software element not
associated to any specific hardware via a platform device.  Why not do it
on userspace asking for one as we do with the sysfs file based trigger?
I suppose this is a general question how we want to mange our triggers in
general. None of the other existing trigger drivers does direct IO access
and just use existing infrastructure. They could all be easily be
instantiated by writing a string or number to a sysfs file. So where do we
draw the line?
Fair point. Personally I'd go for whether it is about explicit hardware or not. So gpio / general interrupt makes sense in platform code. I suspect we'll kill off the RTC one anyway on the way
out of staging.
Isn't there an issue of accuracy? the timing accuracy of sysfs/userspace is
non-existant with respect to what you need to do in most of these cases.
I'm not suggesting there is any problem with having a hrtimer based trigger (in fact I am throughly in favour!) its just a question of whether it should be registered in the board file / device tree or done via a magic string write as Lars-Peter mentions above (which is what we do the sysfs file based trigger precisely because there was a pretty
strong feeling against implying non existent hardware...)

--
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