Probleme mit Uart (Atmega88)

[horst (Gast)]

Hallo!

Hab mir soeben ein Programm zum Testen für den Uart geschrieben. 
Allerdings will der µC kein Zeichen ausgeben. Ich befürchte schon das 
ich irgendetwas falsch gemacht oder vergessen habe.

Kann mir da jemand helfen?

(Programm im Anhang)

[Karl H. (kbuchegg)]

USART_Init:
  ; Set baud rate
  sts UBRR0H, r17
  sts UBRR0L, r16

Aha. Und welche Werte stehen in R16 bzw. R17 zum Zeitpunkt, an dem du 
die USART_Init aufrufst?

[spess53 (Gast)]

>.def temp    = r16            ; Register für kleinere Arbeiten
>.def zeichen = r17            ; in diesem Register wird das Zeichen an die

>    ldi r16, 'A'

>sts UBRR0H, r17
> sts UBRR0L, r16

Was denn nun? temp/zeichen oder r16/r17?

Entscheide dich für eine Variante. So wirst du früher oder später, 
eher früher, Probleme bekommen.

MfG Spess

[oldmax (Gast)]

Hi
Ich hab dir mal die Initialisierung angehängt, die ich für meinen 
Atmega8 immer nehme. Wichtig ist, das du auch den 16 MHz Takt hast, also 
einen Quarz. Ich kenn den Atmega48 nicht, aber er soll ja zum Atmega8 
kompatibel sein. Die folgenden Zeilen hast du vermutlich wie ich auch 
aus dem Tutorial entnommen.

;***********************************************************************
;* Die folgenden Zeilen definieren Constanten. Diese Werte werden  *
;* nur einmal vom  Compiler zugewiesen und dann nicht mehr verändert.  *
;***********************************************************************
.EQU   F_CPU  = 1600000    ; Systemtakt in Hz
.EQU   BAUD  = 9600      ; Baudrate ; Berechnungen
.EQU   UBRR_VAL  = ((F_CPU+BAUD*8)/(BAUD*16)-1)    ; clever runden
.EQU   BAUD_REAL= (F_CPU/(16*(UBRR_VAL+1)))   ; Reale Baudrate
.EQU   BAUD_ERROR= ((BAUD_REAL*1000)/BAUD-1000)    ; Fehler in Promille 

.if ((BAUD_ERROR>10) || (BAUD_ERROR<-10))    ; max. +/-10 Promille Fehler  
.error "Systematischer Fehler der Baudrate grösser 1 Prozent und damit zu hoch!"
.endif

Du darfst hier nicht davon ausgehen, das du mit
.EQU   F_CPU  = 1600000
den Systemtakt setzt, sondern den mußt du über die Fuses setzen. Aber 
den externen Takt nicht vergessen, sonst ist es Essig mit weiteren 
Zugriffen auf den µC.
Die Initialisierung erfolgt dann bei mir wie folgt:

;---------------- Serielle Schnittstelle parametrieren ------------------
;***********************************************************************
;* Die Parametrierung ist mit dem Koppelpartner abzustimmen.    *
;***********************************************************************
INIT_UART:  
  LDI  Reg_A, HIGH(UBRR_VAL)    ; Baudrate einstellen
  OUT  UBRRH, Reg_A
  LDI  Reg_A, LOW(UBRR_VAL)
  OUT  UBRRL, Reg_A
  ; Frame-Format: 8 Bit
   LDI  Reg_A, (1<<URSEL)|(3<<UCSZ0)
  OUT  UCSRC, Reg_A
   SBI  UCSRB, TXEN    ; TX aktivieren
  SBI  UCSRB, RXCIE                ; Interrupt bei Empfang    
  SBI  UCSRB, RXEN                 ; RX (Empfang) aktivieren
RET

Den Interrupt für den Empfang brauchst du ja noch nicht freigeben, wenn 
du es aber tust, dann muß auch eine ISR existieren, oder zumindest ein 
RETI hinter

.org URXCaddr  JMP int_rxc  ; USART Receive Complete Interrupt Vector Address

Die Senderoutine ist jedenfalls einfach:

;------------- serielle Ausgabe  -----------------------

SerOut:  SBIS  UCSRA,UDRE  ; Warten Freigabe UDR 
  RJMP  SerOut
      OUT  UDR, Send_Byte    ; Register Send_byte enthält die Daten 
RET

Deine Sendesequenz ist

     ldi r16, 'A'
     sts UDR0, r16

Wie du siehst, wartest du nicht ab, bis der Wert vom UART auch als 
"versendet" gemeldet wird. des weiteren ist UDR im EA-Bereich und muß 
mit "Out" angesprochen werden.
Zumindest ist das beim Atmega8 so.
Gruß oldmax

[horst (Gast)]

OK danke soweit...

Ich hab mal das ganze überarbeitet, assemblieren lässt es sich zwar aber 
ich bekomm noch immer keine Zeichen??

Mit dem Beispielprogramm vom Atmega8 (AVR_Tutorial) Funktioniert es.

Aber mit meinem Atmega88 bekomm ich es einfach nicht hin.
Außerdem versteh ich nicht ganz warum es UCSR0A, UCSR0B und UCSR0C gibt, 
kann ich mir da irgendeinen aussuchen?

(Code im Anhang)

[oldmax (Gast)]

Hi
Wie wär es mal mit dem Datenblatt ? Kann ich zwar auch auf machen und 
dir dann alles vorlesen, doch ich glaub, du hast dann bestimmt eine 
weitere Frage und ich muß dann weiterlesen....
Gruß oldmax

[spess53 (Gast)]

Hi

> TX Aktivieren

>    lds temp, UCSR0C,
>    sbrs temp, TXEN0

Da wird nix aktiviert. TXEN0 befindet sich im Register UCSR0B.

>serout:
>    lds temp, UCSR0C
>    sbrs temp, UDRE0

Ähnlichr Fall: UDRE0 ist im Register UCSR0A.

>Außerdem versteh ich nicht ganz warum es UCSR0A, UCSR0B und UCSR0C gibt,
>kann ich mir da irgendeinen aussuchen?

Nein. Die Bits in den Registern UCSR0A, UCSR0B und UCSR0C haben 
unterschiedliche Funktionen.

Tu dir einen Gefallen und lies das Datenblatt. Dort sind sogar Beispiele 
in Assembler drin.

MfG Spess

[Stefan E. (sternst)]

spess53 schrieb:
>> TX Aktivieren
>
>>    lds temp, UCSR0C,
>>    sbrs temp, TXEN0
>
> Da wird nix aktiviert. TXEN0 befindet sich im Register UCSR0B.

Und selbst wenn es das richtige Register wäre ...

sbrs = Skip if Bit in Register is Set
und nicht etwa "Set Bit in Register"
(außerdem würde in dem Fall dann auch immer noch das Zurückschreiben 
fehlen)

[spess53 (Gast)]

Hi

>Und selbst wenn es die richtigen Register wären ...

Ich weiss. Aber der TO hat primär erst einmal ein absolutes 
Verständnisproblem mit der Funktion der einzelnen Register und der darin 
enthaltenen Bits.

MfG Spess

[horst (Gast)]

Als erstes habe ich versucht das Programm von Atmega8 zu ändern.. ohne 
Erfolg
Dann habe ich einige Codebeispiele aus google zusammengestöbseln.. ohne 
Erfolg

Jetzt habe ich mir das Datenblatt angesehen (etwas spät aber doch) und 
auch die Beispielcodes angepasst, da ich ja IN/OUT nicht verwenden kann. 
Warum auch immer das im Datenblatt so geschrieben ist.

USART_Transmit:
    ldi temp, UCSR0A
    sbis temp, UDRE0
    rjmp USART_Transmit
    sts UDR0, temp

Bei sbis bekomm ich einen I/O Fehler, warum?
Ich kann mit sbis im register UCSR0A nicht direkt kontrollieren ob das 
bit 5 (udre0) gesetzt ist. Deshalb auch der umweg über ein Register.

Keine Ahnung aber irgendwie steh ich total auf der Leitung, kann mir 
jemand helfen?

Wenn ich es mit sbrs versuche funktioniert das assemblieren, aber am pc 
tut sich nix?

[spess53 (Gast)]

Hi

>Bei sbis bekomm ich einen I/O Fehler, warum?

'sbis' kann nur auf IO-Register zugreifen. 'temp' ist aber ein Register 
aus r0...r31, auf das man mit 'SBRS' zugreift.

MfG Spess

[horst (Gast)]

spess53 schrieb:
> Ich weiss. Aber der TO hat primär erst einmal ein absolutes
> Verständnisproblem mit der Funktion der einzelnen Register und der darin
> enthaltenen Bits.

