All of lore.kernel.org
 help / color / mirror / Atom feed
* Anyone working on iio/cdc?
@ 2015-12-29 19:07 Shraddha Barke
  2016-01-04 12:36 ` Jonathan Cameron
  0 siblings, 1 reply; 2+ messages in thread
From: Shraddha Barke @ 2015-12-29 19:07 UTC (permalink / raw)
  To: Lars-Peter Clausen, Jonathan Cameron, Michael Hennerich; +Cc: linux-iio

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Anyone working on iio/cdc?
  2015-12-29 19:07 Anyone working on iio/cdc? Shraddha Barke
@ 2016-01-04 12:36 ` Jonathan Cameron
  0 siblings, 0 replies; 2+ messages in thread
From: Jonathan Cameron @ 2016-01-04 12:36 UTC (permalink / raw)
  To: Shraddha Barke, Lars-Peter Clausen, Michael Hennerich; +Cc: linux-iio

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


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-01-04 12:36 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-29 19:07 Anyone working on iio/cdc? Shraddha Barke
2016-01-04 12:36 ` Jonathan Cameron

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.