Problem UART kommunikation

[Daniel L. (daniel_l49)]

Hallo zusammen!

Ich möchte eine kommunikation zwischen microcontroller und Pc mit hyper 
terminal machen. Funktioniert irgendwie nicht.

#include <asf.h>
#include <avr/io.h>
#define F_CPU 8000000UL
#include <util/delay.h>
#include <stdio.h>
#include <interrupt.h>
#define Baud 19200UL
#define UART_empty ((UCSRA&0x20)==0x20)

char string1[5] = {0};
unsigned char speicher ; 
unsigned char start;
void senden(char* Text);





int main (void)
{
  
  DDRB  |= 0xff;
  DDRA  |= 0x00;  
  UBRRH |= 0;
  UBRRL |= 25;
  UCSRB |= (1<<TXEN) | (1<<RXEN); 
  UCSRC |= (1<<URSEL) | (1<< UCSZ1) | (1<<UCSZ0);
  
  void senden (string1);
  
  while(1)
  {
    if (start ==1)
    {
      start = 0;
      void senden (string1);
    }
 
  } 
}

void senden (char* Text)
{
  unsigned int i = 0;
  while(Text[i] != '\0')
  
  if (UART_empty)
  {
    UDR = (int)Text[i];
    i++;
  }
}

ISR(USART_RXC_vect)
{
  speicher = UDR;
  start = 1;
  switch(speicher)
  {
    case '1' :
    {
      string1[0] = 'H';
      string1[1] = 'A';
      string1[2] = 'L';
      string1[3] = 'L';
      string1[4] = 'O';
      string1[5] = '\0';
      
      break;
    }
  }
}

Was mache ich falsch ?!

Ich bedanke mich im voraus.

Mfg Daniel

[Klaus W. (mfgkw)]

Zumindest fehlen volatile überall, wo globale Variablen auch in einer 
ISR verwendet werden.

[Wolfgang (Gast)]

Daniel L. schrieb:
> Was mache ich falsch ?!

Wie soll in dein 5 Zeichen langes Array ein Text mit 5 Zeichen 
reinpassen?

[Wegstaben V. (wegstabenverbuchsler)]

Daniel L. schrieb:
> Was mache ich falsch ?!

du gibst keine genaue Fehlerbeschreibung. "funktioniert irgendwie nicht" 
ist zu wenig.

[Gerhard W. (gerhard86)]

Hast du den TXD denn auf Ausgang gesetzt? Das musst du imho manuell 
machen. Ist bei den Atmega ein Pin auf Port D. Dann sollte er schon 
einmal senden.

Du hast den Receive Interrupt nicht an (RXCIE oder so in UCSRB) und
soweit ich das sehen kann auch keinen globalen Interrupt (sei()). Daher 
wird er nichts empfangen. Und wie schon erwähnt, dein String ist zu 
kurz, aber daran liegt nicht dass nichts passiert, das wird 
möglicherweise eher dafür sorgen dass er so bald nicht mehr aufhört ;-).

Und das da: DDRA  |= 0x00; macht überhaupt nix.

[spess53 (Gast)]

Hi

>Hast du den TXD denn auf Ausgang gesetzt? Das musst du imho manuell
>machen.

Nein. Mit TXEN und/oder RXEN übernimmt die UART die Kontrolle über die 
Pins.

MfG Spess

[Karl H. (kbuchegg)]

Daniel L. schrieb:

weder das hier

>   UCSRC |= (1<<URSEL) | (1<< UCSZ1) | (1<<UCSZ0);
> 
>   void senden (string1);

nach das hier

>     if (start ==1)
>     {
>       start = 0;
>       void senden (string1);

ist der Aufruf einer Funktion


Wurde ja schon angesprochen

>       string1[0] = 'H';
>       string1[1] = 'A';
>       string1[2] = 'L';
>       string1[3] = 'L';
>       string1[4] = 'O';
>       string1[5] = '\0';

zähl doch mal. Das sind 6 Zuweisungen. Dein Array

char string1[5] = {0};

ist aber nur 5 Elemente groß!



> Hast du den TXD denn auf Ausgang gesetzt?
Nö. Muss man nicht. Wenn dir USART den Pin übernimmt (weil der 
Transmitter eingeschaltet wird), übernimmt die die komplette Kontrolle 
über den Pin.

[Klaus W. (mfgkw)]

Tip: wenn man die grundlegenden Sachen nicht hinbekommt, kann man auch 
mal ins Tutorial schauen und mit einem Beispiel von dort starten.
Mit dem, was dort schon dankenswerterweise breit ausgetreten ist, muß 
man ja nicht nochmal alle Leute beschäftigen.

[Michael (Gast)]

Gerhard W. schrieb:
> Und wie schon erwähnt, dein String ist zu
> kurz, aber daran liegt nicht dass nichts passiert, das wird
> möglicherweise eher dafür sorgen dass er so bald nicht mehr aufhört ;-).

... und genau beim ersten Empfang von '1' wird der Inhalt der nach 
string1 im speicher liegenden Variablen platt gemacht. ;-)