Re: [PATCH 1/2 v3] iio: st_sensors: support active-low interrupts

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux