Zeichenketten ueber UART

[Owen S. (senmeis)]

Hi,

ich nutze IAR zur Entwicklung des AVR. Mit dem folgenden Code möchte ich 
einfach eine Zeichenkette über UART senden lassen. Es kommt aber vor, 
dass mit dem nachstehenden Code

 temp = Usart_Rx();

 die UART Übertragung verfälscht wird (0x31, 0x32, 0x01, 0x01, 0x35 
statt 0x31, 0x32, 0x33, 0x34, 0x35), mit

 while(1);

 danach ist alles OK.

Der Code ist hier. Hat das mit dem Timing zu tun?

unsigned char temp;
static char enter = 1;
unsigned char str[] = "12345";

if (enter)
{
  enter = 0;
  uart_put_string(str);
}
else
{        
    temp = Usart_Rx();
//while(1);
}

char Usart_Rx(void)
{
    while (!(UCSR0A & (1<<RXC0)));
    return UDR0;
}

void uart_put_string (unsigned char *data_string)
{
    while(*data_string) Usart_Tx(*data_string++);
}

Gruss
Owen

[Michael D. (etzen_michi)]

Was mir erstmal auffällt:

if (enter)
{
  enter = 0;
  uart_put_string(str);
}
else
{        
  temp = Usart_Rx();   // enter = Usart_Rx();
  //while(1);
}

[Owen S. (senmeis)]

Sorry, habe for Schleife vergessen zu posten.

for(;;) // main loop
{
  if (enter)
  {
    enter = 0;
    uart_put_string(str);
  }
  else
  {        
    temp = Usart_Rx();
    //while(1);
    if (temp == '\n') // Endzeichen
    {
    enter = 1;
    }
  }
}

Owen

[Karl H. (kbuchegg)]

Und wo ist deine Usart_Tx?

[Owen S. (senmeis)]

Sorry, Usart_Tx() hier.

void Usart_Tx(char data)
{
    while (!(UCSR0A & (1<<UDRE0)));
    UDR0 = data;
}

Owen

[Krapao (Gast)]

Mit dem Code aus Beitrag "Re: Zeichenketten ueber UART" ist 
mir unklar, wie du den Fehler aus 
Beitrag "Zeichenketten ueber UART" feststellen kannst. 
Du hast doch nirgends eine Ausgabe des enpfangenen temp. Gehst du mit 
einem ICD im Einzelschritt in das laufende System und verlierst dabei 
eventuell Zeichen?

[Stefan (Gast)]

Owen Senmeis schrieb:

> while(*data_string) Usart_Tx(*data_string++);
                               ^

du musst klammern

  while(*data_string) Usart_Tx(*(data_string++));

[Krapao (Gast)]

Die Klammern erhöhen die Lesbarkeit, sind aber nach den C Operatoren 
Vorrangregeln nicht nötig.

*a++ entspricht *(a++)

[Owen S. (senmeis)]

Ich poste den ganzen Code hier. Es geht um ein Ablesen aus einem CO2 
Sensor.

int main( void )
{
  unsigned char i = 0, temp;
  unsigned char buffer[STRLENGTH];
  unsigned char co2_value[STRLENGTH], temp_index = 0;
  volatile static char enter = 1;  
  unsigned char str[] = "12345";
  
  clock_init_8MHz();
  USART_Init(103); // 9600bps by 8MHz
   
  __enable_interrupt();

  for(;;) // main loop
  {
      if (enter)
      {
        enter = 0;   
        //uart_put_string(str);   // data request
        uart_send();
      }
      else
      {        
          temp = Usart_Rx();
          
          if (temp == '\n')   // end character
          {
            if ((buffer[0] == 'Z') && (buffer[i-1] == '\r'))  // if the format is correct
            { 
              for (temp_index = 2; temp_index < (i-1); temp_index++)
              {
                co2_value[temp_index-2] = buffer[temp_index]; // delete format character
              }
              co2_value[temp_index-1] = '\0';   // add null character
                           
              uart_put_string(co2_value);   // output co2 value
            }
            enter = 1;  // trigger request
          }
          else 
          {
            if (temp == 'Z')  // start character
            {  
              i = 0;
            }
            buffer[i] = temp;   // save data
            i++;  
          }
      }
  }
}

