On Tue, Aug 30, 2011 at 3:05 AM, Jonathan Cameron <jic23@xxxxxxxxx> wrote: .... >> I did change one basic behavior that I think was also broken: cache >> the value regardless of if the transaction completed successfully or >> not. > Don't do that. That means userspace will get an invalid value if it reads > in the meantime. If you have an error on a hardware bus - tell userspace about > it and don't 'guess' what is in the register. Ah ok - I didn't know this was part of the path to/from user space. > > Otherwise patch looks fine to me. OK - I'll restore previous behavior and submit another patch. cheers! grant -- 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
