UART String senden

[Martin (Gast)]

Guten Abend,

Ich versuche gerade vergebens eine Zeichenkette mithilfe eines Atmega8 
an meinen Computer zu schicken. Scheinbar sind meine C Kenntnisse etwas 
eingerostet da einzelne Zeichen mit der Funktion "uart_putc" einwandfrei 
übertragen werden, innerhalb einer Schleife erhalte ich aber nur Müll.
Möglicherweise liegt ein Compilerfehler vor?

Code:

#include <avr/io.h>
#include <string.h>

#define OSCSPEED 8000000    /*OSC in Hz*/


void InitUART(uint32_t baud)
{
  uint16_t baudrate=(OSCSPEED/(16*baud))-1;
  /*Set baud rate*/
  UBRRH = (unsigned char)(baudrate>>8);
  UBRRL = (unsigned char)baudrate;
  /* Enable Receiver and Transmitter */
  UCSRB = (1<<RXEN)|(1<<TXEN);
  /* Set frame format: 8data, 1stop bit */
  UCSRC = (1<<URSEL)|(3<<UCSZ0); 
}

unsigned char UARTReceive(void)
{
  if (UCSRA & (1<<RXC))
    return UDR;
  else  /* no data pending */
    return 0;
}

int uart_putc(unsigned char c)
{
    while (!(UCSRA & (1<<UDRE)))  /* warten bis Senden moeglich */
    {
    }                             
 
    UDR = c;                      /* sende Zeichen */
    return 0;
}
 
/* puts ist unabhaengig vom Controllertyp */
void uart_puts (const char *s)
{
  while (*s) {     
    uart_putc(*s);
    s++;
  }
}

int main()
{
  char *str = "TEST";
  unsigned char c;

  InitUART(9600);

  while(1){
   
    c=UARTReceive();
    
    if (c){
      if(c=='p') {
  uart_putc('S');
  uart_putc('T');
  uart_putc('R');
  uart_putc('I');
  uart_putc('N');
  uart_putc('G');
  uart_putc(':');

  while (*str) {     
    uart_putc(*str);
    str++;
  }
      }      
    }
    
  }
  
  return 0;
}

Ausgabe am Terminalprogramm:

53 54 52 49 4E 47FFFFFFFFFFFFFF- FFFFFFFFFFFFFF      STRING:.      .
FF

Compiler/programer Anweisungen:

avr-gcc -mmcu=atmega8 -Wall -gdwarf-2 -Os -fsigned-char -I./ -o uart.out rs.c
avr-objcopy -j .text -O ihex uart.out uart.hex

avrdude -p m8 -P usb -c avrispv2 -U flash:w:uart.hex

Bin für jeden Hinweis dankbar!

beste Grüße
Martin

[Karl H. (kbuchegg)]

Martin schrieb:

> Ich versuche gerade vergebens eine Zeichenkette mithilfe eines Atmega8
> an meinen Computer zu schicken. Scheinbar sind meine C Kenntnisse etwas
> eingerostet da einzelne Zeichen mit der Funktion "uart_putc" einwandfrei
> übertragen werden, innerhalb einer Schleife erhalte ich aber nur Müll.

Lass mich raten:
Bei der ersten Antwort, die du auf einen Tastendruck hin erhältst, 
funktioniert noch alles. Bei den darauffolgenden Antworten kommt dann 
der 'Müll':

> Möglicherweise liegt ein Compilerfehler vor?

In 99% aller Fälle sitzt das Problem vor dem Bildschirm

