ISR code must be in ICACHE_RAM - the check was commited in ab125162bf #6002

dok-net · 2019-04-20T11:28:07Z

#5995 hasn't actually merged any PR - the minimum to get actual code working is in this PR.
Caveat: I don't know how to check if ICACHE_RAM_ATTR has the desired effect on lambda expressions, if it does not, there will still be an adverse performance (no real-time behavior!) impact over the standard Arduino attachInterrupt() usage.
Q: why not implement void attachInterruptArg(uint8_t pin, void ()(void), void * arg, int mode) like the ESP32 has already?

d-a-v · 2019-04-20T13:37:46Z

why not implement void attachInterruptArg(uint8_t pin, void ()(void), void * arg, int mode) like the ESP32 has already?

It is the same

typedef void (*voidFuncPtr)(void);
extern void ICACHE_RAM_ATTR __attachInterruptArg(uint8_t pin, voidFuncPtr userFunc, void *arg, int mode) {

d-a-v · 2019-04-20T13:39:56Z

I don't know how to check if ICACHE_RAM_ATTR has the desired effect on lambda expressions

I used that code: #5780 (comment)

ISR code must be in ICACHE_RAM - the check was commited in ab12516

706c507

dok-net mentioned this pull request Apr 20, 2019

ISR: check for address in IRAM #5995

Merged

d-a-v merged commit 0dd6549 into esp8266:master Apr 20, 2019

dok-net mentioned this pull request Apr 24, 2019

Fixes and implementation to expose attachInterruptArg in Arduino.h #6003

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

ISR code must be in ICACHE_RAM - the check was commited in ab125162bf #6002

ISR code must be in ICACHE_RAM - the check was commited in ab125162bf #6002

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!