Library Interrupts Conflict about rbddimmer

[siteswapjuggler]

Ok some investigation later :

• commenting #define INT_vect lines in the header file make the compilation ok (I did not test the functionnal aspect as I debug this remotely for a friend without hardware to test it).

• seems like the interrupt method used in void dimmerLamp::ext_int_init(void) is not compatible with the Arduino standard which use attachInterrupt to keep track of the different IST method associated to each interrupt pins.

• doing so the ISR(INT_vect) come in conflic with the core library

I will try some fixes but I will not have so much time to work on it correctly. This error could be common with other open issues so I hope it will help someone to correct it at one point :)

[siteswapjuggler]

Replacing content of ext_int_init by attachInterrupt(INTx,ISR_RBD,RISING); should do the trick. You will also have to declare ISR function befor ext_int_init and change ISR name by ISR_RBD.

[blackbird257]

Hi! , thank you for the research of the library issue. I have same issue with the interruptions (I use another device on port D3).
I'm trying to follow your advices, but I don't exactly know how to "declare ISR function before ext_int_init" - I just copied it before ext_int_init and renamed it to ISR_RBD. I get this error:
/mnt/create-efs/webide/16/d4/16d4ee6b75059aa8801ddf6d27b06fde:lubos257/libraries_v2/RBDdimmer/src/avr/RBDmcuAVR.cpp:80:8: error: expected constructor, destructor, or type conversion before '(' token

ISR_RBD(INT_vect)

^

/mnt/create-efs/webide/16/d4/16d4ee6b75059aa8801ddf6d27b06fde:lubos257/libraries_v2/RBDdimmer/src/avr/RBDmcuAVR.cpp: In member function 'void dimmerLamp::ext_int_init()':

/mnt/create-efs/webide/16/d4/16d4ee6b75059aa8801ddf6d27b06fde:lubos257/libraries_v2/RBDdimmer/src/avr/RBDmcuAVR.cpp:92:23: error: 'ISR_RBD' was not declared in this scope

attachInterrupt(INTx,ISR_RBD,RISING);

Could you be more specific with "declare ISR function before ext_int_init" ,
2. question, down in the code of ccp file is also anothre ISR function: ISR (TIMER_COMPA_VECTOR(DIMMER_TIMER))
, should I rename also that one?
Sorry for stupid questions, I'm beginner with programming.

[siteswapjuggler]

Well by replacing the normal content of ext_int_init activate the function which implicitely link to

ISR(INT_vect)
{
for (int i = 0; i < current_dim; i++ )
if (dimState[i] == ON)
{
zeroCross[i] = 1;
}
}

By changing it to attachInterrupt(INTx,ISR_RBD,RISING); the link become explicit so you can replace the previous function by

ISR_RBD()
{
for (int i = 0; i < current_dim; i++ )
if (dimState[i] == ON)
{
zeroCross[i] = 1;
}
}

But then you need to declare it (move it) befor the declaration of ext_int_init ;)