Uart String empfangen

[Peter (Gast)]

Hallo,

ich möchte mit einem Mega8 Strings per Uart empfangen. Basis ist dabei 
die Lib von P. Fleury. Ich schicke Strings vom PC an den µC und wenn 
dort der String komplett angekommen ist soll er zurück zum PC geschickt 
werden. Leider kommt der String häufig unvollständig zurück.

void uart_gets(char *test)
{
  char c;
  c = uart_getc();
  if(!(c & UART_NO_DATA))
  {
  *test = c;
  do {
    test++;
    *test = uart_getc();
    }while(*test != '\0');
  }
}
int main(void) 
{
  ck_1000 = 0;
  sei();
  uart_init(UART_BAUD_SELECT(UART_BAUD_RATE,F_CPU) ); 
  adc_init();      
  while(1)
  {
    char test[20] = "";  
    uart_gets(test);  //Serielle Daten empfangen
    uart_puts(test);  //Serielle Daten zur Kontrolle 
        }
  return 0;
}

Hat jemand einen Tip wo es hakt?

Schönen Abend

[Karl H. (kbuchegg)]

Peter schrieb:


> void uart_gets(char *test)
> {
>   char c;
>   c = uart_getc();
>   if(!(c & UART_NO_DATA))
>   {
>   *test = c;
>   do {
>     test++;
>     *test = uart_getc();

Du musst nach JEDEM uart_getc() Aufruf überprüfen, ob uart_getc ein 
Zeichen für dich hat! Nicht nur beim ersten!

Die Übertragung von Zeichen ist wesentlich langsamer als diese Schleife 
hier. d.h. in dieser Schleife wird nur ab und zu von uart_getc() ein 
Zeichen geliefert werden. Die meiste Zeit wird uart_getc() mit gesetzten 
UART_NO_DATA Bit zurückkehren. Und nur dann, wenn UART_NO_DATA nicht 
gesetzt ist, ist wieder ein Zeichen eingetrudelt, welches du an den 
String anhängen kannst.

[Peter (Gast)]

Habe versucht deine Infos in die Empfangsroutine einfließen zu lassen, 
klappt leider noch nicht so richtig:

void uart_gets(char *test)
{
  char c;
  c = uart_getc();
  if(!(c & UART_NO_DATA))
  {
  *test = c;
  test++;
  do {
    if(!(c & UART_NO_DATA))
    {
      *test = uart_getc();
      test++;
    }
    }while(*test != '\0');
  }
}

[Karl H. (kbuchegg)]

Peter schrieb:
> Habe versucht deine Infos in die Empfangsroutine einfließen zu lassen,
> klappt leider noch nicht so richtig:
> [c]
> void uart_gets(char *test)
> {
>   char c;
>   c = uart_getc();
>   if(!(c & UART_NO_DATA))
>   {
>   *test = c;
>   test++;
>   do {
>     if(!(c & UART_NO_DATA))
>     {
>       *test = uart_getc();
>       test++;

Du gehst immer noch davon aus, dass das was von uart_getc() kommt ein 
gültiges Zeichen ist.

Du musst das, was du von uart_getc() bekommst TESTEN, ob UART_NO_DATA 
nicht gesetzt ist, und nur dann wenn UART_NO_DATA nicht gesetzt ist, 
erst dann kannst du es durch Zuweisung an *test an den String anhängen!

Es hilft dir nichts, wenn du immer nur c abtestest, ob UART_NO_DATA 
gesetzt ist. c hat mit dem LETZTEN Aufruf von uart_getc() nix zu tun!

Du hast zwar dein Programm ein wenig umgestellt, aber du hast an der 
Logik nichts geändert. Das Grundübel ist immer noch das gleiche.

[Karl H. (kbuchegg)]

Ausserdem muss c ein unsigned int sein!
Siehe die Doku von P.Fleury

void uart_gets(char *test)
{
  unsigned int c;

  c = uart_getc();
  if(!(c & UART_NO_DATA))
  {
    *test = c;
    test++;

    do {
      c = uart_getc();
      if(!(c & UART_NO_DATA))
      {
        *test = c;
        test++;
      }
    } while( c != '\0');  // da ist das Bit UART_NO_DATA implizt mit drinnen
                          // die Schleife läuft also auch weiter, wenn
                          // kein Zeichen angekommen ist.
  }
}

Edit: Bist du sicher, dass der Sender die Übertragung eines Strings mit 
einem '\0' beendet? Es ist zwar nicht unmöglich, aber eher unüblich das 
zu tun.

[Peter (Gast)]

Geht jetzt, vielen Dank für deine Hilfe!