On 05/22/2013 10:27 PM, Jonathan Cameron wrote: > 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. Actually no I'm not. I'd completely forgotten that I wasn't happy with some of the patches during review. Over to Denis to post a new version or convince me the review comments were wrong ;) > > 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 > -- 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
