It just doesn't work. - Githubissues

EUA / ESP32_LogicAnalyzer

ESP32 Logic Analyzer

GNU General Public License v3.0

206 stars 27 forks source link

It just doesn't work. #3

Open celesteking opened 3 years ago

celesteking commented 3 years ago

Thanks for the project, but it doesn't work. First of all, 1) user documentation is missing, I had hard time installing and getting it running. Secondly, 2) it's impossible to know what pins are used for what channels. Thirdly, 3) it doesn't work, it shows garbage. This is on devkitc (wrover32).

I was able to capture proper data, but it's very unreliable. I also had to modify .h header and use uart0 as I had no idea, see 1). I don't need 16 channels, I'm fine with 1-2. Deselecting them in sigrok makes it go nuts. see 3).

Also, have you compared it to real analyzer, how far is it off? I have suspicion it's a bit lying, the timing is a bit wrong. Thanks.

celesteking commented 3 years ago

The only reason I came here is because I am stingy and can't afford myself a $20 logic anal.

crbble commented 2 years ago

It doesn't work for you? Make it work.

Quickstart: Running a ESP-WROOM-32, change #define USE_SERIAL2_FOR_OLS 0 in ESP32_LogicAnalyzer.h if you are lazy and want to use the USB connection. Compile, download to ESP, start PulseView, Select ols device, serial connection, 921600 baud, device should now show up in device list. Select the device.

Screenshot_20211103_222031

Impossible to know which IO is which channel? You have a 3.3 V reference on your board, so try it out. Hint: D12 is channel 12. The software has plenty of bugs, though. Triggering somewhat works, but seems unreliable. Sample count is really low, still enough for some UART debugging at 9600 baud :-)

EUA commented 10 months ago

I found a problem, might be it's related with this. Espressif IDF changed and now we can't use LEDC output to I2S input from same pin, directly. Only solution that I have is connecting 2 pins together. I make a code and looks good.

Also, pulseview is now draining com port before trying to communicate. That means, not need to GPIO15 to GND anymore. I make a new & better code. Sorry for previous state of code... New one probably will be easy to use for everyone.