-
Framing error if I disable transmitter after shift register is empty?
TX pin can be set manually, BUT THIS IS ONLY MASKING THE PROBLEM, NOT A REAL SOLUTION.
See Richard's post #33 and onwards for details.
Code:
UsartTX var LatC.6
UsartTX = 1
TXSTA.5 = 0
------------------------------------------------------
16F1937
Gutted my program to try to figure this out.
Code:
#CONFIG
__CONFIG _CONFIG1, _FOSC_INTOSC & _WDTE_OFF & _PWRTE_ON & _MCLRE_ON & _CP_OFF & _CPD_OFF & _BOREN_OFF & _CLKOUTEN_OFF & _IESO_OFF & _FCMEN_OFF
__CONFIG _CONFIG2, _WRT_OFF & _VCAPEN_OFF & _PLLEN_OFF & _STVREN_OFF & _BORV_LO & _LVP_OFF
#ENDCONFIG
DEFINE OSC 32
SPLLEN CON %1 ' PLL enable
IRCF CON %1110 ' to enable 8 MHz
SCS CON %00 ' system clock determined by FOSC
OSCCON = (SPLLEN << 7) | (IRCF << 3) | SCS
DEFINE HSER_RCSTA 90h ' Enable serial port & continuous receive
DEFINE HSER_TXSTA 24h ' Enable transmit, BRGH = 1
Define HSER_BAUD 115200
DEFINE HSER_CLROERR 1 ' Clear overflow automatically
DEFINE HSER_SPBRGH 0
DEFINE HSER_SPBRG 68
BAUDCON.3 = 1 ' Enable 16 bit baudrate generator
ANSELA = %00000000
ANSELB = %00000000
'ANSELC = %00000000 ' ...not available
ANSELD = %00000000
ANSELE = %00000000
TRISA = %00000000
TRISB = %00000000
TRISC = %00000000
TRISD = %00000000
TRISE = %00000000
BlinkLED VAR LatB.5
Pause 500
hserout ["Hello world", 10]
TXSTA.5 = 1 ' TXEN: Transmit Enable bit
hserout ["TEST", 10]
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
TXSTA.5 = 0 ' <----- Causes Framing error after last byte !
Mainloop:
BlinkLED = 1
BlinkLED = 0
goto mainloop
end
Checking on Logic Probe, the line remains LOW after the last HSEROUT (4K7 pull-up on pin).
Yet processing is normal if I comment TXSTA.5 = 0.
-
1 Attachment(s)
Re: Framing error if I disable transmitter after shift register is empty?
as best i can see the data sheet indicates that what u are trying should be ok
Attachment 9753
could hserout pacing be too slow to buffer tx data, and not really triggering back-back transmissions ?
if so then the last byte is still not sent yet.
if you used a decent chip you could make the tx pin open drain and not need this sort of worry
-
1 Attachment(s)
Re: Framing error if I disable transmitter after shift register is empty?
I'm using a 16F1937, and I've done this successully in the past on this PIC model.
Check this out, it's always after the last HSEROUT:
Code:
TXSTA.5 = 1 ' TXEN: Transmit Enable bit
hserout ["1"]
hserout ["2"]
hserout ["3"]
hserout ["4"]
hserout ["5"]
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
' TXSTA.5 = 0 ' <----- Causes Framing error after last byte !
Attachment 9754
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
richard
...could hserout pacing be too slow to buffer tx data, and not really triggering back-back transmissions ?
if so then the last byte is still not sent yet....
I thought this would ensure all bytes were transmitted?
Code:
while TXSTA.1 = 0
wend
Quote:
Originally Posted by
richard
...if you used a decent chip you could make the tx pin open drain and not need this sort of worry
I don't see why a 16F1937 could not do this reliably...?
-
1 Attachment(s)
Re: Framing error if I disable transmitter after shift register is empty?
The last byte DOES get transmitted.
Code:
TXSTA.5 = 1 ' TXEN: Transmit Enable bit
hserout ["1"]
pause 1
hserout ["2"]
pause 1
hserout ["3"]
pause 1
hserout ["4"]
pause 1
hserout ["5"]
pause 1
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
TXSTA.5 = 0 ' <----- Causes Framing error after last byte !
Something happens when the transmitter is disabled:
Attachment 9755
I compare with my code and wiring on my old test circuit, and I can't see anything different...?
-
Re: Framing error if I disable transmitter after shift register is empty?
The Shift register you mentioned is the 2 Bytes Buffer?
Ioannis
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
Ioannis
The Shift register you mentioned is the 2 Bytes Buffer?
Ioannis
I suppose so. This is from the datasheet for TXSTA register:
Quote:
bit 1 TRMT: Transmit Shift Register Status bit
1 = TSR empty
0 = TSR full
-
1 Attachment(s)
Re: Framing error if I disable transmitter after shift register is empty?
It's getting even dumber by the minute. :D
TX code on 1st 16F1937:
Code:
#CONFIG
__CONFIG _CONFIG1, _FOSC_INTOSC & _WDTE_OFF & _PWRTE_ON & _MCLRE_ON & _CP_OFF & _CPD_OFF & _BOREN_OFF & _CLKOUTEN_OFF & _IESO_OFF & _FCMEN_OFF
__CONFIG _CONFIG2, _WRT_OFF & _VCAPEN_OFF & _PLLEN_OFF & _STVREN_OFF & _BORV_LO & _LVP_OFF
#ENDCONFIG
DEFINE OSC 32
SPLLEN CON %1 ' PLL enable
IRCF CON %1110 ' to enable 8 MHz
SCS CON %00 ' system clock determined by FOSC
OSCCON = (SPLLEN << 7) | (IRCF << 3) | SCS
DEFINE HSER_RCSTA 90h ' Enable serial port & continuous receive
DEFINE HSER_TXSTA 24h ' Enable transmit, BRGH = 1
Define HSER_BAUD 115200
DEFINE HSER_CLROERR 1 ' Clear overflow automatically
DEFINE HSER_SPBRGH 0
DEFINE HSER_SPBRG 68
BAUDCON.3 = 1 ' Enable 16 bit baudrate generator
ANSELA = %00000000
ANSELB = %00000000
'ANSELC = %00000000 ' ...not available
ANSELD = %00000000
ANSELE = %00000000
TRISA = %00000000
TRISB = %00000000
TRISC = %10000000
TRISD = %00000000
TRISE = %00000000
BlinkLED VAR LatB.5
Pause 200
TXSTA.5 = 1 ' TXEN: Transmit Enable bit
hserout [ "[", 1, "]"]
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
TXSTA.5 = 0 ' <----- Causes Framing error after last byte !
Mainloop:
BlinkLED = 1
BlinkLED = 0
goto mainloop
end
RX code on 2nd 16F1937:
Code:
#CONFIG
__CONFIG _CONFIG1, _FOSC_INTOSC & _WDTE_OFF & _PWRTE_ON & _MCLRE_ON & _CP_OFF & _CPD_OFF & _BOREN_OFF & _CLKOUTEN_OFF & _IESO_OFF & _FCMEN_OFF
__CONFIG _CONFIG2, _WRT_OFF & _VCAPEN_OFF & _PLLEN_OFF & _STVREN_OFF & _BORV_LO & _LVP_OFF
#ENDCONFIG
include "DT_INTS-14.bas"
include "ReEnterPBP.bas"
ASM
INT_LIST macro ; IntSource, Label, Type, ResetFlag?
INT_Handler RX_INT, _ReceiveInterrupt, PBP, yes
endm
INT_CREATE ; Creates the interrupt processor
ENDASM
DEFINE OSC 32
SPLLEN CON %1 ' PLL enable
IRCF CON %1110 ' to enable 8 MHz
SCS CON %00 ' system clock determined by FOSC
OSCCON = (SPLLEN << 7) | (IRCF << 3) | SCS
DEFINE HSER_RCSTA 90h ' Enable serial port & continuous receive
DEFINE HSER_TXSTA 24h ' Enable transmit, BRGH = 1
Define HSER_BAUD 115200
DEFINE HSER_CLROERR 1 ' Clear overflow automatically
DEFINE HSER_SPBRGH 0
DEFINE HSER_SPBRG 68
BAUDCON.3 = 1 ' Enable 16 bit baudrate generator
ANSELA = %00000000
ANSELB = %00000000
'ANSELC = %00000000 ' ...not available
ANSELD = %00000000
ANSELE = %00000000 ' Pin E0 = ADC input
TRISA = %00000000
TRISB = %00000000
TRISC = %10000000
TRISD = %00000000
TRISE = %00000000
LEDblink var PORTD.0
RecvData var BYTE[11]
LEDblink = 0
goto Start
ReceiveInterrupt:
hserin [ wait("["), STR RecvData\11\"]" ]
TXSTA.5 = 1
hserout [ "[", 0, "]" ]
while TXSTA.1 = 0 ' Check TRMT bit
wend
TXSTA.5 = 0
@ INT_RETURN
Start:
Pause 200 ' Let PIC and LCD stabilize
@ INT_ENABLE RX_INT
Mainloop:
LEDblink = 1
LEDblink = 0
GOTO Mainloop
end
The RX PIC has the exact same HSEROUT code:
- enable transmitter
- transmit data
- wait until buffer is empty
- disable transmitter
DISABLE transmitter causes a FRAMING ERROR on 1st PIC, but not on Ack message on 2nd PIC.
Attachment 9756
This has to be a really stupid error cause there's practically no code left. :D
EDIT: Does it make a difference if I disable transmitter inside the Interrupt routine?
EDIT SOME MORE: I moved the DISABLE out of the ISR and into Mainloop, it didn't make a difference during RX on the 2nd PIC.
-
Re: Framing error if I disable transmitter after shift register is empty?
I'm not sure exactly what you intend to do, but TRMT gets asserted half-way into the STOP bit, so if you immediately disable the TX then the STOP bit will be half the time and generate a framing error.
The datasheet doesn't really specify that TXEN controls the state of the TX output pin (see datasheet section 25.1.1.1), but if it does return control back to the IO PORTC/TRISC registers then you'd have to make sure that the TX pin (RC6) is set to output high.
If you want it to be an input/tristate with an external pullup then set TRISC6.
-
Re: Framing error if I disable transmitter after shift register is empty?
checking your stuff........ if you use interrupts (DT ints), for RX interrupts, you would not use PB hserin in the int routine. Rather get the characters in your routine and store them in the locations you designated and advance counter and check for some beginning or ending character....... with this I receive up to 1000 characters and store them in ram on 18f with 4k ram..... you would fix for your code. If successful receive, raised a flag telling basic to process the data for whatever your looking for
Code:
'''-------------RCV INT ROUTINE----------------''''
RCint:
RCX=RCREG
'@ bcf RCSTA,4
'@ bsf RCSTA,4
'''@ CLRF CREN ;RESET RCV overrun
'''@ SETF CREN
IF RCX=8 THEN '''CHECK FOR BACK SPACE BS 8dec
SELECT CASE RCVindx
CASE 0
GOTO LEAVEOK1
CASE 1
RCVindx=RCVindx-1 :RCVOK=0
RCVdata[RCVindx]=0
GOTO LEAVEOK1
CASE IS >1
RCVindx=RCVindx-1
RCVdata[RCVindx]=0
GOTO LEAVEOK1
END SELECT
ENDIF
if RCX = "[" then
RCVindx=0:RCVok=1:DATAready=0
goto leaveok
endif
if RCVok=0 then LEAVEOK1
if RCX = "]" then
rcvok=0:dataready=1
endif
leaveok: 'LEAVE AND STORE RCVD CHAR
RCVdata[RCVindx]=RCX
RCVindx=RCVindx+1
RCVdata[RCVindx]=0 'TACK A 0 AT END OF ARRAY
IF RCVindx > 999 THEN
RCVindx=999
ENDIF
LEAVEOK1: 'LEAVE WITHOUT STORING RCVD CHAR
RCFLAG=1
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
tumbleweed
I'm not sure exactly what you intend to do, ...
I'll be adding several more PICs on the network later, along with a tri-state BUSY line. The BUSY line will determine which PIC will be permitted to TRANSMIT at any given time. And that's why I ENABLE / TX / DISABLE, that code is in anticipation of more PICs being added.
Quote:
Originally Posted by
tumbleweed
...If you want it to be an input/tristate with an external pullup then set TRISC6
I have 4K7 pull-ups on both TX and RX pins. I only have RX set as Input. I plan on using a separate Busy line as tri-state.
I still don't see why PIC #1 gets a framing error, when PIC #2 doesn't and it's using essentially the exact same code...?
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
amgen
checking your stuff........ if you use interrupts (DT ints), for RX interrupts, you would not use PB hserin in the int routine. Rather get the characters in your routine and store them in the locations you designated and advance counter and check for some beginning or ending character...
But the 2nd RX PIC receives perfectly. It's the 1st TX PIC that gives a framing error. And yet the 2nd RX gives an ACK using the HSEROUT structure, and it can DISABLE transmitter just fine.
I just don't get why PIC #1 cannot TXSTA.5 = 0, but PIC #2 can.
(I am going to keep that code on file, cause I'm sure it's going to be useful one day)
EDIT: I'm starting to think maybe I'm too dense to see the forest...
-
Re: Framing error if I disable transmitter after shift register is empty?
Question: why do people use TX INT?
I can understand why you want to use RX INT, cause you don't know when the data will be incoming, but I do know when the data is being transmitted.
I'm just wondering out loud that maybe I'm not using the best technique to transmit...?
-
Re: Framing error if I disable transmitter after shift register is empty?
don't give up on yourself or sell yourself short !...... it is a good project :)
-
1 Attachment(s)
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
amgen
don't give up on yourself or sell yourself short !...... it is a good project :)
It's an awesome project. I can almost see the light at the end of the tunnel. I've done unit tests of every component all the way up to interfacing with MS Flight Sim. I'm just not seeing why I can't disable the transmitter in PIC #1.
I added a PAUSE to guarantee that everything is sent on PIC #1. The 2 PICs transmit successfully back and forth, and then I get a lone FRAMING ERROR all by itself, and PIC #2 locks up (there are no more blinking LED on channel 3).
Code:
TXSTA.5 = 1 ' TXEN: Transmit Enable bit
hserout [ "[1]" ]
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
PAUSE 100
TXSTA.5 = 0 ' <----- Causes Framing error after last byte !
So it's not an issue of losing bytes, it's definitely something peculiar about TXSTA.5 = 0.
Attachment 9757
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
DISABLE transmitter causes a FRAMING ERROR on 1st PIC, but not on Ack message on 2nd PIC
In post 8 the trace is showing that the analyzer is detecting a framing error being generated by the transmitter when it sends the ']' and disables TXEN.
The TX data line should never idle low... that looks like a serial BREAK condition to the receiving side and will likely generate framing and overrun errors on that end.
A framing error by itself doesn't really cause any issues, but an overrun error will stop everything in its tracks until you clear the error. I see you have 'DEFINE HSER_CLROERR 1', so hopefully that's trying to take care of it, but maybe not. The way you have the receive arranged if things get out of sync then it might lock up waiting for something that's not going to happen.
As I said, if setting TXEN=0 causes the TX data line to go low that you need to change the TRIS/LAT register settings in your setup to stop that from happening.
Try initializing the pins with LATC6 = 1 and TRISC6 = 1. That should set the default state of the TX pin to high, which is the proper idle state.
You'll have to rethink all of this later when you try to add more devices since you can't just tie multiple TX outputs together, even if they're "disabled".
They would need to be tri-state/open-drain in the idle state, not driven high or low.
-
1 Attachment(s)
Re: Framing error if I disable transmitter after shift register is empty?
I decreased the pause to 300uSec so everything is in the window when zoomed in.
The 2 transmits are working just fine, with the framing error over on the right.
Attachment 9758
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
tumbleweed
...You'll have to rethink all of this later when you try to add more devices since you can't just tie multiple TX outputs together, even if they're "disabled".
They would need to be tri-state/open-drain in the idle state, not driven high or low.
Yes, they'll definitely be tri-state.
-
1 Attachment(s)
Re: Framing error if I disable transmitter after shift register is empty?
PIC #1:
Code:
TXSTA.5 = 1 ' TXEN: Transmit Enable bit
hserout [ "[1]" ]
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
hserin [ wait("["), STR RecvData\11\"]" ]
hserout [ "[2]" ]
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
pauseus 300
TXSTA.5 = 0 ' <----- Causes Framing error after last byte !
Mainloop:
BlinkLED = 1
BlinkLED = 0
goto mainloop
end
PIC #2:
Code:
ReceiveInterrupt:
hserin [ wait("["), STR RecvData\11\"]" ]
UsartFlag = 1
@ INT_RETURN
Start:
Pause 200 ' Let PIC and LCD stabilize
Mainloop:
LEDblink = 1
if UsartFlag = 1 then
TXSTA.5 = 1
hserout [ "[0]" ]
while TXSTA.1 = 0 ' Check TRMT bit
wend
UsartFlag = 0
TXSTA.5 = 0
endif
LEDblink = 0
GOTO Mainloop
end
1. PICs can talk back and forth with no errors.
2. PIC #2 can disable transmitter every single time.
3. Framing error as soon as PIC #1 disables transmitter, causing PIC #2 to lock up (no more blink LED on bottom channel).
Attachment 9759
-
1 Attachment(s)
Re: Framing error if I disable transmitter after shift register is empty?
IT'S THE DARN PIC !
I swapped the coding between the 2 PICs, and PIC #1 now has a framing error every time it tries to disable the transmitter.
Attachment 9760
I'm going to swap it for another unit and see if that changes things.
-
Re: Framing error if I disable transmitter after shift register is empty?
when you disable the transmitter how are you setting the tx pin
code ?
-
1 Attachment(s)
Re: Framing error if I disable transmitter after shift register is empty?
Hmmm, I moved PIC #2 up in PIC #1 spot, and put in a new unit in PIC #2 spot.
PIC#1 can now disable transmitter at will, YAY! So that confirms that there's nothing on the breadboard interfering with that PIC.
New PIC #2 gets a framing error, but neither of the PICs freeze now (I put the blinky channel right under each PIC).
Attachment 9761
PIC#1:
Code:
Start:
TXSTA.5 = 1 ' TXEN: Transmit Enable bit
hserout [ "[1]" ]
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
TXSTA.5 = 0 ' <----- Causes Framing error after last byte !
hserin [ wait("["), STR RecvData\11\"]" ]
TXSTA.5 = 1 ' TXEN: Transmit Enable bit
hserout [ "[2]" ]
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
TXSTA.5 = 0 ' <----- Causes Framing error after last byte !
Mainloop:
BlinkLED = 1
BlinkLED = 0
GOTO Mainloop
end
PIC #2:
Code:
@ INT_ENABLE RX_INT
goto Start
ReceiveInterrupt:
hserin [ wait("["), STR RecvData\11\"]" ]
UsartFlag = 1
@ INT_RETURN
Start:
Mainloop:
LEDblink = 1
if UsartFlag = 1 then
TXSTA.5 = 1
hserout [ "[0]" ]
while TXSTA.1 = 0 ' Check TRMT bit
wend
UsartFlag = 0
TXSTA.5 = 0
endif
LEDblink = 0
GOTO Mainloop
end
Might have a few ideas more ideas to try out...
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
richard
when you disable the transmitter how are you setting the tx pin
code ?
I'm not doing anything to it. Should I?
-
Re: Framing error if I disable transmitter after shift register is empty?
explain how disabling tx-module sets the pin to "tristate"
-
Re: Framing error if I disable transmitter after shift register is empty?
I'm not using tri-state yet, cause for now I have only two PICs talking, and they're not sharing any lines (but I will be the future, hence why I'd like to master enable/disable of transmitter).
It's direct TX-to-RX and RX-to-TX with 1 set of 4K7 pull-ups.
Right now it's only an exercise.
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
It's direct TX-to-RX and RX-to-TX with 1 set of 4K7 pull-ups.
yet when tx is disabled the pin is hard driven to logic 0 , explain how disabling tx-module sets the pin to "tristate"
-
Re: Framing error if I disable transmitter after shift register is empty?
I'm not setting anything to tri-state, yet. Using TXSTA.5 = 1 / TXSTA.5 = 0 is just preparing myself for when I will use Usart pins in tri-state.
I thought I could just turn Usart TX and RX on/off whenever I wanted (when nothing was in the buffer).
Or was that a wrong assumption? :confused:
-
Re: Framing error if I disable transmitter after shift register is empty?
post the code . no point guessing
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
richard
post the code . no point guessing
Mainloop is in post #22.
Complete code PIC #1:
Code:
#CONFIG
__CONFIG _CONFIG1, _FOSC_INTOSC & _WDTE_OFF & _PWRTE_ON & _MCLRE_ON & _CP_OFF & _CPD_OFF & _BOREN_OFF & _CLKOUTEN_OFF & _IESO_OFF & _FCMEN_OFF
__CONFIG _CONFIG2, _WRT_OFF & _VCAPEN_OFF & _PLLEN_OFF & _STVREN_OFF & _BORV_LO & _LVP_OFF
#ENDCONFIG
DEFINE OSC 32
SPLLEN CON %1 ' PLL enable
IRCF CON %1110 ' to enable 8 MHz
SCS CON %00 ' system clock determined by FOSC
OSCCON = (SPLLEN << 7) | (IRCF << 3) | SCS
TXSTA = %00100100 ' DEFINE HSER_TXSTA 24h ' Enable transmit, BRGH = 1
' bit 7 CSRC: Clock Source Select bit
' Asynchronous mode:
' Don’t care
' Synchronous mode:
' 1 = Master mode (clock generated internally from BRG)
' 0 = Slave mode (clock from external source)
' bit 6 TX9: 9-bit Transmit Enable bit
' 1 = Selects 9-bit transmission
' 0 = Selects 8-bit transmission
' bit 5 TXEN: Transmit Enable bit(1)
' 1 = Transmit enabled
' 0 = Transmit disabled
' bit 4 SYNC: EUSART Mode Select bit
' 1 = Synchronous mode
' 0 = Asynchronous mode
' bit 3 SENDB: Send Break Character bit
' Asynchronous mode:
' 1 = Send Sync Break on next transmission (cleared by hardware upon completion)
' 0 = Sync Break transmission completed
' Synchronous mode:
' Don’t care
' bit 2 BRGH: High Baud Rate Select bit
' Asynchronous mode:
' 1 = High speed
' 0 = Low speed
' Synchronous mode:
' Unused in this mode
' bit 1 TRMT: Transmit Shift Register Status bit
' 1 = TSR empty
' 0 = TSR full
' bit 0 TX9D: Ninth bit of Transmit Data
' Can be address/data bit or a parity bit.
RCSTA = %10010000 'DEFINE HSER_RCSTA 90h ' Enable serial port & continuous receive
' bit 7 SPEN: Serial Port Enable bit
' 1 = Serial port enabled (configures RX/DT and TX/CK pins as serial port pins)
' 0 = Serial port disabled (held in Reset)
' bit 6 RX9: 9-bit Receive Enable bit
' 1 = Selects 9-bit reception
' 0 = Selects 8-bit reception
' bit 5 SREN: Single Receive Enable bit
' Asynchronous mode:
' Don’t care
' Synchronous mode – Master:
' 1 = Enables single receive
' 0 = Disables single receive
' This bit is cleared after reception is complete.
' Synchronous mode – Slave
' Don’t care
' bit 4 CREN: Continuous Receive Enable bit
' Asynchronous mode:
' 1 = Enables receiver
' 0 = Disables receiver
' Synchronous mode:
' 1 = Enables continuous receive until enable bit CREN is cleared (CREN overrides SREN)
' 0 = Disables continuous receive
' bit 3 ADDEN: Address Detect Enable bit
' Asynchronous mode 9-bit (RX9 = 1):
' 1 = Enables address detection, enable interrupt and load the receive buffer when RSR<8> is set
' 0 = Disables address detection, all bytes are received and ninth bit can be used as parity bit
' Asynchronous mode 8-bit (RX9 = 0):
' Don’t care
' bit 2 FERR: Framing Error bit
' 1 = Framing error (can be updated by reading RCREG register and receive next valid byte)
' 0 = No framing error
' bit 1 OERR: Overrun Error bit
' 1 = Overrun error (can be cleared by clearing bit CREN)
' 0 = No overrun error
' bit 0 RX9D: Ninth bit of Received Data
' This can be address/data bit or a parity bit and must be calculated by user firmware.
Define HSER_BAUD 115200
DEFINE HSER_CLROERR 1 ' Clear overflow automatically
DEFINE HSER_SPBRGH 0
DEFINE HSER_SPBRG 68
BAUDCON.3 = 1 ' Enable 16 bit baudrate generator
ANSELA = %00000000
ANSELB = %00000000
'ANSELC = %00000000 ' ...not available
ANSELD = %00000000
ANSELE = %00000000
TRISA = %00000000
TRISB = %00000000
TRISC = %10000000
TRISD = %00000000
TRISE = %00000000
BlinkLED VAR LatB.5
RecvData var BYTE[11]
Pause 200 ' Let PIC and LCD stabilize
BlinkLED = 0
Start:
TXSTA.5 = 1 ' TXEN: Transmit Enable bit
hserout [ "[1]" ]
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
TXSTA.5 = 0 ' <----- Causes Framing error after last byte !
hserin [ wait("["), STR RecvData\11\"]" ]
TXSTA.5 = 1 ' TXEN: Transmit Enable bit
hserout [ "[2]" ]
while TXSTA.1 = 0 ' Check TRMT: Transmit Shift Register Status bit
wend
TXSTA.5 = 0 ' <----- Causes Framing error after last byte !
Mainloop:
BlinkLED = 1
BlinkLED = 0
GOTO Mainloop
end
Complete code PIC #2:
Code:
#CONFIG
__CONFIG _CONFIG1, _FOSC_INTOSC & _WDTE_OFF & _PWRTE_ON & _MCLRE_ON & _CP_OFF & _CPD_OFF & _BOREN_OFF & _CLKOUTEN_OFF & _IESO_OFF & _FCMEN_OFF
__CONFIG _CONFIG2, _WRT_OFF & _VCAPEN_OFF & _PLLEN_OFF & _STVREN_OFF & _BORV_LO & _LVP_OFF
#ENDCONFIG
include "DT_INTS-14.bas"
include "ReEnterPBP.bas"
ASM
INT_LIST macro ; IntSource, Label, Type, ResetFlag?
INT_Handler RX_INT, _ReceiveInterrupt, PBP, yes
endm
INT_CREATE ; Creates the interrupt processor
ENDASM
DEFINE OSC 32
SPLLEN CON %1 ' PLL enable
IRCF CON %1110 ' to enable 8 MHz
SCS CON %00 ' system clock determined by FOSC
OSCCON = (SPLLEN << 7) | (IRCF << 3) | SCS
TXSTA = %00100100 ' DEFINE HSER_TXSTA 24h ' Enable transmit, BRGH = 1
' bit 7 CSRC: Clock Source Select bit
' Asynchronous mode:
' Don’t care
' Synchronous mode:
' 1 = Master mode (clock generated internally from BRG)
' 0 = Slave mode (clock from external source)
' bit 6 TX9: 9-bit Transmit Enable bit
' 1 = Selects 9-bit transmission
' 0 = Selects 8-bit transmission
' bit 5 TXEN: Transmit Enable bit(1)
' 1 = Transmit enabled
' 0 = Transmit disabled
' bit 4 SYNC: EUSART Mode Select bit
' 1 = Synchronous mode
' 0 = Asynchronous mode
' bit 3 SENDB: Send Break Character bit
' Asynchronous mode:
' 1 = Send Sync Break on next transmission (cleared by hardware upon completion)
' 0 = Sync Break transmission completed
' Synchronous mode:
' Don’t care
' bit 2 BRGH: High Baud Rate Select bit
' Asynchronous mode:
' 1 = High speed
' 0 = Low speed
' Synchronous mode:
' Unused in this mode
' bit 1 TRMT: Transmit Shift Register Status bit
' 1 = TSR empty
' 0 = TSR full
' bit 0 TX9D: Ninth bit of Transmit Data
' Can be address/data bit or a parity bit.
RCSTA = %10010000 'DEFINE HSER_RCSTA 90h ' Enable serial port & continuous receive
' bit 7 SPEN: Serial Port Enable bit
' 1 = Serial port enabled (configures RX/DT and TX/CK pins as serial port pins)
' 0 = Serial port disabled (held in Reset)
' bit 6 RX9: 9-bit Receive Enable bit
' 1 = Selects 9-bit reception
' 0 = Selects 8-bit reception
' bit 5 SREN: Single Receive Enable bit
' Asynchronous mode:
' Don’t care
' Synchronous mode – Master:
' 1 = Enables single receive
' 0 = Disables single receive
' This bit is cleared after reception is complete.
' Synchronous mode – Slave
' Don’t care
' bit 4 CREN: Continuous Receive Enable bit
' Asynchronous mode:
' 1 = Enables receiver
' 0 = Disables receiver
' Synchronous mode:
' 1 = Enables continuous receive until enable bit CREN is cleared (CREN overrides SREN)
' 0 = Disables continuous receive
' bit 3 ADDEN: Address Detect Enable bit
' Asynchronous mode 9-bit (RX9 = 1):
' 1 = Enables address detection, enable interrupt and load the receive buffer when RSR<8> is set
' 0 = Disables address detection, all bytes are received and ninth bit can be used as parity bit
' Asynchronous mode 8-bit (RX9 = 0):
' Don’t care
' bit 2 FERR: Framing Error bit
' 1 = Framing error (can be updated by reading RCREG register and receive next valid byte)
' 0 = No framing error
' bit 1 OERR: Overrun Error bit
' 1 = Overrun error (can be cleared by clearing bit CREN)
' 0 = No overrun error
' bit 0 RX9D: Ninth bit of Received Data
' This can be address/data bit or a parity bit and must be calculated by user firmware.
Define HSER_BAUD 115200
DEFINE HSER_CLROERR 1 ' Clear overflow automatically
DEFINE HSER_SPBRGH 0
DEFINE HSER_SPBRG 68
BAUDCON.3 = 1 ' Enable 16 bit baudrate generator
ANSELA = %00000000
ANSELB = %00000000
'ANSELC = %00000000 ' ...not available
ANSELD = %00000000
ANSELE = %00000000 ' Pin E0 = ADC input
TRISA = %00000000
TRISB = %00000000
TRISC = %10000000
TRISD = %00000000
TRISE = %00000000
LEDblink var LatD.0
RecvData var BYTE[11]
UsartFlag Var byte
Pause 200 ' Let PIC and LCD stabilize
LEDblink = 0
UsartFlag = 0
@ INT_ENABLE RX_INT
goto Start
ReceiveInterrupt:
hserin [ wait("["), STR RecvData\11\"]" ]
UsartFlag = 1
@ INT_RETURN
Start:
Mainloop:
LEDblink = 1
if UsartFlag = 1 then
TXSTA.5 = 1
hserout [ "[0]" ]
while TXSTA.1 = 0 ' Check TRMT bit
wend
UsartFlag = 0
TXSTA.5 = 0
endif
LEDblink = 0
GOTO Mainloop
end
I prefer to use:
TXSTA = %00100100
RCSTA = %10010000
instead of:
DEFINE HSER_TXSTA 24h
DEFINE HSER_RCSTA 90h
So that I can actually see what bits are set (or is there more going on in the background with DEFINE?)
-
Re: Framing error if I disable transmitter after shift register is empty?
so there is the problem
TRISC = % 10000000
portc pin 6 is set as an output
the latc.6 value is undefined but obviously 0
when tx module is disabled the pin reverts to be an output driven low , hence framing error
tumble weed pointed this out twice
and i said in post#2
if you used a decent chip you could make the tx pin open drain and not need this sort of worry
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
richard
so there is the problem
TRISC = % 10000000
portc pin 6 is set as an output
the latc.6 value is undefined but obviously 0...
Should I set LatC.6 to 1 when I disable the transmitter? :confused:
Quote:
Originally Posted by
richard
...when tx module is disabled the pin reverts to be an output driven low , hence framing error
tumble weed pointed this out twice...
I thought the pull-ups were supposed to handle that?
Quote:
Originally Posted by
richard
...and i said in post#2
if you used a decent chip you could make the tx pin open drain and not need this sort of worry
I chose 16F1937 mainly because of availability and price over at JLCPCB.
And I don't know how to "make the tx pin open drain"; I've never done that.
Microchip lists the 16F1937 as "in production", is there something I should know about this chip?
I don't honestly don't understand why you consider it not a decent chip?
-
Re: Framing error if I disable transmitter after shift register is empty?
Code:
UsartTX var LatC.6
UsartTX = 1
TXSTA.5 = 0
Framing error is gone on PIC #2
WOOT!
My new sig line "I understand quick, but you gotta explain slowly and a repeat a dozen times!" :D
-
1 Attachment(s)
Re: Framing error if I disable transmitter after shift register is empty?
there is another issue with that chip
from data sheet async reception chapter
Attachment 9762
Quote:
Should I set LatC.6 to 1 when I disable the transmitter?
no , it will look like it works but the pin is now hard driven high , not tristated at all
Quote:
I thought the pull-ups were supposed to handle that?
if the pin was tristated then it would
Quote:
And I don't know how to "make the tx pin open drain"; I've never done that
.
that chip cannot do it, more suitable to task chips have a ODCONx register
Quote:
I don't honestly don't understand why you consider it not a decent chip?
poorly suited to task at hand in my view
-
Re: SOLVED - Framing error if I disable transmitter after shift register is empty?
you have now declared the thread solved when it is not!
the problem is only temporally masked and will rear its ugly head when you enable another transmitter
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
richard
there is another issue with that chip
from data sheet async reception chapter ...
Oh crap, I didn't see that. :sad:
-
Re: SOLVED - Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
richard
you have now declared the thread solved when it is not!
the problem is only temporally masked and will rear its ugly head when you enable another transmitter
Yeah, I thought the problem went away. I'll edit them back.
-
Re: Framing error if I disable transmitter after shift register is empty?
Since the output on that specific chip will remain as driven output, and if you insist on using this, a crapy solution maybe to add a diode at the output before the pin is connected to the bus.
The cathode to the PIC usart output and anode to the bus.
Like an OR connection of all the PICs that will be in parallel. Then a pull up will hold line to HIGH state.
This is subject to capacitances over the bus and the pull-up resistor may not be able to charge that capacitances fast enough but you can test it and see if your are OK with the speed you may obtain.
The chip that you found to be faulty, may have been hit by ESD charge. Are you familiar with ESD protection? Are you taking measures when you handle ESD sensitive components?
Ioannis
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
Ioannis
Since the output on that specific chip will remain as driven output, and if you insist on using this, a crapy solution maybe to add a diode at the output before the pin is connected to the bus. ...
I "like" the 16F1937, but I have no allegiance towards it (I only have 15 on hand, I'll keep them for "something else"). :D I'm starting the slow/annoying process of choosing another PIC, with added requirement for ODCON register.
Quote:
Originally Posted by
Ioannis
... The chip that you found to be faulty, may have been hit by ESD charge. Are you familiar with ESD protection? Are you taking measures when you handle ESD sensitive components?
Ioannis
I don't have carpet, and that's the extent of my ESD protections. I think I somehow messed it up when I first powered it up on the breadboard. It's a real possibility that I applied power to the wrong pins.
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Originally Posted by
richard
... if the pin was tristated then it would
that chip cannot do it, more suitable to task chips have a ODCONx register ...
Like this on a 16F18877?
Code:
ODCONC.6 = 0 ' Permit transmission (source current at will)
hserout [ ...message... ]
while TXSTA.1 = 0 ' Check TRMT bit
wend
ODCONC.6 = 1 ' Deny transmission (sink current only)
Am I supposed to set TRIS as well?
(I remember reading somewhere that USART ignores TRIS, or something...?)
-
Re: Framing error if I disable transmitter after shift register is empty?
Quote:
Like this on a 16F18877?
not really , why would you want to change it back and forth?
is there a pullup for the tx line ?
post entire code snippets are nil value
