By the way, I did notice some of my comments are not up to date. All interupts are on rising edge and some of the timing has been revised from the comments. Hope this didn't cause any confusion.
|
By the way, I did notice some of my comments are not up to date. All interupts are on rising edge and some of the timing has been revised from the comments. Hope this didn't cause any confusion.
|
Hi Butch,
Have not gone over the code carefully, but there are a few things that don't look quite right...
It looks like OPTION_REG is set for falling edge trigger (though I guess you mentioned that already).
Pullups are ENABLED, but the WPU register is not configured to set which pins have pullups enabled.
Can't find "SENSOR" GPIO.2 set as input. Don't know what it defaults to in the '629.
There should be a DISABLE instruction before your interrupt service routines, with an ENABLE to restart interrupts when needed.
You might try tossing the interrupt service sub-routines up at the top of your code before the "ON INTERRUPT" command is first used. Than just jump around them with a "GOTO StartHere". Others prefer to put the subs at the end of their code.
There's a "RESUME BASE" at the end of your "REFRESH" section. Since you jumped to REFRESH with a "GOTO", the RESUME is not needed (don't know if including it would actually cause problems or not).
Arch
|
Arch, thanks for the reply. I ended up redefining a few things and moving a couple of things around and got it to work.
Members who have read this thread : 0You do not have permission to view the list of names.
Posting Permissions
Bookmarks