Re: Anyone working on iio/cdc?

[Jonathan Cameron <jic23@kernel.org>]

On 29/12/15 19:07, Shraddha Barke wrote:
> Hello all,
> I'm helping Greg do cleanup of drivers in the staging directory as part
> of Outreachy intern program.
> I've sent out checkpatch patches for iio/cdc, meanwhile could you suggest
> what other tasks are left to be done so that iio/cdc can be moved out of
> staging?
> 
> Kind Regards,
> 
> Shraddha
I'll leave this one mostly to Lars to answer.

A quick glance at one of the drivers ad7150 - suggests the main issue
is one of ABI that we don't currently have and of conforming to what what
we do have!

The events on these are 'interesting' as they are adaptive in that we
are looking at a threshold on the absolute  value (rather than a ROC)
but the threshold is adapting based on more or less a moving average
(with the timeout quirk)

This type of event needs to be fully described in some fashion...
We do have thresh_adaptive which is used in this driver, but it's not
properly documented so that would be a good start!
The trick is then to define the ABI to describe how the threshold is
adapting and the controls on it.

I don't think the current ABI for this description is specific or detailed enough..
Timeout is the 'unusual' control parameter.  Hmm. Naming is tricky on this
one.

Perhaps _thresh_tracking_timeout so we'd end up with something like.
in_capacitance0_threshold_adaptive_thresh_tracking_timeout as a value in seconds.

Jonathan