Die Arbeitsweise sieht so aus:

1. Eine Zeichenkette ausgeben (z.B. „12345“), um Messwerte anzufordern.

2. Wenn der Messwert kommt wird dieser entpackt und zurückgeschickt.
Z.B. Empfangsbytes: 0x5a 0x20 0x34 0x35 0x30 0x0D 0x0A
Nach dem Entpacken: 0x34 0x35 0x30, also 450

Es gibt engentlich zwei Probleme mit dem Code.

Problem 1: Wie erwähnt wird die Zeichenkette nicht richtig ausgegeben. 
Vermutlich liegt das an uart_put_string(), denn mit der folgenden 
Methode geht es.

void uart_send(void)
{
  Usart_Tx('1');
  Usart_Tx('2');
  Usart_Tx('3');
  Usart_Tx('4');
  Usart_Tx('5');
}

Problem 2: Der Messwert wird nicht zurückgeschickt.
Man gibt im Terminal ein: 0x5a 0x20 0x34 0x35 0x30 0x0D 0x0A und 
erwartet 0x34 0x35 0x30 als Rückmeldung, aber nichts kommt.

Ich bin sehr dankbar für jeden Vorschlag.
Gruss
Owen

[Karl H. (kbuchegg)]

Deine Hilfsfunktionen sehen erst mal korrekt aus.

Aber nur um sicher zu gehen, sollte man die erst mal für sich alleine 
testen. Aber nicht mit deinem umfangreichen Code, sondern mit einem 
einfachen Testprogramm!

int main()
{
  unsigned char str[] = "12345\r\n";

  clock_init_8MHz();
  USART_Init(103); // 9600bps by 8MHz  

  while( 1 ) {
    uart_put_string( str );
  }
}

das muss eine saubere Ausgabe auf dein Terminal zaubern. Wenn es das 
nicht tut, dann hast du höchst wahrscheinlich irgendein Taktproblem.

[Krapao (Gast)]

int main( void )
{
  unsigned char buffer[STRLENGTH];
  unsigned char co2_value[STRLENGTH];
  unsigned char i = 0;
  
  clock_init_8MHz();
  USART_Init(103); // 9600bps by 8MHz
   
  __enable_interrupt();

  // prepare rx line
  i = 0; 
  buffer[0] = 0; 
 
  for(;;) // main loop
  {
    unsigned char temp;
    temp = Usart_Rx();
    if ( temp == '\n' )   // end character
    {
      // format correct? then output result
      if ( buffer[0] == 'Z' && buffer[1] == ' ' && buffer[i-1] == '\r' )
      { 
        buffer[i-1] = 0; // delete \r
        strncpy(co2_value, &buffer[2], STRLENGTH); // skip "Z "
        uart_put_string("co2_value: [");
        uart_put_string(co2_value);
        uart_put_string("]\r\n");
      } 
      // prepare rx line
      i = 0; 
      buffer[0] = 0; 
    }
    else 
    {
      if ( i+1<STRLENGTH )
      {
        buffer[i++] = temp;
        buffer[i] = 0;  
        uart_put_string("buffer: [");
        uart_put_string(buffer);
        uart_put_string("]\r\n");
      }
    }
  }
}

[Karl H. (kbuchegg)]

Der zuletzt gepostete Code, ist das jetzt auch wirklich alles? Kommt da 
noch irgendeine Funktionalität dazu?
(Der sieht nämlich für die bisher bekannte Aufgabenstellung ziemlich 
umständlich aus. Auch hast du dir keinerlei Hilfen in den Code 
eingebaut, bei denen dir das Programm selbst mitteilen würde, was es 
wann und warum macht. Das einfachste ist erst mal, dass das Programm 
jedes Zeichen, das es empfängt auch gleich mal zurückschickt. Auch wäre 
interessant, was sich an Zeichen im Buffer nach jedem empfangenen 
Zeichen angesammelt hat. Du hast eine UART, die hoffentlich 
funktioniert. Benutze sie, damit dir das Programm selber hilft, es zu 
debuggen!)