Atmega 48 UART Daten empfangen

[Jochen (Gast)]

Hallo, ich habe folgendes Programm geschrieben um per Hyperterminal 
gesendete Daten mit einem Atmega 48 zu empfangen:

.include "m48def.inc"
.org 0x000
       rjmp RESET
.org 0x012
    rjmp USART_RXC

USART_RXC:
    ldi r16, 0b01000000
    out TCCR0A, r16
    reti
reset:
    rcall INIT
  


    sei
main:
    rjmp main



INIT:
  ;Ports setzen
    ;Summer
    ldi r16, 0b11000000
    out DDRD, r16

    


  ;Timer für Summer (aus)
    ldi r16, 0b00000000
    out TCCR0A, r16
    ldi r16, 0b00000010
    out TCCR0B, r16

  ;UART RX 2400 baud, RX INT
    ldi r16, 0b10010000
    sts UCSR0B, r16
    ldi r16, 0b00000110
    sts UCSR0C, r16
    ldi r16, 95
    sts UBRR0L, r16
    ret

Es soll einfach nur ein Summer angehen wenn irgendein Zeichen empfangen 
worden ist. Der Summer funktioniert, daten an den Pc senden funktioniert 
auch (hatte ich mal kurz getestet)
Sieht jemand einen Fehler?

[Jochen (Gast)]

Ach ja: Ich benutze natürlich einen Quarz :)

[Achim M. (minifloat)]

Jochen schrieb:
>   ;UART RX 2400 baud, RX INT
>     ldi r16, 0b10010000
>     sts UCSR0B, r16
>     ldi r16, 0b00000110
>     sts UCSR0C, r16
>     ldi r16, 95
>     sts UBRR0L, r16
>     ret

Kann man die elend langen Bitschreibweise mal durch was klar 
ersichtliches ersetzen(oder gleich verbieten)? (1 << UMSEL)|(1 << 
wasweißich) kann ich viel leichter optisch dekodieren, als das 
Bitgewusel. In ein paar Monaten schon kann die Schreibfaulheit nach 
hinten los gehen, wenn man sich slebst nicht mehr auskennt. Code, der 
selbstdokumentierend ist, braucht nicht viel weitere Erklärungen.
mfg mf

PS: Genau darum muss ich jetzt fragen: Ist RXEN gesetzt?

[Jochen (Gast)]

Ich schreib mal kurz auf was die Bytes machen:
    ldi r16, 0b10010000
    sts UCSR0B, r16
RXCIE0 setzen
RXEN0 setzen


    ldi r16, 0b00000110
    sts UCSR0C, r16
UCSZ01 setzen
UCSZ00 setzen

[Jochen (Gast)]

Noch ein Nachtrag. Ich benutze einen USB Seriell Wandler mit CP2102. Der 
funktioniert auch, TX und RX verbinden führt zu einem Echo

[Karl H. (kbuchegg)]

Jochen schrieb:
> Ich schreib mal kurz auf was die Bytes machen:
>     ldi r16, 0b10010000
>     sts UCSR0B, r16
> RXCIE0 setzen
> RXEN0 setzen

UCSR0B wird laut Datenblatt mittels out angesprochen und nicht mit sts

Und warum schreibst du dann nicht gleich

     ldi r16, (1<<RXCIE0) | (1<<RXEN0)

dann brauchst du das keinem erklären und auch keinen Kommentar

>     ldi r16, 0b00000110
>     sts UCSR0C, r16
> UCSZ01 setzen
> UCSZ00 setzen


AUch UCSR0C wird laut Datenblatt mit out angesprochen.

Das ist NICHT egal, weil die Adresslage eine andere ist, je nachdem ob 
du einen out oder einen sts benutzt. Im Beschreibungsteil des 
Datenblattes sind immer wieder Codeschnipsel enthalten, aus denen 
hervorgeht, welchen Befehl du benutzen musst.

[Jochen (Gast)]

Hallo Karl Heinz. Also wenn ich das mit out schreibe kriege ich immer 
einen Fehler "Operand out of Range"
Vielleicht gibts da Unterschiede zwischen den 3 Atmega (48/88/168) ?

[Karl H. (kbuchegg)]

Jochen schrieb:
> Hallo Karl Heinz. Also wenn ich das mit out schreibe kriege ich immer
> einen Fehler "Operand out of Range"
> Vielleicht gibts da Unterschiede zwischen den 3 Atmega (48/88/168) ?

