for esp32 chips with version V3

Open Sheimy opened this issue 3 years ago • 3 comments

for esp32 chips with version V3 you need to change Update ESP32SJA1000.cpp

writeRegister(REG_IER, 0xef); // enable all interrupts like this

Sep 07 '22 06:09 Sheimy

I've tested this with both V3 and older boards and they work with the suggested register configuration. Safe to merge

Sep 14 '22 14:09 aldoir

Works for me too. Why isn't this merged in @sandeepmistry ?

Apr 07 '23 10:04 jLynx

Here's the commit by Kenji Takahashi on mister0wl's forked version of the library: https://github.com/sandeepmistry/arduino-CAN/pull/71/commits/232562c985cc63127036cf587e855064f54aa415

It has: modifyRegister(REG_IER, 0xef, 0xef); // enable all interrupts instead of writeRegister(REG_IER, 0xef); // enable all interrupts

Does anybody know which one is right/why they're different?

May 21 '23 23:05 zalexzperez