> int main()
> {
>   char *str = "TEST";
>   unsigned char c;
>
>   InitUART(9600);
>
>   while(1){
>
>     c=UARTReceive();
>
>     if (c){
>       if(c=='p') {
>   uart_putc('S');
>   uart_putc('T');
>   uart_putc('R');
>   uart_putc('I');
>   uart_putc('N');
>   uart_putc('G');
>   uart_putc(':');
>
>   while (*str) {
>     uart_putc(*str);
>     str++;

Gratuliere.
Du hast dir soeben den einzigen Pointer den du zum konstanten String 
"TEST" hast, zerstört.

Warum benutzt du nicht die Funktion uart_puts? Dazu ist sie da.

[Martin (Gast)]

Ich verwende die Schleife zu Testzwecken, da der Funktionsaufruf mit

   uart_puts(str);

genau das gleiche Ergebnis liefert.

Bei der geposteten Ausgabe handelt es sich um den ERSTEN Aufruf der 
Schleife, sprich eigentlich (nach meinen Programmierkenntnissen) müsste 
der String korrekt gesendet werden.

Hast du sonst noch eine Idee?

[Karl H. (kbuchegg)]

Martin schrieb:

> Hast du sonst noch eine Idee?

Nicht wirklich.
Bis auf das Problem in main() sieht alles richtig aus.

[Tobias W. (hacker-tobi)]

Hi,

nimm mal die Optimierung -0s raus,evtl. hilfts.

Der C-Code sieht (abgesehen vom Problem im Main) gut aus, da geb ich dem 
Vorposter recht. Ergo sollte der erste Versand klappen.

Oder verschluckt hier irgendwas den ersten Versand? Was passiert denn, 
wenn du das Program so änderst, dass der Pointer nicht mehr ruiniert 
wird, also z.B. eine temp-variable einsetzt?



gruß

tobi

[Martin (Gast)]

int main()
{
  char *str = "TEST";
  char *p = str;

  unsigned char c;

  InitUART(9600);

  while(1){
   
    c=UARTReceive();
    
    if (c){
      if(c=='p') {
  uart_putc('S');
  uart_putc('T');
  uart_putc('R');
  uart_putc('I');
  uart_putc('N');
  uart_putc('G');
  uart_putc(':');

  while (*str) {     
    uart_putc(*str);
    str++;
  }
  str = p;
      }      
    }
    
  }
  
  return 0;
}

Code Änderung + Default Optimierung, ändert nichts.
Auch bei mehrfachem Aufruf, selbes Ergebnis.

53 54 52 49 4E 47 3A FF - FF FF FF FF FF FF FF FF - FF FF FF FF FF FF FF FF    ........  ........
FF 53 54 52 49 4E 47 3A - FF FF FF FF FF FF FF FF    .STRING:  ........
FF FF FF FF FF FF FF FF FF - FF 53 54 52 49 4E 47 3A - FF FF FF FF FF FF FF FF    .STRING:  ........
FF FF FF FF FF FF FF FF FF - FF 53 54 52 49 4E 47 3A - FF FF FF FF FF FF FF FF    .STRING:  ........
FF FF FF FF FF FF FF FF FF - FF 53 54 52 49 4E 47 3A - FF FF FF FF FF FF FF FF    .STRING:  ........
FF FF FF FF FF FF FF FF FF - FF 53 54 52 49 4E 47 3A - FF FF FF FF FF FF FF FF    .STRING:  ........
FF FF FF FF FF FF FF FF FF - FF 53 54 52 49 4E 47 3A - FF FF FF FF FF FF FF FF    .STRING:  ........
FF FF FF FF FF FF FF FF FF - FF 53 54 52 49 4E 47 3A - FF FF FF FF FF FF FF FF    .STRING:  ........
FF FF FF FF FF FF FF FF FF - FF 53 54 52 49 4E 47 3A - FF FF FF FF FF FF FF FF    .STRING:  ........
FF FF FF FF FF FF FF FF FF - FF

Seltsamerweise gibt es nicht bloss 4 Zeichen aus, sondern mehr.
Muss man vielleicht dem Compiler noch ein Keyword mitgeben damit er den 
String sauber im SRAM ablegt?

[Egberto (Gast)]

Quartz oder int. Oszillator ? Fuses richtig?

[Karl H. (kbuchegg)]

Nimm mal den Teil raus oder verkürz ihn (besser)

  uart_putc('S');
  uart_putc('T');
  uart_putc('R');
  uart_putc('I');
  uart_putc('N');
  uart_putc('G');
  uart_putc(':');

vielleicht bricht dir am MAX die Spannung zusammen.

[Martin (Gast)]

Egberto schrieb:
> Quartz oder int. Oszillator ? Fuses richtig?

Habe das Makefile von einem Beispielprogramm für UART genommen.
Sollte 8MHz intern sein. Fuses habe ich noch nicht überprüft sollten 
aber stimmen:

  avrdude -p m8 -P usb -c avrispv2 -U lfuse:w:0xe4:m -U hfuse:w:0xd9:m

 Außerdem würden ja dann die einzelnen Zeichen nicht korrekt übertragen 
werden, oder?


Karl Heinz Buchegger schrieb:
> Nimm mal den Teil raus oder verkürz ihn (besser)
>
>   uart_putc('S');
>   uart_putc('T');
>   uart_putc('R');
>   uart_putc('I');
>   uart_putc('N');
>   uart_putc('G');
>   uart_putc(':');
>
> vielleicht bricht dir am MAX die Spannung zusammen.
>

Habe ich versucht ebenfalls nur FF empfangen

FF FF FF FF FF FF                                    ......

Seltsam finde ich außerdem das strlen() für den String "TEST" den Wert 6 
zurückgibt:

 4C 45 4E 47 54 48 3A 06 -                            LENGTH:.

Habe leider kaum Erfahrung mit µC deshalb kann ich mir darauf einfach 
keinen Reim machen.

[Felix N. (time2ride)]

>> Quartz oder int. Oszillator ? Fuses richtig?
>
> Habe das Makefile von einem Beispielprogramm für UART genommen.
> Sollte 8MHz intern sein. Fuses habe ich noch nicht überprüft sollten
> aber stimmen:

UART und interner Osc. geht in den meisten Fällen schief. Probier mal 
einen externen.

[Stefan E. (sternst)]

Die Ursache deines Problems liegt in dieser Zeile:

avr-objcopy -j .text -O ihex uart.out uart.hex

Du musst auch die .data Section in das HEX-File exportieren.

[Martin (Gast)]

Stefan Ernst schrieb:
> Die Ursache deines Problems liegt in dieser Zeile:avr-objcopy -j .text -O ihex 
uart.out uart.hexDu musst auch die .data Section in das HEX-File exportieren.


Danke!
Das war der Fehler, es funktioniert jetzt einwandfrei.