HSerin problems on power up

[Luckyborg]

This is a problem I've encountered for years on several different chips and have always just kind of ignored it. I have a problem using the Hserin command on initial power up. Below is a stripped down version of my code. Basically I power up the chip and wait a little while then send a burst of data. that data is basically ignored, I send a second burst and it is processed as is every burst after. if I hit the reset button every burst is processed, but if I turn off the power and wait a few seconds before powering back up the problem comes back for the first burst of data. I've checked the overflow and framing error bits, and see no problem. There are only a few registers I can even think to look at, but nothing is popping out at me. it seems like it would have to be something obvious.



INCLUDE "DT_INTS-18.bas" ; Base Interrupt System
INCLUDE "ReEnterPBP-18.bas" ; Include if using PBP interrupts

Clear
DEFINE OSC 40 ' Set Xtal Frequency
DEFINE HSER_CLROERR 1 'will automatically clear overflow errors

stream VAR BYTE[80] 'the last stream in
view_stream var byte[104] 'outsream converted to patterns

a VAR BYTE 'loop counter variable
b VAR BYTE
x VAR BYTE 'loop counter variable
B0 VAR BYTE
B1 VAR BYTE
B2 VAR BYTE
B3 VAR BYTE
B4 VAR BYTE
B5 var byte

pattern var byte
new_stream var bit

red_LED VAR PORTC.4 'red led

CSALL CON %00000000 'Chip Select for all chips
CSOFF CON %00001111 'chip select for no chips

CSB0 VAR PORTB.0
CSB1 VAR PORTB.1
CSB2 VAR PORTB.2
CSB3 VAR PORTB.3

TRISA = %11101100
TRISB = %11110000 'inputs for upper half switches outputs for lower CS
TRISC = %11001111 '0-3 for switches, 4-5 for LEDs, 6-7 for serial data
TRISD = 0
TRISE = 0

PORTB = CSOFF
PORTE.0 = 1
PORTE.1 = 1
PORTE.2 = 0

ADCON1 = %00001111

;****************Using Darrel Taylor Interrupts****************************
;----[High Priority Interrupts]--------------------------------------------
ASM
INT_LIST macro ; IntSource, Label, Type, ResetFlag?
INT_Handler RX_INT, _GetData, PBP, no
endm
INT_CREATE ;Creates the High Priority interrupt processor
ENDASM

red_LED = 0 'turn on red LED

Pause 1000

top:

TXSTA = $20 ' Enable transmit, BRGH = 0
SPBRG = 34 ' 1200 Baud @ 0.0%
SPBRGH = 8
BAUDCON.3 = 1 ' Enable 16 bit baudrate generator
RCSTA = $90 ' Enable serial port & continuous receive

GoSub send_data

countdown:
'count from 9 to 0

For x = 9 TO 0 STEP - 1
For a = 0 TO 79
stream[a] = x + "0"
Next a
GoSub send_data
Pause 150
Next x

end_countdown:
'check buttons

@ INT_ENABLE RX_INT ; enable RX_INT interrupts

RCSTA = $80 'clear CREN
RCSTA = $90 'Enable receive



mainloop:


if new_stream = 1 then
RCSTA.4 = 0
RCSTA.4 = 1 'clear over run error
new_stream = 0
GoSub send_data
endif

GoTo mainloop

send_data:

'this sub will send bit patterns via psp (non interrupt driven) to another chip

Return


'---[USART RX - interrupt handler]-----------------------[High Priority]---
GetData:

hserin 1000,No232,[str stream\80\10]

red_LED = red_LED +1 'heartbeat
new_stream = 1

No232:
@ INT_RETURN

End

Thanks
David

[mister_e]

If there's HSERIN/HSEROUT, you should use DEFINEs, unless you just need to write the USART registers.



INT_LIST macro ; IntSource, Label, Type, ResetFlag?
INT_Handler RX_INT, _GetData, PBP, no

You want to reset flags, unless, your program will loop in the ISR forever.

I haven't look the whole thing, but those are the really first thing that jump in my face.

[Luckyborg]

Thanks, I'll give the reset flags a try latter. I can't use defines as the baud rate needs to be user configurable in the field.

[mister_e]

Yes you can do it, you define one at the top, then you change it later on the fly by writing SPBRG, RCSTA, TXSTA. Check the datasheet for the recommended sequence.

[Luckyborg]

I finally had a chance to try out your suggestions by putting the defines at the beginning of the program and trying to reset the flag, but I have the same result of garbage in on the first transmission, then clear after that. Any other ideas would be appreciated.

David

[mister_e]

It may happen if your transmitter use SEROUT/SEROUT2/DEBUG and the pin idle state is not set before sending the first data.

Post your both code, receiver and transmitter end.

[Luckyborg]

I use a VB program not a pic as the transmitter. I can leave the program running while the pic is powered down. I give the pic a few seconds stabilization time after power up before the next transmission is sent

David

[mister_e]

Ok then, modify your VB program to send few garbage character, then a Synch String (let's say OK), and in your ISR use WAIT("OK") . See if this solve your problem.

[Luckyborg]

I'm sure this would work, I just figured I would see if anyone knew why I can't get it to work as expected. It isn't critical to the application, just one of those annoyances I figured most people weren't living with :)