AVR-UART master slave

Mit dem "Multi-processor Communication Mode" wird nur eine Hilfestellung 
zur automatischen Reaktion auf das 8. Bit (TXB8) bereitgestellt.

Wird ein Zeichen empfangen, bei dem der Sender das 8. Bit gesetzt hat, 
dann werten (per von dir zu schreibender Software) die Slaves dieses 
Zeichen als Adresse und vergleichen diese Adresse mit der eigenen.
Ist sie gleich, dann löscht die Software des betroffenen Slave-uC sein 
MPCM-Bit und kann so die nachfolgenden Daten (mit Bit 8 = 0) empfangen.
Beim erneuten Empfang einer Adresse (mit Bit 8 = 1) geht das Spiel mit 
dem MPCM-Bit wieder von vorne los.


Diese Slave-Adresse ist also nichts, was man einfach in irgendeinem 
Register einstellen kann. Du siehst, da gibt es noch einiges zu 
Programmieren.

Hi

>To do a 16-bit write, the High byte must be written before the Low byte. For a 
16-bit read, the Low byte must be read before the High byte.

Bei dir ist es umgedreht. Mehr fällt mir im Moment nicht ein. Den 
Mastermode habe ich noch nicht benutzt.

MfG Spess

HI

>kann mir denn keiner sagen ob meine initialisierung vom sender her richtig >ist.

Ja. Falsch, siehe oben.

>anscheinend kennt sich wohl niemand mit mpcm mode aus...

Da hast du wahrscheinlich Recht. Ist aber für dich die einmalige 
Gelegenheit das herauszubekommen und es hier kundzutun.

Also was im Datenblatt vom 168 steht, ist doch ziemlich verständlich. Du 
hast ganz oben geschrieben, das die Kommunikation zwischen den AVRs 
funktioniert.
Wo ist jetzt eigentlich das Problem?

MfG Spess

Hi

>nun sollten die slavs ihre eigene adresse bekommen und das haut nicht...

Die Adressen musst du doch den Slaves verpassen.

Der Mastermode macht eigendlich nichts anderes, als erst mal alle 
Kommunikation mit den Slaves zu unterbinden, solange Bit 8 nicht gesetzt 
ist. Wenn das Bit gesetzt ist steht bei allen Slaves die vom Master 
gesendete Adresse im UDR. Das musst du auslesen (bei jedem Slave) und 
jeder Slave muss die Adresse mit seiner eigenen vergleichen. Wenn die 
Adresse nicht stimmt kann der Slave weiter ein Nickerchen machen. Bei 
übereinstimmender Adresse in den Normalmode schalten um die 
nachfolgenden Daten (mit Bit 8=0) empfangen zu können. Nach Ende der 
Übertragung wieder in den Mastermode schalten und die Nachricht 
verarbeiten. Fertig. Diese Erkenntnisse entstammen 2 min Datenblatt.

MfG Spess

Hi

Das kann nicht gehen. Du überprüfst bei jedem Interrupt ob in UDR die 
Adresse steht. Wenn nicht schaltest du den Mastermode wieder ein. Danach 
ist der Slave für 'nomale' Datenbyte wieder taub. Mit dem Code kannst du 
nur Bytes die gleich der Adresse sind empfangen.

MfG Spess

Hi

Indikator ist das RXB8n im UCSR0B. Wenn gesetzt -> Adresse, wenn 0 
Datenbyte.
Also die Adresse nur überprüfen, wenn gesetzt.

MfG Spess

Hi


>müsste ich das in der init so initialisieren..
Nein, das wird vom Master mitgesendet.


UART_RX_int:
            in regsave, sreg

            lds temp,UCSR0B
            sbrs tmp, 1<<RXB8n
            rjmp empfang2        ; wenn Datenbyte

            lds ADRES, UDR0      ; Adresse
            cpi ADRES,168        ; vergleichen
            brne ende            ; wenn kein Adresse->ende

            lds tmp, UCSR0A      ; Normalmode einstellen
            cbr tmp,1<<MPCM0
            sts UCSR0A.temp
            rjmp ende


empfang2:   ....


ende:       out sreg, regsave
            reti


Sollte in etwa so gehen.

MfG Spess

Hi

Zum 3.Mal: Nein

Beim laden von 16-Bit-Registern MUSS erst das High-Byte und DANACH das 
Low-Byte geschrieben werden (UBRR0)

>ldi  temp,(0<<MPCM0); "double tranmission speed" aus    ????????

Gewöhne dir bitte von Anfang an an, nur die Bits zu verändern, die 
geändert werden müssen. Mit dem vorherigen Befehl und

>out UCSR0A,temp      ; MPCM aus