Stimmt allerdings, da ich beim Atmega8 einfach die im AVR-Tutorial 
verwendeten Codeabschnitte mit den Registern abgeschrieben habe.

Da muss ich mir wohl das Datenblatt des Atmega88 genau durchlesen. Aber 
dennoch verstehe ich nicht warum die Codebeispiele mit IN und OUT 
angegeben sind?

[spess53 (Gast)]

Hi

>dennoch verstehe ich nicht warum die Codebeispiele mit IN und OUT
>angegeben sind?

Du hast recht, die sind falsch. Ich nehme an, die sind etwas gedankenlos 
von älteren Datenblättern übernommen worden.

Bei älteren oder kleineren AVRs liegen die (meisten) IO-Register in dem 
Bereich, den Befehle in, out und eingeschränkt cbi, sbi, sbis und sbic 
adressieren können.

Im Datenblatt deines AVRs gibt es ein 'Register Summary'. Dort sind die 
Adressen der IO-Register enthalten. Bei Fehlermeldungen kannst du da 
nachsehen ob der Befehl zu dem benutzten Register passt.

MfG Spess

[horst (Gast)]

spess53 schrieb:
> Im Datenblatt deines AVRs gibt es ein 'Register Summary'. Dort sind die
> Adressen der IO-Register enthalten. Bei Fehlermeldungen kannst du da
> nachsehen ob der Befehl zu dem benutzten Register passt.

Danke, das wusste ich auch nicht, aber sehr hilfreich!

Hast du meinen letzten Code angeguckt und irgendwas auffälliges gefunden 
warum mein Code noch immer nicht funktioniert?

[spess53 (Gast)]

Hi

>Hast du meinen letzten Code angeguckt

Bis eben nicht

>und irgendwas auffälliges gefunden
>warum mein Code noch immer nicht funktioniert?

Was mir sofort auffällt, da (<<<<<<<) fehlt  ein 'ret'

>    rjmp USART_Transmit
>    sts UDR0, temp
     ret                 <<<<<<<<<

>ende:
>    rjmp ende

MfG Spess

[Stefan E. (sternst)]

    ldi temp, 'T'
    rcall USART_Transmit
...
USART_Transmit:
    ldi temp, UCSR0A
    sbis temp, UDRE0
    rjmp USART_Transmit
    sts UDR0, temp

Es könnte von Vorteil sein, wenn man das Register mit dem zu sendenden 
Zeichen nicht vor dem eigentlichen Senden für was anderes benutzt.

[horst (Gast)]

Super, Danke!

Peinlicher Fehler

[horst (Gast)]

Hier das aktuelle Programm.
Bis jetzt noch immer kein Erfolg :(

Fuses geprüft.. einfach alles. Es kommt nicht einmal Datenmüll :/

[spess53 (Gast)]

Hi

>USART_Transmit:
    ldi temp2, UCSR0A    !!!!!!!!!!!!!!!!!
    sbrs temp2, UDRE0
    rjmp USART_Transmit
    sts UDR0, temp
    ret

Du willst den Inhalt von UCSR0A, nicht die Adresse.

->     lds temp2, UCSR0A
       ^^^

MfG spess

[horst (Gast)]

Sehr Gut!
Jetzt bekomm ich Datenmüll =)

Aber es ist schon ein Anfang...

Ich habe lt. Engbedded AVR Fuse Calculator folgende Fuse Einstellungen 
vorgenommen: -U lfuse:w:0x7f:m -U hfuse:w:0xdf:m -U efuse:w:0x01:m 
(letzte Quarzauswahl in der Liste)

Die Baudrate beträgt am µC und PC 9600
Der Quarz und Takt 8Mhz

[horst (Gast)]

Meine Aktuelles Programm wieder im Anhang

Scheinbar werden alle Zeichen missintrerpretiert, da nicht einmal der 
Zeilenumbruch als solches erkannt wird.

Die Fuses Einstellungen müssten stimmen oder?

[Stefan E. (sternst)]

horst schrieb:
> Die Fuses Einstellungen müssten stimmen oder?

Nein. CKDIV8!

[horst (Gast)]

Juhu!!

Vielen Dank an alle! Jetzt funktioniert es!

Verwendete Fuses: -U lfuse:w:0xff:m -U hfuse:w:0xdf:m -U efuse:w:0x01:m