On Wed, Apr 1, 2015 at 5:39 PM, Lars-Peter Clausen <lars@xxxxxxxxxx> wrote: > On 04/01/2015 04:04 PM, Daniel Baluta wrote: > [...] >> >> >>> +static const struct iio_chan_spec_ext_info ltr501_ext_info[] = { >>> + { >>> + .name = "intr_persist", >>> + .read = ltr501_read_intr_prst, >>> + .write = ltr501_write_intr_prst, >>> + .shared = IIO_SHARED_BY_TYPE, >>> + }, >>> + {}, >>> +}; >>> + >> >> Would be nice to standardize persistence attribute >> (IIO_CHAN_INFO_PERSISTENCE). > > > If I understand the behavior correctly it causes that the event needs to be > triggered at least n times before the event is reported by the chip. In my > opinion 'persistence' is not a good term for that. I'm not sure what a > better term is but I think it should go more in the direction of ratelimit > or something. I've seen this term used for many devices: * TSL25911 ambient light sensor [1] [ One set of thresholds can be configured to trigger an interrupt only when the ambient light exceeds them for a configurable amount of time (persistence) ] * TAOS TCS34725 ambient light sensor [2] [ The interrupt persistence filter allows the user to define the number of consecutive out-of-threshold events necessary before generating an interrupt. ] * Avago SAPDS-9950, Sensortek STK3310 I think the TSL25911 datasheet best describes this parameter, as the amount of time that ambient light should exceed a threshold until an interrupt is generated. thanks, Daniel. [1] http://www.adafruit.com/datasheets/TSL25911_Datasheet_EN_v1.pdf [2] http://www.adafruit.com/datasheets/TCS34725.pdf -- 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