überschreibst du das ganze Register mit $00, egal was vorher drin war. 
Also erst Einlesen, Bit(s) ändern und wieder ausgeben.
Deine Codeschnipsel sehen erst mal OK aus. Aber sortiere erst mal deine 
Kommentare.

MfG Spess

Hi

Zum 4.mal nein!

>ldi  wl,low((clock/(16*baud0))-1)  ;RS232-1 auf 9600 Baud;;
>out  ubrr0l,wl    ;Baudratenwert L setzen
>ldi  wl,high((clock/(16*baud0))-1) ;einstellen
>sts  ubrr0h,wl    ;Baudratenwert H setzen

Irgendwie scheinst du etwas lernresistend zu sein. Da ist zum Beispiel 
der gleiche Fehler drin, auf den ich dich mindesten drei mal hingewiesen 
habe.

>ldi  wl,(7<<ucsz00)
>sts  UCSR0C,wl

Das Gleiche. Gedankenloses Überschreiben eines Registers. Siehe oben.

Warum testest du das nicht im Simulator. Im Einzelschrittbetrieb kannst 
du genau verfolgen, was in den Registern passiert. Du schreibst ein 
kleines Programm, in dem du die notwendigen Register lädst und dann 
deine Routine aufrufst.
Ich helfe eigentlich ganz gern. Aber das sollte als Hilfe zur 
Selbsthilfe verstanden werden.

MfG Spess

Hi

Du hast mich noch nicht auf der Palme gesehen.

In deinem Code sehe erst mal keine Fehler. Allerdings habe das nicht mit 
dem Datenbatt verglichen.

Die Frage zum Simulator hast du aber auch nicht beantwortet.

Du solltest dir angewöhnen, dein Programm in überschaubare 'Häppchen' 
aufzuteilen. In deinem Fall z.B.:

Initialisierung UART
Sende Adresse
Sende Datenbyte
Sende eine bestimmte Anzahl Datenbytes
...

Diese Funktionen packst du in Unterprogramme. Die notwendigen Register 
werden vor dem Aufruf geladen. Im Unterprogramm, alle verwendeten 
Register, die nicht verändert werden dürfen mit 'push' sichern und mit 
'pop' rekonstruieren.
Die Unterprogramme ausgiebig testen (z.B. Simulator). Wenn du sicher 
bist, das es funktioniert kannst du das nächste Problem angehen. Damit 
vermeidest du es, mehrere Baustellen zu haben und überhaupt keinen Plan 
zu haben, wo ein Fehler sein könnte.

MfG Spess

Hi

Welche Hardware/Software benutzt du eigentlich? An deiner Stelle würde 
ich erst mal mit der Erkennung der Adresse anfangen. Also den Slave so 
programmieren, daß er bei Erkennen einer gültigen Adresse eine Led 
einschaltet.

Und was ist mit Simulator?

MfG Spess

Hi

>es gibt ja nicht mal ein tutorial wo dieser mpcm mode ausführlich  >beschrieben 
wird.

Alles, was es da zu wissen gibt, steht im Datenblatt.

>die normale uart übertragung funktioniert tadellos...

Hast du den Code dazu selbst geschrieben?

Und zum jetzt dritten und letzten mal: Hast du deinen Code schon mal im 
Simulator laufen lassen?

MfG Spess

Im Datenblatt gibt's ein Kapitel "Using MPCM", da ist sogar eine Liste, 
was du machen sollst (wenn Englisch das Problem ist, sag es wir können 
dir notfalls übersetzen helfen).

(Hab mir nur die "mcpm mode senden.asm" angeschaut:)
Was ist z.B. DAS hier?
> ldi  temp,(9<<ucsz00) ;Datenformat 9N1;
(Es gibt nur drei UCSZnm-Bits, aber auf zwei Register verteilt)

Auch da ist ein subtiler Fehler drin:

1

  ldi temp, high((clock/(16*baud1))-1)  ;RS232-1 auf 9600 Baud

2

  sts ubrr1l, temp    ;Baudratenwert L setzen

3

  ldi temp, low((clock/(16*baud1))-1) ;einstellen

4

  sts ubrr1h, temp    ;Baudratenwert H setzen

Es gibt die Direktive ".equ", um das besser lesbar zu machen:

1

  .equ UBRR_VAL=(clock/(16*baud1))-1

2

3

  ldi temp, high(UBRR_VAL)

4

  sts UBRR1H, temp

5

  ldi temp, low(UBRR_VAL)

6

  sts UBRR1L, temp

Um im MCPM-Mode eine Addresse zu schicken musst du das 9. Bit setzen, 
genau anders als du es gerade machst.

hth. Jörg
ps.: der AVR-Studio assembler versteht macros:

1

.macro set_sram_bit

2

  lds @1, @0

3

  sbr @1, @2

