dsPIC33E - UART

[Flo (Gast)]

Hallo zusammen,

Ich habe ein kleines Problem. Und zwar muss ich einen dsPIC 
programmieren.
Erste Aufgaeb dabei ist eine Kommunikation zwischen µC und PC per UART 
herzustellen.
Ich hab keine Erfahrungen mit dem dsPIC und mit allg. µC immernoch 
Anfänger.

Nun zum Problem:
Eine Verbindung besteht. Ich kann vom PC ein Zeichen senden, dieses wird 
am Controller empfangen und darauf eine Zeichenfolge zurückgesendet.
Dies lässt sich Anhnd von LEDs und dem Terminal feststellen. Das Problem 
ist, dass nicht das ankommt was gesendet wurde sondern völliges Zeichen 
wirrwarr.

Vielleicht kann mir ja hier jemand helfen?

Meine Vermutung ist die Baudrate. Dem PC glaube ich mal das es dort 
stimmt (auf 115200 eingestellt).

Danke im vorraus,

Flo

[Florian T. (florian_t91)]

Ich hab vergessen zu erwähnen:

dsPIC33EP512MU810
8Mhz Quarz
mikromedia workstation 7

Gruß Flo

[Hermann U. (Firma: www.pcb-devboards.de) (gera82)]

Hallo,


Ich habe dein Code nicht angeschaut, ich tippe mal esliget an dem Takt 
von dsPIC und/oder falscher BRGVAL Einstellung.

Florian T. schrieb:
> 8Mhz Quarz
Das sagt nicht, wie schnell läuft das teil?


Hier mal meine kleiner Ausschnitt (dsPIC33EP512MU814), was ohne Problem 
funtzt, die wichtigsten Stellen habe ich mal raus kopiert:

#define  SYS_FREQ     120000000UL     // Mikrocontroller Takt 120Mhz = 60Mips
#define  FCY          (SYS_FREQ/2)  // instruction cycle time

#define BAUDRATE         115200UL     
#define BRGVAL           ((FCY / BAUDRATE)/ 16UL) - 1UL

/********************************************************************************
 *                                                                              *
 *  Funktion:      OSCswitching                                                 * 
 *                                                                              *
 *------------------------------------------------------------------------------*
 *  Description: 
 ********************************************************************************/
void OSCswitching(void) 
{
   
    /*
     * Configure oscillator to operate the device at Fcy = 60Mhz
     * Fosc = Fin*M/(N1*N2)
     * Fcy = Fosc/2
     * Fosc = 10Mhz * 48/(2 * 2) = 120MHz for 10MHz input clock
     * 60Mips
     *
     */
 PLLFBD = 46;    /* M = 48 */
 CLKDIVbits.PLLPOST = 0;  /* N1 = 2 */
 CLKDIVbits.PLLPRE = 0;  /* N2 = 2 */

 CLKDIVbits.DOZEN = 0;  /* Processor clock and peripheral clock ratio forced to 1:1 */

 RCONbits.SWDTEN = 0; //Disable Watch Dog Timer

 __builtin_write_OSCCONH(0b011);  //Initiate clock switch to HS with PLL

 __builtin_write_OSCCONL(0b001);  //Start clock switching

 while (OSCCONbits.COSC != 0b011) //Wait for clock switch to occur

 while(OSCCONbits.LOCK!=1) {};    //Wait for PLL to lock
}

Hier mal die UART Einstellung:

   U3MODEbits.USIDL = 0;        /* Continue in Idle mode */
   U3MODEbits.LPBACK = 0;        /* Disable LoopBack */
    U3MODEbits.PDSEL = 0b00;      /* 8-bit data, no parity */
    U3MODEbits.STSEL = 0;        /* One Stop bit */
                       
    U3BRG = BRGVAL;              /* Baud rate generator value*/

    IFS5bits.U3RXIF = 0;        /* Clear interrupt flag */
    IPC20bits.U3RXIP = 1;        /* Set to highest priority */
    IEC5bits.U3RXIE = 1;        /* Enable receive interrupts */

    
    U3MODEbits.UARTEN = 1;              /* Enable UART module 1 */
    U3STAbits.UTXEN = 1;              /* Enable transmission, overrides TRIS settings */

Gruß
Hermann

[Florian T. (florian_t91)]

Wie schnell er läuft? Das würde ich auch gern wissen.

Quarz hat 8Mhz, Instruction Cycle Time ist (glaube ich) 2.
Meines Erachtens habe ich auf den extern Taktquelle geschaltet.

=> µC Takt: 4Mhz

Oder habe ich da falsch gedacht?