Du hast recht.
Hab grade den Abschnitt 'Register Summary' studiert. Das scheint sich 
tatsächlich um einen Fehler im UART Abschnitt im Datenblatt zu handeln. 
Die Register sind allesamt nicht per out zu erreichen. Da war ich zu 
voreilig und habe dem Code im Abschnitt vertraut.

[Spess53 (Gast)]

Hi

>Vielleicht gibts da Unterschiede zwischen den 3 Atmega (48/88/168) ?

Gibt es nicht.

MfG Spess

[Karl H. (kbuchegg)]

Ich konnte im Datenblatt nichts darüber finden, dass der Stackpointer 
automatisch korrekt initialisiert wird. D.h. du solltest das selbst in 
die Hand nahmen.

[Spess53 (Gast)]

Hi

Siehe Datenblatt S.13:

6.6.1 SPH and SPL – Stack Pointer High and Stack Pointer Low Register

MfG Spess

[Stefan E. (sternst)]

Karl Heinz Buchegger schrieb:
> Ich konnte im Datenblatt nichts darüber finden, dass der Stackpointer
> automatisch korrekt initialisiert wird.

"Initial Value" unter 6.5.1.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Aargh, wenn du im Interrupt R16 benutzt, muss das vorher gerettet 
werden.
Normalerweise beginnt eine ISR mit:

; sichererplatz ist ein beliebiges, sonst unbenutztes register
       mov sichererplatz, SREG
       push   r16
; nun mache ich mit r16 rum
; und am ende:
      pop     r16
      mov     SREG,sichererplatz
      reti

Da deine Hauptschleife noch nix tut, ist das im Moment noch nicht 
wichtig. Aber später wunderst du dich.

[Karl H. (kbuchegg)]

@Spess + @Stefan

Hä?  Hab ich das falsche Datenblatt oder ihr?
Ich verwende das da

http://www.atmel.com/images/doc2545.pdf


Abschnitt 7.6

This Stack space in the data SRAM must be defined by the program
before any subroutine calls are executed or interrupts are enabled.
The Stack Pointer must be set to point above 0x0100, preferably RAMEND.

[Jochen (Gast)]

So, Stackpointer ergänzt. Außerdem habe ich r16 sowie sreg in der 
Interruptroutine gesichet und wieder hergestellt.
Funktioniert immer noch nicht.

[Jochen (Gast)]

Fehler gefunden. Ich habe nur 2 USB Ports und habe daher mein AVR ISP 
MKII herausgezogen und den USB seriell Wandler eingesteckt. Aber 
anscheinend hält der stromlose Programmer den Controller im Reset.
ARGH
Vielen Dank für eure Hilfe!

[Stefan E. (sternst)]

Karl Heinz Buchegger schrieb:
> Hä?  Hab ich das falsche Datenblatt oder ihr?
> Ich verwende das da

Und dort unter 7.6.1 "Initial Value".

Der Text suggeriert in der Tat was anderes. Ist meiner Meinung nach aber 
nur als Hinweis gedacht, wo der Stack-Pointer starten muss, ohne dabei 
zu erwähnen, dass er bereits einen passenden Default-Wert hat.

In "meinem" Datenblatt sieht der Text übrigens so aus:

The Stack in the data SRAM must be defined by the program before any
subroutine calls are executed or interrupts are enabled. Initial Stack
Pointer value equals the last address of the internal SRAM and the Stack
Pointer must be set to point above start of the SRAM, see Table 7-3 on
page 18.

[Karl H. (kbuchegg)]

Stefan Ernst schrieb:
> Karl Heinz Buchegger schrieb:
>> Hä?  Hab ich das falsche Datenblatt oder ihr?
>> Ich verwende das da
>
> Und dort unter 7.6.1 "Initial Value".

Hmm.
Anscheinend gibt es da mehrere Versionen. Goggle hat mir diesen Hit als 
ersten Treffer bei Atmel rausgeworfen

7.6.1 ist bei mir
7.6.1 SPH and SPL – Stack pointer high and stack pointer low register

Und Initial Values werden da keine erwähnt.

[Thomas E. (thomase)]

Karl Heinz Buchegger schrieb:
> Und Initial Values werden da keine erwähnt.
Steht doch darunter:
Initial Value: RAMEND

mfg.

[Karl H. (kbuchegg)]

Thomas Eckmann schrieb:
> Karl Heinz Buchegger schrieb:
>> Und Initial Values werden da keine erwähnt.
> Steht doch darunter:
> Initial Value: RAMEND
>

Autsch. Du hast recht.
Wenn man mir freundlich mit dem Brett den Hinterkopf tätschelt, dann seh 
ich das sogar :-)
Danke.