search

sparkfun / Qwiic_Twist

A digital RGB encoder controllable and readable over I2C.
https://www.sparkfun.com
Other
3 stars 2 forks source link

Change INT to a state machine #2

Closed nseidle closed 5 years ago

nseidle commented 5 years ago

The interrupt pin toggles in bad ways.

When a user reads any bytes from Twist it causes the INT pin to go to high impedance, indicating the clearing of interrupts. However, back in the main loop the interrupt bits are still set so the INT pin is then set back to low. This looks like a toggle INT pin to the user.

Better to go to a state machine such as the one used on Qwiic MP3 Trigger.

nseidle commented 5 years ago

Fixed in v1.2 of firmware now pushed to production.