Umstig von PN auf VAR Studio - Baudrate Fehler

Matthias R. (matsch) OP 05.11.2010 15:57 #1925494

Lesenswert?

•

Moin und Hallo,
ich habe ein Projekt bisher immer mit 'Programmers Notepad' (PN) 
bearbeitet, und aus dem PN auch die Hex Files erzeugt (Tool/[WinAVR] 
Make All).

Nun habe ich das gleiche Projekt in AVR Studio 4 angelegt. Unter den 
Projekt Options habe ich meine F_CPU auf 11059200 Hz eingestellt.

Die Baudrate berechne ich wie im Tutorial beschrieben:

#ifndef F_CPU
   #define F_CPU 11059200L    // Systemtakt in Hz, das L am Ende ist wichtig, NICHT UL verwenden!
#define UART_BAUD_RATE 19200L 
#define UBRR_VAL (F_CPU+UART_BAUD_RATE*8)/(UART_BAUD_RATE*16)-1)    // clever runden
#define BAUD_REAL (F_CPU/(16*(UBRR_VAL+1)))                  // Reale Baudrate
#define BAUD_ERROR ((BAUD_REAL*1000)/UART_BAUD_RATE-1000)        // Fehler in Promille
#if ((BAUD_ERROR>10) || (BAUD_ERROR<-10))
   #error Systematischer Fehler der Baudrate grösser 1% und damit zu hoch! 

Wenn ich nun ein Build erstellen möchte, bekomme ich diesen Fehler:

../src/prjdef.c:267:41: warning: the right operand of "<" changes sign when promoted
../src/prjdef.c:268:4: error: #error Systematischer Fehler der Baudrate grösser 1% und damit zu hoch!
make: *** [prjdef.o] Error 1
Build failed with 1 errors and 1 warnings...

Der Fehler zeigt auf die Baudrate Berechnung.

Der Fehler ist beim Kompilieren über PN nicht aufgetreten. Warum meldet 
das AVR Studio diesen Fehler, und wie bekomme ich ihn weg?
Hat jemand einen Tipp?
Gruß
matsch

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

Matthias R. (matsch) OP 08.11.2010 14:16 #1928847

Lesenswert?

•

▲
▼

Hat keiner eine Idee?

Wenn ich die Fehlerabfrage auskommentiere läuft das Programm 
einwandfrei. Das kann doch aber nicht der Sinn sein, oder?

Gruß
Matsch

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Hc Z. (mizch) 08.11.2010 14:32 #1928884

Lesenswert?

•

▲
▼

Hier

1	`#define UBRR_VAL (F_CPU+UART_BAUD_RATE8)/(UART_BAUD_RATE16)-1)`

fehlt eine „(“.  Das müsste aber später eine andere Fehlermeldung geben, 
womit sich die Frage stellt, ob Du uns wirklich Deinen aktuellen Code 
gezeigt hast.  Bitte immer Copy&Paste verwenden, sonst werden Mitleser 
potentiell in die Irre geführt und suchen sich einen Wolf.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Matthias R. (matsch) OP 08.11.2010 14:44 #1928900

Lesenswert?

•

▲
▼

Da hast du recht.
Hier daher noch einmal der Original-copy-and-paste code:

//-----------------------------------------------------------------------------
// UART initialisieren 
// es wird die Library von Peter Fleury benutzt.
//------1---------2---------3---------4---------5---------6---------7---------8 
  #ifndef F_CPU
  /* In neueren Version der WinAVR/Mfile Makefile-Vorlage kann
    F_CPU im Makefile definiert werden, eine nochmalige Definition
    hier wuerde zu einer Compilerwarnung fuehren. Daher "Schutz" durch
    #ifndef/#endif 
    Dieser "Schutz" kann zu Debugsessions führen, wenn AVRStudio 
    verwendet wird und dort eine andere, nicht zur Hardware passende 
    Taktrate eingestellt ist: Dann wird die folgende Definition 
    nicht verwendet, sondern stattdessen der Defaultwert (8 MHz?) 
    von AVRStudio */
    #define F_CPU 11059200L    // Systemtakt in Hz, das L am Ende ist wichtig, NICHT UL verwenden!
//    #define F_CPU  8000000L    // Systemtakt in Hz, das L am Ende ist wichtig, NICHT UL verwenden!
  #endif
  // #define UART_BAUD_RATE 2400L          // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  // #define UART_BAUD_RATE 4800L          // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  // #define UART_BAUD_RATE 9600L          // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  // #define UART_BAUD_RATE 14400L         // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  #define UART_BAUD_RATE 19200L         // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  // #define UART_BAUD_RATE 28800L         // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  // #define UART_BAUD_RATE 38400L         // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  // #define UART_BAUD_RATE 57600L         // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  // #define UART_BAUD_RATE 76800L         // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  // #define UART_BAUD_RATE 115200L        // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  // #define UART_BAUD_RATE 230400L        // Baudrate, das L am Ende ist wichtig, NICHT UL verwenden!
  // Berechnungen
  #define UBRR_VAL ((F_CPU+UART_BAUD_RATE*8)/(UART_BAUD_RATE*16)-1)    // clever runden
  #define BAUD_REAL (F_CPU/(16*(UBRR_VAL+1)))                  // Reale Baudrate
  #define BAUD_ERROR ((BAUD_REAL*1000)/UART_BAUD_RATE-1000)        // Fehler in Promille
  #if ((BAUD_ERROR>10) || (BAUD_ERROR<-10))
    #error Systematischer Fehler der Baudrate grösser 1% und damit zu hoch! 
  #endif  
  uart_init(UART_BAUD_SELECT(UART_BAUD_RATE,F_CPU));

Ich habe beim Kommentare entfernen wohl die Klammer mit reduziert. 
Sorry.
Gruß
Matsch

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Matthias R. (matsch) OP 08.11.2010 14:56 #1928924

Lesenswert?

•

▲
▼

Ich vermute den Fehler auch eher in den Einstellungen vom AVR Studio als 
im Code. Ich finde da nur leider nichts.
AVR Studio: 4.18 Build 700
WinAVR: 20100110
Gruß
Matsch

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Hc Z. (mizch) 08.11.2010 17:38 #1929213

Lesenswert?

•

▲
▼

Obiges Codestück kompiliert ohne Fehler durch.  Es kommt aber Dein 
Fehler, wenn man F_CPU in 11059200UL ändert.  Wetten, Deine 
Kommandozeile enthält -DF_CPU=11059200UL?  Schau mal ins 
Compile-Fenster.

Die Abhilfe hängt davon ab, woher das „UL“ kommt.  Am einfachsten 
änderst Du im Macro selbst F_CPU in (long)F_CPU.

Das Problem ist nämlich, dass BAUD_ERROR durch die Promotionsregeln 
sonst unsigned long wird und deshalb das „-10“ im 
Fehlerüberprüfungs-Makro zu unsigned long promotet wird, zu einer 
riesigen Zahl (ULONG_MAX-9). Und da passt dann (fast) alles drunter.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Matthias R. (matsch) OP 09.11.2010 17:19 #1930601

Lesenswert?

•

▲
▼

Hc Zimmerer schrieb:
> Obiges Codestück kompiliert ohne Fehler durch.  Es kommt aber Dein
> Fehler, wenn man F_CPU in 11059200UL ändert.  Wetten, Deine
> Kommandozeile enthält -DF_CPU=11059200UL?  Schau mal ins
> Compile-Fenster.
>
> Die Abhilfe hängt davon ab, woher das „UL“ kommt.  Am einfachsten
> änderst Du im Macro selbst F_CPU in (long)F_CPU.

Moin,
ich habe mir schon so etwas gedacht.
In der Kommandozeile steht:

avr-gcc -I"D:\Projekte\Test\SoftwareMega16\src"  -mmcu=atmega16 -Wall -gdwarf-2 -std=gnu99                                    -DF_CPU=11059200UL -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -MD -MP -MT main.o -MF dep/main

Da findet sich das -DF_CPU=11059200UL.

In den 'Project Options' (siehe Bild) habe ich die Quarz-Frequenz 
eingestellt. Ein U bzw. ein UL anhängen geht in dem Dialog nicht.

In meinem alten MAKE File stehen folgende Zeilen:

# Processor frequency.
#     This will define a symbol, F_CPU, in all source code files equal to the 
#     processor frequency. You can then use this symbol in your source code to 
#     calculate timings. Do NOT tack on a 'UL' at the end, this will be done
#     automatically to create a 32-bit value in your source code.
#     Typical values are:
#         F_CPU =  1000000
#         F_CPU =  1843200
#         F_CPU =  2000000
#         F_CPU =  3686400
#         F_CPU =  4000000
#         F_CPU =  7372800
#         F_CPU =  8000000
#         F_CPU = 11059200
#         F_CPU = 14745600
#         F_CPU = 16000000
#         F_CPU = 18432000
#         F_CPU = 20000000
F_CPU = 11059200

Binde ich das Makefile ein (gleicher Dialog wie Bild), ist der Fehler 
weg.
Es muss doch aber auch im AVR Studio einzustellen sein, bzw. wo muss ich 
welches Macro ändern?.
Gruß
Matsch

Angehängte Dateien:

ProjectOptions.png (46 KB)

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Hc Z. (mizch) 09.11.2010 18:06 #1930671

Lesenswert?

•

▲
▼

Ich würde hier eingreifen:

#if ((BAUD_ERROR>10) || ((long)BAUD_ERROR<-10))


EDIT: Wenn ich mir's recht überlege, wäre es hier sauberer:

 #define BAUD_ERROR (((long)BAUD_REAL*1000)/UART_BAUD_RATE-1000)        // Fehler in Promille

Das ersetzt den oberen Vorschlag.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Matthias R. (matsch) OP 11.11.2010 12:07 #1933051

Lesenswert?

•

▲
▼

Hallo,
wenn ich das (long) einfüge, bekomme ich folgenden Fehler:

../src/prjdef.c:268:33: error: missing binary operator before token "("

Habe in einem anderen Thread gelesen: "Der Präprozessor kennt weder 
'unsigned char' noch 'double'". Kann das auch für 'long' gelten?

Ich habe jetzt einmal eine andere UART Fehlerberechnung aus einem 
anderen Projekt verwendet:

#define UART_BAUD 19200L
// calculate real baud rate:
#define UBRR_VAL ((F_CPU+UART_BAUD*8)/(UART_BAUD*16)-1)            // round
#define BAUD_REAL (F_CPU/(16*(UBRR_VAL+1)))                    // real baudrate
#define BAUD_ERROR ((BAUD_REAL*1000)/UART_BAUD)                  // error in promille
#if ((BAUD_ERROR < 990) || (BAUD_ERROR > 1010))
#  error Error of baud rate of RS232 UART is more than 1%. That is too high!
  uart_init(UART_BAUD_SELECT(UART_BAUD,F_CPU));

Mit der Berechnung bekomme ich keinen Fehler.

Die Unterschiede sind, soweit ich es erkenne

Fehler: #define BAUD_ERROR ((BAUD_REAL*1000)/UART_BAUD_RATE-1000)
    OK: #define BAUD_ERROR ((BAUD_REAL*1000)/UART_BAUD)

und

Fehler: #if ((BAUD_ERROR>10) || (BAUD_ERROR<-10))
    OK: #if ((BAUD_ERROR < 990) || (BAUD_ERROR > 1010))

Es fehlt das '-1000' und die Abfrage ist eine andere. Mathematisch 
gesehen ist es doch das gleiche, oder?

Wie kann ich mir die Werte von BAUD_ERROR, F_CPU, usw. vom AVR Studio 
anzeigen lassen?

Gruß
Matsch

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Hc Z. (mizch) 11.11.2010 21:14 #1933878

Lesenswert?

•

▲
▼

Matthias R. schrieb:
> wenn ich das (long) einfüge, bekomme ich folgenden 
Fehler:../src/prjdef.c:268:33: error: missing binary operator before token "("

Es passiert selten, aber es passiert: Ich habe übersehen, dass es sich 
um reine Präprozessor-Arithmetik handelt, und die versteht keine casts.

> Die Unterschiede sind, soweit ich es erkenne
> Fehler: #define BAUD_ERROR ((BAUD_REAL*1000)/UART_BAUD_RATE-1000)
>     OK: #define BAUD_ERROR ((BAUD_REAL*1000)/UART_BAUD)
> undFehler: #if ((BAUD_ERROR>10) || (BAUD_ERROR<-10))
>     OK: #if ((BAUD_ERROR < 990) || (BAUD_ERROR > 1010))
> Es fehlt das '-1000' und die Abfrage ist eine andere. Mathematisch
> gesehen ist es doch das gleiche, oder?

In Präprozessor-Mathematik (oder auch C) ist es nicht genau das gleiche: 
Die jeweils zweite Version bleibt im unsigned-long-Zahlenbereich, 
negative Zahlen sind ausgeschlossen.  Deshalb funktioniert sie.

> Wie kann ich mir die Werte von BAUD_ERROR, F_CPU, usw. vom AVR Studio
> anzeigen lassen?

Nur indem Du sie in einer C-Zeile verwendest, z.B.

ultoa(BAUD_ERROR, puffer, 10);

und das dann ausgibst.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Matthias R. (matsch) OP 12.11.2010 09:35 #1934245

Lesenswert?

•

▲
▼

Hc Zimmerer schrieb:
> Nur indem Du sie in einer C-Zeile verwendest, z.B.
>

> ultoa(BAUD_ERROR, puffer, 10);

> und das dann ausgibst.

Manche Dinge sind so einfach, da sieht man den Code vor lauter Zeichen 
nicht ;-)
Danke für die Ausführungen und Hilfe.
Gruß
Matsch

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Umstig von PN auf VAR Studio - Baudrate Fehler

Antwort schreiben