DT_INT + 16F72 - what am I missing?

[Megahertz]

Hi, I am trying to build a timer based application and using PIC16F72. My problem is that my PIC is not responding to the Serin data. I am using DT_INT to achieve the timing function.
Please help me understand if I am missing something.
My code goes like this:

;
;-----------------PIC16F72---------------------------
Include "modedefs.bas"
INCLUDE "DT_INTS-14.bas"     ; Base Interrupt System
INCLUDE "ReEnterPBP.bas"     ; Include if using PBP interrupts
DEFINE OSC 4       ' OSCCON defaults to 4MHz on reset
 
;-------------CONFIGURATION FUSES SET HERE-------------------------
@ __Config _XT_OSC & _WDT_ON & _PWRTE_ON & _BODEN_ON & _CP_ALL
 
;---------VARIABLES DECLARED HERE----------------
PAUSE 100
Rx Var PortB.7
buz var PortA.2
green var PortA.1
red var PortA.0
relay var PortC.2
delay VAR word
delayset var word
get0 var byte
o var bit
s var bit
get1 var byte
 
;------------REGISTERS SET HERE------------------------
  ADCON1=7
  PORTA=0
  PORTB=0
  PORTC=0
  TRISA = %000000
  TRISC = %00000000
  TRISB = %10000000
  OPTION_REG = %10000000  ' RAPU = off
  PORTA=0
  PORTC=0
  PORTB=0
 
s=0
high red : pause 500 : high relay : pause 500 : high green : pause 500 : high buz : pause 1000
low red : pause 500 :low relay : pause 500 : low green : pause 500 : low buz : pause 1000
; the above works fine
;---------INTERRUPTS DECLARED HERE------------------------------
ASM
INT_LIST  macro    ; IntSource,        Label,  Type, ResetFlag?
        INT_Handler    TMR1_INT,  _delaystop,   PBP,  yes
    endm
    INT_CREATE               ; Creates the interrupt processor
ENDASM
 
T1CON=$31
 
;-------------INTERRUPTS ENABLED HERE-----------------------------
@   INT_ENABLE   TMR1_INT     ; Enable external (INT) interrupts
 ; the int works fine as I have toggled an led in ISR and it does toggle at regular intervals
;----------------MAIN PROGRAM STARTS HERE------------------------

main:
 
game:
 Serin PortB.7, N2400,["D33",get0]
high buz : pause 50 : low buz : pause 500
 
   Select case get0
; all select case statements go here which switch on the relay part & leds as well
   End Select 
goto main
 
;------------------[INTERRUPTS - interrupt handler]---------------------------------------------------
delaystop:
; All code to switch off the relay goes here after certain time has elapsed
@ INT_RETURN

I dont know what I am missing but my PIC is not respnding to the serial data. The buzzer should beep once, but it doesn't. Am I missing something?
I can upload the missing code if needed.

[Ioannis]

What is your hardware? I mean how is the controller connected to the PC?

How are you sending the serial data stream?

Ioannis

[Megahertz]

What is your hardware? I mean how is the controller connected to the PC?

How are you sending the serial data stream?

Ioannis

I have an RF receiver attached to the unit. I even tried to remove the receiver and connect both PICs hard wired. Still no response.
By the way, just to confirm that for the 16F72 do I need to connect 100nF anywhere else as well, I have one already connected to pin 19/20. Vss is supplied to pin 19 & 8, Vdd goes to pin20 only.

[Bruce]

Have you tried disabling the Timer1 interrupt to see if it responds to serial data?

[Megahertz]

Have you tried disabling the Timer1 interrupt to see if it responds to serial data?

Yes I did, still no luck.

[Bruce]

If you're using v2.6 you may want to switch to SERIN2 or DEBUGIN. SERIN has known timing issues: http://melabs.com/support/pbpissues.htm

[Megahertz]

I have got V2.5. Can I still try using DEBUG?

[Bruce]

Yes. DEBUGIN should work fine. I would disable interrupts too. At least until you can get your serial input working.

[Darrel Taylor]

> Serin PortB.7, N2400,["D33",get0]

With that statement, get0 is a qualifier.
I think you wanted that to be the variable that receives the data.
So it should be outside of the square brackets.

Serin PortB.7, N2400,["D33"],get0

[Megahertz]

> Serin PortB.7, N2400,["D33",get0]

With that statement, get0 is a qualifier.
I think you wanted that to be the variable that receives the data.
So it should be outside of the square brackets.

Serin PortB.7, N2400,["D33"],get0

Thanks to all. The problem is now sorted.