[Hermann U. (Firma: www.pcb-devboards.de) (gera82)]

Florian T. schrieb:
> Wie schnell er läuft? Das würde ich auch gern wissen.

Oszi dran dann weist du es;-)
Oder einfach mal Pin toggeln
Z.B. so, mit Oszi oder LED anklemmen:

PIN = high
delay_s(1)
PIN = low

Wenn PIN zu schnell oder zu langsam toggelt, dann weißt du das da was 
nicht stimmt ;-)

#define delay_ns(d) { __delay32( (unsigned long) ((d)*(FCY)/1000000000UL)); }
#define delay_us(d) { __delay32( (unsigned long) ((d)*(FCY)/1000000UL)); }
#define delay_ms(d) { __delay32( (unsigned long) ((d)*(FCY)/1000UL)); }
#define delay_s(d)  { __delay32( (unsigned long) ((d)*(FCY)/1UL)); }

Mach mal folgendes, setze void OSCswitching(void) bei dir 1:1 ein, dann 
solltest du 96Mhz haben. Dann füge diesen abschnitt bei dir:

#define  SYS_FREQ     96000000UL     // Mikrocontroller Takt 96Mhz = 48Mips
#define  FCY          (SYS_FREQ/2)  // instruction cycle time

#define BAUDRATE         115200UL    //Baudrate bei dir anpassen!!! 
#define BRGVAL           ((FCY / BAUDRATE)/ 16UL) - 1UL

UxBRG = BRGVAL;              /* Baud rate generator value*/



Gruß
Hermann

[Hermann (Gast)]

Florian T. schrieb:
> Quarz hat 8Mhz, Instruction Cycle Time ist (glaube ich) 2.
> Meines Erachtens habe ich auf den extern Taktquelle geschaltet.
>
> => µC Takt: 4Mhz
Dann solltest du auch richtig einsetzten:
#define F_CPU ( 8000000 / 2 )  // Und nicht 4, wie bei dir
#define UBRG(x) ((( F_CPU / x ) / 16 ) - 1 )

[Florian T. (florian_t91)]

Hermann danke schonmal für die Anworten und Tipps.

Der delay_test.c war erfolgreich. LED war 2s an (soweit ich das mit 
meiner Handystoppuhr sagen kann).
uart_test.c hat trotzdem nicht funktioniert. Immernoch undefinierte 
Zeichen.

Statt dem String "test" kommen folgende Hexwerte:

B1 2B B4 C8 F8 00

Die '00' sind denk ich i.O.. Bei den anderem Werten varriert es zum Teil 
auch ein wenig ( 91 2B B4 CC F8 00 ).

Als ich dein OSCswitching() einfügen wollte ging gar nichts mehr.

Vielleicht weitere Tipps?

Flo


EDIT:
Das mit meinen falsch angegeben F_CPU: ich hatte verschiedene 
Einstellungen versucht und doch die falsch kopiert ;)

[Patrick B. (p51d)]

wie sieht die Funktion 'putsUART1()' aus? Was macht die?

Ich tippe hier auf falsch eingestelltes Terminal oder falscher Clock, da 
der String "test" ja aus 4 Zeichen besteht und du 6 erhälst, die auch 
variiren...

[EDIT]:
Ansonsten versuchs mal hiermit (von Microchip code example Seite für 
dsPIC33E/F). Wenn dies auch nicht funktioniert mit den entsprechenden 
einstellungen hast du ein anderes Problem:

/**********************************************************************
* © 2006 Microchip Technology Inc.
*
* FileName:        main.c
* Dependencies:    Header (.h) files if applicable, see below
* Processor:       dsPIC33FJ256GP710
* Compiler:        MPLAB® C30 v3.00 or higher
* Tested On:     dsPIC33FJ256GP710 HP16
*
* SOFTWARE LICENSE AGREEMENT:
* Microchip Technology Incorporated ("Microchip") retains all ownership and 
* intellectual property rights in the code accompanying this message and in all 
* derivatives hereto.  You may use this code, and any derivatives created by 
* any person or entity by or on your behalf, exclusively with Microchip's
* proprietary products.  Your acceptance and/or use of this code constitutes 
* agreement to the terms and conditions of this notice.
*
* CODE ACCOMPANYING THIS MESSAGE IS SUPPLIED BY MICROCHIP "AS IS".  NO 
* WARRANTIES, WHETHER EXPRESS, IMPLIED OR STATUTORY, INCLUDING, BUT NOT LIMITED 
* TO, IMPLIED WARRANTIES OF NON-INFRINGEMENT, MERCHANTABILITY AND FITNESS FOR A 
* PARTICULAR PURPOSE APPLY TO THIS CODE, ITS INTERACTION WITH MICROCHIP'S 
* PRODUCTS, COMBINATION WITH ANY OTHER PRODUCTS, OR USE IN ANY APPLICATION. 
*
* YOU ACKNOWLEDGE AND AGREE THAT, IN NO EVENT, SHALL MICROCHIP BE LIABLE, WHETHER 
* IN CONTRACT, WARRANTY, TORT (INCLUDING NEGLIGENCE OR BREACH OF STATUTORY DUTY), 
* STRICT LIABILITY, INDEMNITY, CONTRIBUTION, OR OTHERWISE, FOR ANY INDIRECT, SPECIAL, 
* PUNITIVE, EXEMPLARY, INCIDENTAL OR CONSEQUENTIAL LOSS, DAMAGE, FOR COST OR EXPENSE OF 
* ANY KIND WHATSOEVER RELATED TO THE CODE, HOWSOEVER CAUSED, EVEN IF MICROCHIP HAS BEEN 
* ADVISED OF THE POSSIBILITY OR THE DAMAGES ARE FORESEEABLE.  TO THE FULLEST EXTENT 
* ALLOWABLE BY LAW, MICROCHIP'S TOTAL LIABILITY ON ALL CLAIMS IN ANY WAY RELATED TO 
* THIS CODE, SHALL NOT EXCEED THE PRICE YOU PAID DIRECTLY TO MICROCHIP SPECIFICALLY TO 
* HAVE THIS CODE DEVELOPED.
*
* You agree that you are solely responsible for testing the code and 
* determining its suitability.  Microchip has no obligation to modify, test, 
* certify, or support the code.
*
* REVISION HISTORY:
*~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
* Author            Date      Comments on this revision
*~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
* Rob Welch         11/28/06    Original Release
*
*~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
* ADDITIONAL NOTES: 
* Small, bare bones program to guide designers with UART implementation
*  To run program, plug DB9 cable to Terminal (9600,8,N,1) - 
*  Data sent from TERMINAL END will be represented as binary equivalent
*  of ASCII on 7 LSB LEDs of HPC16 Explorer board
*
*  Data is sent from HPC16 by pressing S3 - S6.  A single letter will
*  be transmitted upon each keypress.
*
* Peripheral Library was not utilized, Bit Addressing was used in order
* to show all details involved in initialization of UART.
* 
* 
* The Processor starts with the Internal oscillator without PLL enabled and then the Clock is switched to PLL Mode.
*
**********************************************************************/
#include <p33FJ256GP710.h>

_FOSCSEL(FNOSC_FRC);      // Internal FRC oscillator
_FOSC(FCKSM_CSECMD & OSCIOFNC_OFF  & POSCMD_NONE);  
                // Clock Switching is enabled and Fail Safe Clock Monitor is disabled
                // OSC2 Pin Function: OSC2 is Clock Output
                // Primary Oscillator Mode: XT Crystanl


#define TRUE  1
#define FALSE  0

unsigned char S3Flag, S4Flag, S5Flag, S6Flag;

void __attribute__ ((interrupt, no_auto_psv)) _U2RXInterrupt(void) {
  LATA = U2RXREG;
  IFS1bits.U2RXIF = 0;
}
void __attribute__ ((interrupt, no_auto_psv)) _U2TXInterrupt(void) {
  IFS1bits.U2TXIF = 0;
}

void InitClock() {
  PLLFBD = 38;  // M = 40
  CLKDIVbits.PLLPOST = 0;  // N1 = 2
  CLKDIVbits.PLLPRE = 0;  // N2 = 2
  OSCTUN = 0;
  RCONbits.SWDTEN = 0;

// Clock switch to incorporate PLL
  __builtin_write_OSCCONH(0x01);    // Initiate Clock Switch to 
                          // FRC with PLL (NOSC=0b001)
  __builtin_write_OSCCONL(0x01);    // Start clock switching
  while (OSCCONbits.COSC != 0b001);  // Wait for Clock switch to occur  

  while(OSCCONbits.LOCK != 1) {};
}

