Skip to content

Conversation

@HouseOfBeck
Copy link

I've found an error in read_value due to a collision in the AIN* file. The result was bad data returned
to the user. I added a simple change to detect the error and retry the read up to 3 times.

@zthorson
Copy link
Contributor

The retry on ADC error commit seems to be a good candidate to merge. I have hit the same issue, though it only comes up when trying to alternate between reading from two different ADC ports.

It seems to be a low level driver bug in the ADC channel switching on the 3.8 kernel.

Reproduction of the issue and a better description is found here:
https://groups.google.com/forum/#!msg/beagleboard/99kDAfTs_ds/zVO1UKX1WhoJ

I have not found any kernel level patches or fixes that have been release, so at the moment, a retry function seems to be the best workaround.

@jwcooper
Copy link
Member

I've merged a similar pull request. Thank you for submitting code to the library though!

@jwcooper jwcooper closed this Nov 25, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants