Re: [PATCH 3/3] iio:pressure: Add STMicroelectronics pressures driver

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

 



On 05/20/2013 10:47 AM, Jacek Anaszewski wrote:
> On 05/20/2013 09:51 AM, Denis CIOCCA wrote:
>> Hi Jonathan,
>>
>>> Now going forward I'd of course like to see everyone happy with the
>>> resulting driver.  Denis do you have plans to implement the threshold
>>> events that were in the other driver proposal?
>> Now I'm very full but in the future I have plans to support the threshold
>> events, I'm very happy if Jacek will help me! ;)
> 
> Hi Jonathan, Denis,
> 
> The threshold events weren't supported in my driver proposal - the events were related only to the 'data ready'
> interrupts. I've been
> playing with threshold interrupts during development, but I didn't
> find the device reliable by minimal threshold levels.
Oops, I missed that you were 'abusing' these entirely in review.

> I've lately
> devised a procedure of testing greater pressure differences and
> I will test it within few weeks, as currently I am working on
> another driver.
Given the driver simplicity (as it's adding to the existing drivers that
are well tested) I'm going to assume it works fine for now
and merge it based on review.  Do shout if it doesn't of course
or if you have any improvements to suggest.

I'd definitely like to see full event support for the threshold interrupts
in these parts (which will be needed to finally kill off the my lis3l02dq
driver :)  Been trying to get rid of that one for years.).

Jonathan
> 
> Thanks,
> Jacek
> 
> -- 
> 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
--
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