esp32_can icon indicating copy to clipboard operation
esp32_can copied to clipboard

2515 reset watchdog

Open j4m3s opened this issue 4 years ago • 2 comments

Have combined two changes here - happy to discuss. Both changes are to the MCP2515 code only.

  • The first aligns the priorities of the MCP2515 threads to match the priorities in the 2517 code and the internal controller code.
  • The second adds a watchdog thread that runs every 2 seconds to check the chip for an error state and reset it if needed. This brings the MCP2515 chip code up to match the 2517 code.

j4m3s avatar Oct 21 '20 21:10 j4m3s

Hi James, is the MCP2515 working for you with this changes? Did you try it with ESP32RET? I spend many hours to fix the 2515 driver and it is working now. (See my push). However I get ESP32RET not working with it.

Modellfan avatar Feb 04 '23 18:02 Modellfan

Hi @Modellfan

I didn't have any issues with MCP2515 code. My PR was mainly behind-the-scenes improvements rather than anything necessary to get it to work. I've had a scan through the changes on your fork; there's quite a lot of whitespace and logging changes, and a few hard-coded values - I can't quite tell what changes you made that were necessary to get it to work for you?

In the end I switched to an MCP2518FD chip (which uses the 2517 code) so haven't been using the MCP2515 code. Maybe something has changed since I last used it successfully...

sorry, to answer your second question, I never tried it with ESP32RET, I used something else for CAN sniffing/ decoding.

j4m3s avatar Feb 04 '23 18:02 j4m3s