search

gillham / logic_analyzer

Implementation of a SUMP compatible logic analyzer for the Arduino
Other
463 stars 99 forks source link

restrict to 4 channels, get 2x buffer #32

Open dhiltonp opened 8 years ago

dhiltonp commented 8 years ago

What do you think about restricting input to 4 channels? It would then be easy to double-up on memory, giving 2x the buffer space.

markkimsal commented 8 years ago

:+1: I don't personally need to sniff more than 4 wires at a time. uart spi twi canbus all use 4 or fewer, right?

gillham commented 5 years ago

I don’t know that the SUMP metadata spec allows the buffer size to change based on the sample rate. Possibly. This would significantly complicate the code that is hand padded with nop instructions to get the timing as precise as possible. I’ve been considering splitting the code into separate sketches for different hardware that isn’t as easily supported with #ifdef now. That might make it easier to do special versions with fewer lines and more buffer at the cost of lower sample rates as well. Thoughts?