void InitUART2() {
  // This is an EXAMPLE, so brutal typing goes into explaining all bit sets

  // The HPC16 board has a DB9 connector wired to UART2, so we will
  // be configuring this port only

  // configure U2MODE
  U2MODEbits.UARTEN = 0;  // Bit15 TX, RX DISABLED, ENABLE at end of func
  //U2MODEbits.notimplemented;  // Bit14
  U2MODEbits.USIDL = 0;  // Bit13 Continue in Idle
  U2MODEbits.IREN = 0;  // Bit12 No IR translation
  U2MODEbits.RTSMD = 0;  // Bit11 Simplex Mode
  //U2MODEbits.notimplemented;  // Bit10
  U2MODEbits.UEN = 0;    // Bits8,9 TX,RX enabled, CTS,RTS not
  U2MODEbits.WAKE = 0;  // Bit7 No Wake up (since we don't sleep here)
  U2MODEbits.LPBACK = 0;  // Bit6 No Loop Back
  U2MODEbits.ABAUD = 0;  // Bit5 No Autobaud (would require sending '55')
  U2MODEbits.URXINV = 0;  // Bit4 IdleState = 1  (for dsPIC)
  U2MODEbits.BRGH = 0;  // Bit3 16 clocks per bit period
  U2MODEbits.PDSEL = 0;  // Bits1,2 8bit, No Parity
  U2MODEbits.STSEL = 0;  // Bit0 One Stop Bit
  
  // Load a value into Baud Rate Generator.  Example is for 9600.
  // See section 19.3.1 of datasheet.
  //  U2BRG = (Fcy/(16*BaudRate))-1
  //  U2BRG = (37M/(16*9600))-1
  //  U2BRG = 240
  U2BRG = 240;  // 40Mhz osc, 9600 Baud

  // Load all values in for U1STA SFR
  U2STAbits.UTXISEL1 = 0;  //Bit15 Int when Char is transferred (1/2 config!)
  U2STAbits.UTXINV = 0;  //Bit14 N/A, IRDA config
  U2STAbits.UTXISEL0 = 0;  //Bit13 Other half of Bit15
  //U2STAbits.notimplemented = 0;  //Bit12
  U2STAbits.UTXBRK = 0;  //Bit11 Disabled
  U2STAbits.UTXEN = 0;  //Bit10 TX pins controlled by periph
  U2STAbits.UTXBF = 0;  //Bit9 *Read Only Bit*
  U2STAbits.TRMT = 0;  //Bit8 *Read Only bit*
  U2STAbits.URXISEL = 0;  //Bits6,7 Int. on character recieved
  U2STAbits.ADDEN = 0;  //Bit5 Address Detect Disabled
  U2STAbits.RIDLE = 0;  //Bit4 *Read Only Bit*
  U2STAbits.PERR = 0;    //Bit3 *Read Only Bit*
  U2STAbits.FERR = 0;    //Bit2 *Read Only Bit*
  U2STAbits.OERR = 0;    //Bit1 *Read Only Bit*
  U2STAbits.URXDA = 0;  //Bit0 *Read Only Bit*

  IPC7 = 0x4400;  // Mid Range Interrupt Priority level, no urgent reason

  IFS1bits.U2TXIF = 0;  // Clear the Transmit Interrupt Flag
  IEC1bits.U2TXIE = 1;  // Enable Transmit Interrupts
  IFS1bits.U2RXIF = 0;  // Clear the Recieve Interrupt Flag
  IEC1bits.U2RXIE = 1;  // Enable Recieve Interrupts

  U2MODEbits.UARTEN = 1;  // And turn the peripheral on

  U2STAbits.UTXEN = 1;
  // I think I have the thing working now.
}

void InitPorts() {
  // S3 (portD Pin 6, chosen as trigger for sending 'M' to UART)
  // S6 (portD Pin 7, chosen as trigger for sending 'C' to UART)
  // S5 (portA Pin 7, chosen as trigger for sending 'H' to UART)
  // S4 (portD Pin 13, chosen as trigger for sending 'P' to UART)

  TRISD = 0x20C0;  // D6,7,13 inputs
  AD1PCFGHbits.PCFG23 = 1;  // This is important.  RA7 is muxxed with AN23,
    // So we need to config the pin as DIGITAL
  TRISA = 0x0080;  // only 0th bit needs be output. A7 is input
  S3Flag = S4Flag = S5Flag = S6Flag = 0;  // Some Debounce Flags
}