4

  sts @0, @1

5

.endmacro

6

.macro clr_sram_bit

7

  lds @1, @0

8

  clr @1, @2

9

  sts @0, @1

10

.endmacro

11

; wird wie folgt benutzt:

12

  set_sram_bit UCSR1B, temp, (1<<TXB81)

13

; ...

14

  clr_sram_bit UCSR1B, temp, (1<<TXB81)

pps: Es schadet Niemandem, wenn du den Code einheitlich formatierst...

1

  ldi  temp, (1<<UCSZ11)|(1<<UCSZ10)

2

    sts  UCSR1C, temp

3

4

  ldi  temp, (1<<RXEN1)|(1<<TXEN1)|(1<<UCSZ12)

5

                                            ^^^^^^^^^^^

6

    sts  UCSR1B, temp

Im Datenblatt ist hinten das "register summary" drin, (spätestens) da, 
kann man (auch du) nachschauen, WO bestimmte Bits zu finden sind.

Ich gebe mich geschlagen (keine rtfm! Hinweise mehr von mir) :(
hth. Jörg

Hi

Ich habe das am Wochenende ausprobiert. Kann dir heute Abend 
funktionierenden Code reinstellen.

MfG Spess

Hi

Ich habe dir mal beide Programme in ein File kopiert. Einen ATMEGA128 
hatte ich nicht da. Deswegen der 1281.
Das Programm für den ATMEGA1281 sendet abwechselnd die Adresse $55 und 
16 ungerade Zahlen und Adresse $56 gefolgt von 16 geraden Zahlen. 
Wichtig für dich sind die Unterprogramme 'send_address' und 'send_data'. 
Im Register r18 wird jeweils die Adresse bzw. das Datenbyte übergeben.

Im Programm für den ATMEGA88 werden die Daten empfangen und in 'buffer' 
gespeichert.In 'USART_RXC' wird ein empfangenes Datenbyte in 'data' 
gespeichert und 'data_valid' gesetzt. Das Hauptprogramm speichert das 
Byte dann im Puffer. Nach 16 Bytes wird der ATMega88 wieder in den 
Slavemode gesetzt.

Was wichtig ist: Im 9Bit-Modus MUSS RXB81/TXB81 VOR UDR gelesen bzw 
geschrieben werden.

Die Programme sind mit 11.059MHz und 9600Baud getestet 
(AVR-Dragon/JTAJ-ICE  MKII).

MfG Spess

Hi

Ich versuche mir das mal zu Gemüte zu führen. Was mir aufgefallen ist: 
Deine 57600 Baud mit 16MHz erzeugt einen Baudratenfehler, der hart an 
der Grenze liegt.
Noch was: versuche mal deine Programme ohne Tabs zu schreiben. Im 
AVRstudio Tabwidth auf 1 setzen. Jeder Editor reagiert anders auf Tabs.

MfG Spess

Hi

Die Routine kommt mir etwas suspekt vor:

uart_getchar:
  cp WPos, RPos
  breq uart_getchar
  movw YSaveL, YL
  inc RPos
  mov RX, RPos
  andi RX, RX_Buffer-1
  mov RPos, RX
  clr RX
  ldi YL, low(RXBUF)
  ldi YH, high(RXBUF)
  add YL, RPos
  adc YH, RX
  ld RX, Y
  movw YL, YSaveL

   lds temp,UCSR0A
   sbr temp,1<<MPCM0
   sts UCSR0A,temp             ; Mastermode setzen
  ret

Ich nehme an, das du damit ein Zeichen einliest.

1.  uart_getchar:
  cp WPos, RPos
  breq uart_getchar
birgt die Gefahr, daß sich das Programm aufhängt.

2.      mov RX, RPos
  andi RX, RX_Buffer-1
  mov RPos, RX

Reicht eine Zeile: andi RPos, RX_Buffer-1
Verstehe ich aber trotzdem nicht.

3. movw YSaveL, YL
   movw YL, YSaveL

Hast du eigentlich etwas gegen PUSH und POP

4.    lds temp,UCSR0A
      sbr temp,1<<MPCM0
      sts UCSR0A,temp             ; Mastermode setzen

Nach jedem Zeichen?

Mein Beispielcode basiert der Einfachheit halber au einer festen 
Datenlänge. Wenn du mit wechselnden Datenlängen arbeitest, solltest du 
ein Zeichen für das Ende der Übertragung vereinbaren. Beispielsweise 
'EOT' (ASCII $04 : End of Transmission). und danach in den Mastermode 
zurückschalten.

MfG Spess

Hi

Ist das das Teil von Benedikt?

MfG Spess

Hi

Was habe ich die ganze Zeit versucht?


MfG Spess

Hi

Überall Quarze dran?

MfG Spess