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