On Sun, Nov 22, 2015 at 1:22 PM, Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > On 19/11/15 09:15, Linus Walleij wrote: >> Most ST MEMS Sensors that support interrupts can also handle sending >> an active low interrupt, i.e. going from high to low on data ready >> (or other interrupt) and thus triggering on a falling edge to the >> interrupt controller. >> >> Set up logic to inspect the interrupt line we get for a sensor: if >> it is triggering on rising edge, leave everything alone, but if it >> triggers on falling edges, set up active low, and if unsupported >> configurations appear: warn with errors and reconfigure the interrupt >> to a rising edge, which all interrupt generating sensors support. >> >> Create a local header for st_sensors_core.h to share functions >> between the sensor core and the trigger setup code. >> >> Cc: Giuseppe Barba <giuseppe.barba@xxxxxx> >> Cc: Denis Ciocca <denis.ciocca@xxxxxx> >> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > > Looks good to me. Ideally I'd like ack from Denis. > (it's early in the cycle so plenty of time for this one!) Has this fallen off the table? I think this patch 1/2 is fine to merge as-is, I guess it is too late for v4.5 now, but please queue it for the next cycle. Yours, Linus Walleij -- 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