void SoftwareDebounce() {
  if(PORTDbits.RD6 == FALSE) {
    if( S3Flag == FALSE ) {
      S3Flag = TRUE;
      U2TXREG = 'M';
    }
  }
  else {
    S3Flag = FALSE;
  }
  if(PORTDbits.RD7 == FALSE) {
    if( S6Flag == FALSE ) {
      S6Flag = TRUE;
      U2TXREG = 'C';
    }
  }
  else {
    S6Flag = FALSE;
  }
  if(PORTAbits.RA7 == FALSE) {
    if( S5Flag == FALSE ) {
      S5Flag = TRUE;
      U2TXREG = 'H';
    }
  }
  else {
    S5Flag = FALSE;
  }
  if(PORTDbits.RD13 == FALSE) {
    if( S4Flag == FALSE ) {
      S4Flag = TRUE;
      U2TXREG = 'P';
    }
  }
  else {
    S4Flag = FALSE;
  }
}
int main(void) {
  
  InitClock();  // This is the PLL settings

  InitUART2();  // Initialize UART2 for 9600,8,N,1 TX/RX

  InitPorts();  // LEDs outputs, Switches Inputs

  while(1) {  // The ever versatile Infinite Loop!
    SoftwareDebounce();
  }
}

[Florian T. (florian_t91)]

putsUART1 ist eine Funktion aus der Bibliothek <uart.h> des xc16 
Compilers

Ich hoffe doch mal, dass ich die so einfach verwenden kann.
Naja wenn ich am Terminal einfach mal einseitig die Baudrate ändere 
bekomme ich auch unterschiedlich lange Antworten.
Aber danke für deine Antwort.

Flo

Edit:
Ich hatte mir das Beispiel auch schon angeschaut und dachte ich hätte 
alles beachtet. Außerdem würde ich doch gerne die lib verwenden.  Warum 
das Rad neu erfinden?

[Florian T. (florian_t91)]

Ich hab jetzt mal das Beispiel (das gibts auch passend für meinen µC) 
probiert und an den nötigen Stellen angepasst (Anschlussbelegung UART, 
FOSC, etc). Tatsächlich es funktioniert.
Abgesehen vom 'H', das mag nicht. Also gar nicht. Schätze das liegt aber 
an meinem Board und müsste mit entsprechenden Anpassungen auch klappen.

Ich versuch jetzt mal das Beispiel mit meinem ersten Code zu "paaren", 
vielleicht klappts.

Falls noch wer Vorschläge hat wo mein Fehler liegen könnte, immer her 
damit.

Dank an alle,

Flo

EDIT:
Mir ist grad noch aufgefallen: Hab mein 8Mhz Wert zur Berechnung der 
BdRate genommen obwohl das Beispiel den internen FRC verwendet. Und 
trotzdem funktionierts?

[Florian T. (florian_t91)]

Ok. Meine letzte Frage kann ich mir selbst beantworten. Hab fehler mit 
Fehler geschlagen.
Der interne FRC hat nominal 7.37Mhz. Zur Berechnung von FOSC hab ich für 
M 38 statt 40 genommen. Hat dann wohl zufällig noch gepasst.

FOSC = ( FIN * M ) / ( N1 * N2 )
Baudrate gewünscht 9600

alt: FIN = 8Mhz, M = 38     =>    FOSC = 76 000 000    => BRGVAL = 246
neu: FIN = 7.37Mhz, M = 40  =>    FOSC = 73 700 000    => BRGVAL = 238

[Hermann U. (Firma: www.pcb-devboards.de) (gera82)]

Patrick B. schrieb:
> Ich tippe hier auf falsch eingestelltes Terminal oder falscher Clock, da
> der String "test" ja aus 4 Zeichen besteht und du 6 erhälst, die auch
> variiren...

Florian T. schrieb:
> Abgesehen vom 'H', das mag nicht. Also gar nicht. Schätze das liegt aber
> an meinem Board und müsste mit entsprechenden Anpassungen auch klappen.

Besorg dir mal ein Oszilloskop, dann gehts auch schneller voran.
Damit kannst die Fehler schneller einkreisen.


Gruß
Hermann

[Florian T. (florian_t91)]

Ja. Also ein eigenes hab ich nicht und kann ich mir auch nicht leisten. 
Ich hätt hier zwar eventuell Zugriff auf eines aber ich wüsst auch gar 
nicht wo ich da was abgreifen soll 
(http://www.mikroe.com/mikromedia/dspic33ep/ mit der workstation 7).

Ich habs jetzt mit angehängtem Code zum laufen gebracht. Ist ein Mix aus 
allem hier angebotenen inklusive meiner Lösung.

Musste den internern FRC verwenden. Beim Versuch am Anfang des Programs 
auf das externe Quarz mit PLL zu wechseln hat nichts funktioniert.

Was die Geschwindigkeit angeht dürfte es da kein Problem geben. Ob 8MHz 
oder den 7.37MHz ist Wurst. Solange ich sauber per UART kommunizieren 
kann ist die Genauigkeit auch egal.

Danke an alle die geantwortet haben

Flo

PS: Das nächste Problem wartet bestimmt schon :D