ATmega328p resetet ständig

[Flo G. (flolix)]

Liebe Leute,

ich hoffe es kann mir jemand helfen..

Von einem AtMega328p mit 16Mhz Quarz und FTDI FT232 will ich einen 
String an ein Terminal senden..

#define F_CPU 16000000UL

#include <avr/io.h>

#define BAUD 300L                  
#define UBRR_VAL  ((F_CPU ) / (BAUD * 16) - 1)  


void uart_init() {
    UBRR0H = (UBRR_VAL >> 8);
    UBRR0L = UBRR_VAL & 0xFF;
    UCSR0B = (1 << TXEN0);
    UCSR0C = (1<<UCSZ01) | (1<<UCSZ00); // 8bit 1 stopbit
}

void uart_putc (unsigned char data) {
    while (( UCSR0A & (1 << UDRE0)) == 0) {};         
    UDR0 = data;                                   
}

int main(void) {
    uart_init();
    char * s = "halo";
    char * t = s;
    /*
    uart_putc(*s);
    s++;
    uart_putc(*s);
    s++;
    uart_putc(*s);
    s++;
    uart_putc(*s);
    */
    while ( *t != '\0' ) {
        uart_putc(*t);
        t++;
    }
    while (1) { };

Es scheint als ob der Controller ständig resetet und es werden laufend 
'h' gesendet.

Wenn ich die Zeichen einzeln sende, wie im Moment auskommentiert und 
dafür die while-Schleife auskommentiere, geht alles hervorragend.

Ja, Abblockkondensatoren sind verbaut.

Weiß jemand Rat?

Danke schonmal
Flolix

[Carl D. (jcw2)]

Scheint auf den erste Blick nichts falsch zu sein.
Gib doch mal einzelne Zeichen aus.

[Carl D. (jcw2)]

Wirklich einzeln, ohne String.

[Flo G. (flolix)]

sowas wie

   uart_putc('h');
   uart_putc('a');
   uart_putc('l');
   uart_putc('o');

geht problemlos..

[Carl D. (jcw2)]

Was gibt der Compiler aus (am besten mit -v)?
Was steht inm .lss File?

[Flo G. (flolix)]

Nur main..
Mein String "halo" taucht übrigens indem ganze *.lss-File nicht auf! 
Sollte er das?!?

int main(void) {
  aa:  cf 93         push  r28
  ac:  df 93         push  r29
  uart_init();
  ae:  0e 94 41 00   call  0x82  ; 0x82 <uart_init>
  b2:  c0 e6         ldi  r28, 0x60  ; 96
  b4:  d0 e0         ldi  r29, 0x00  ; 0
  s++;
  uart_putc(*s);
  s++;
  uart_putc(*s);
  */
  while ( *t != '\0' ) { 
  b6:  02 c0         rjmp  .+4        ; 0xbc <main+0x12>
    uart_putc(*t);
  b8:  0e 94 4e 00   call  0x9c  ; 0x9c <uart_putc>
  s++;
  uart_putc(*s);
  s++;
  uart_putc(*s);
  */
  while ( *t != '\0' ) { 
  bc:  89 91         ld  r24, Y+
  be:  88 23         and  r24, r24
  c0:  d9 f7         brne  .-10       ; 0xb8 <main+0xe>
  c2:  ff cf         rjmp  .-2        ; 0xc2 <main+0x18>

000000c4 <_exit>:
  c4:  f8 94         cli

000000c6 <__stop_program>:
  c6:  ff cf         rjmp  .-2        ; 0xc6 <__stop_program>

ausgabe von avr-gcc.
compiliert und gelinkt..

Using built-in specs.
COLLECT_GCC=avr-gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/avr/4.5.3/lto-wrapper
Target: avr
Configured with: ../src/configure -v --enable-languages=c,c++ --prefix=/usr/lib --infodir=/usr/share/info --mandir=/usr/share/man --bindir=/usr/bin --libexecdir=/usr/lib --libdir=/usr/lib --enable-shared --with-system-zlib --enable-long-long --enable-nls --without-included-gettext --disable-libssp --build=i686-linux-gnu --host=i686-linux-gnu --target=avr
Thread model: single
gcc version 4.5.3 (GCC) 
COLLECT_GCC_OPTIONS='-Wall' '-c' '-o' 'o/uart.o' '-Os' '-g' '-mmcu=avr5' '-D__AVR_ATmega328P__' '-v'
 /usr/lib/gcc/avr/4.5.3/cc1 -quiet -v -imultilib avr5 -D__AVR_ATmega328P__ uart.c -quiet -dumpbase uart.c -mmcu=avr5 -auxbase-strip o/uart.o -g -Os -Wall -version -o /tmp/ccd9hlQi.s
GNU C (GCC) version 4.5.3 (avr)
  compiled by GNU C version 4.6.2, GMP version 5.0.2, MPFR version 3.1.0-p3, MPC version 0.9
GGC heuristics: --param ggc-min-expand=100 --param ggc-min-heapsize=131072
ignoring nonexistent directory "/usr/lib/gcc/avr/4.5.3/../../../avr/sys-include"
#include "..." search starts here:
#include <...> search starts here:
 /usr/lib/gcc/avr/4.5.3/include
 /usr/lib/gcc/avr/4.5.3/include-fixed
 /usr/lib/gcc/avr/4.5.3/../../../avr/include
End of search list.
GNU C (GCC) version 4.5.3 (avr)
  compiled by GNU C version 4.6.2, GMP version 5.0.2, MPFR version 3.1.0-p3, MPC version 0.9
GGC heuristics: --param ggc-min-expand=100 --param ggc-min-heapsize=131072
Compiler executable checksum: 591284b680435834ad2fbe6642ff4fee
COLLECT_GCC_OPTIONS='-Wall' '-c' '-o' 'o/uart.o' '-Os' '-g' '-mmcu=avr5' '-D__AVR_ATmega328P__' '-v'
 /usr/lib/gcc/avr/4.5.3/../../../avr/bin/as -mmcu=avr5 -o o/uart.o /tmp/ccd9hlQi.s
COMPILER_PATH=/usr/lib/gcc/avr/4.5.3/:/usr/lib/gcc/avr/4.5.3/:/usr/lib/gcc/avr/:/usr/lib/gcc/avr/4.5.3/:/usr/lib/gcc/avr/:/usr/lib/gcc/avr/4.5.3/../../../avr/bin/
LIBRARY_PATH=/usr/lib/gcc/avr/4.5.3/avr5/:/usr/lib/gcc/avr/4.5.3/../../../avr/lib/avr5/:/usr/lib/gcc/avr/4.5.3/:/usr/lib/gcc/avr/4.5.3/../../../avr/lib/
COLLECT_GCC_OPTIONS='-Wall' '-c' '-o' 'o/uart.o' '-Os' '-g' '-mmcu=avr5' '-D__AVR_ATmega328P__' '-v'
Using built-in specs.
COLLECT_GCC=avr-gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/avr/4.5.3/lto-wrapper
Target: avr
Configured with: ../src/configure -v --enable-languages=c,c++ --prefix=/usr/lib --infodir=/usr/share/info --mandir=/usr/share/man --bindir=/usr/bin --libexecdir=/usr/lib --libdir=/usr/lib --enable-shared --with-system-zlib --enable-long-long --enable-nls --without-included-gettext --disable-libssp --build=i686-linux-gnu --host=i686-linux-gnu --target=avr
Thread model: single
gcc version 4.5.3 (GCC) 
COMPILER_PATH=/usr/lib/gcc/avr/4.5.3/:/usr/lib/gcc/avr/4.5.3/:/usr/lib/gcc/avr/:/usr/lib/gcc/avr/4.5.3/:/usr/lib/gcc/avr/:/usr/lib/gcc/avr/4.5.3/../../../avr/bin/
LIBRARY_PATH=/usr/lib/gcc/avr/4.5.3/avr5/:/usr/lib/gcc/avr/4.5.3/../../../avr/lib/avr5/:/usr/lib/gcc/avr/4.5.3/:/usr/lib/gcc/avr/4.5.3/../../../avr/lib/
COLLECT_GCC_OPTIONS='-Wall' '-o' 'elf/uart.elf' '-mmcu=avr5' '-D__AVR_ATmega328P__' '-v'
 /usr/lib/gcc/avr/4.5.3/collect2 -m avr5 -o elf/uart.elf /usr/lib/gcc/avr/4.5.3/../../../avr/lib/avr5/crtm16.o -L/usr/lib/gcc/avr/4.5.3/avr5 -L/usr/lib/gcc/avr/4.5.3/../../../avr/lib/avr5 -L/usr/lib/gcc/avr/4.5.3 -L/usr/lib/gcc/avr/4.5.3/../../../avr/lib o/uart.o -lgcc -lc -lgcc

[Carl D. (jcw2)]

Der String liegt (hier unsichtbar) hinter dem Code und wird in der 
Startup-Routine ins RAM ab 0x60 kopiert.
Code scheint Ok.
Ist der WatchDog aktiv? Fuse!

[M. K. (sylaina)]

Schaut euch den Zeiger-Spass in C nochmal an. Ist doch klar, dass das so 
nicht klappen kann.

[Marc V. (Firma: Vescomp) (logarithmus)]

Flo G. schrieb:
> Mein String "halo" taucht übrigens indem ganze *.lss-File nicht auf!
> Sollte er das?!?

 Nein.
 Muss aber im .hex File sein (ist es wahrscheinlich auch).

[MaWin (Gast)]

M. K. schrieb:
> Ist doch klar, dass das so
> nicht klappen kann.

Würdest du das mal bitte näher beschreiben?

[Stefan E. (sternst)]

Was ist deine Build-Umgebung? Eigenes Makefile?
Der Code im .lss-File wurde nämlich nicht für einen Atmega328 gelinkt. 
Dessen RAM beginnt nämlich bei 0x100, nicht 0x60.

[M. K. (sylaina)]

MaWin schrieb:
> Würdest du das mal bitte näher beschreiben?

Ich würde mal sagen, dass so ein t++; ein Versuch ist, dass "h" um eins 
zu erhöhen was aber nicht geht da durch cahr *s = "halo"; das "h" eine 
Konstante ist und somit das Programm schlicht vor die Wand fährt da 
versucht wird eine Konstante um eins zu erhöhen.

[Flo G. (flolix)]

Carl D. schrieb:
> Der String liegt (hier unsichtbar) hinter dem Code und wird in der

habe ihn gefunden! :)

avr-objdump -j .data  -d -S elf/uart.elf

Watchdog ist aus!

Fuses 17 mal überprüft..
lfuse reads as FF
hfuse reads as D9
efuse reads as 7


@sylaina
Könntest Du das etwas näher erklären?

@Marc
ja im hex file ist der String... gerade geschaut..

[Flo G. (flolix)]

Stefan E. schrieb:
> Was ist deine Build-Umgebung? Eigenes Makefile?
> Der Code im .lss-File wurde nämlich nicht für einen Atmega328 gelinkt.
> Dessen RAM beginnt nämlich bei 0x100, nicht 0x60.

Ja eigenes Makefile, eigentlich nur ein kleines Script. Hat bisher immer 
tadellos funktioniert! Werde ich gleich überprüfen..

[Joe S. (joesch)]

@m.k.
t++ erhöht den zeiger um 'sizeof' char nicht den wert der durch *t 
addressiert wird.

[MaWin (Gast)]

M. K. schrieb:
> Ich würde mal sagen, dass so ein t++; ein Versuch ist, dass "h" um eins
> zu erhöhen

Du solltest das Programm noch einmal lesen und verstehen was 
Zeigerarithmetik ist.
Das Programm ist i.O.

[Carl D. (jcw2)]

MaWin schrieb:
> M. K. schrieb:
>> Ich würde mal sagen, dass so ein t++; ein Versuch ist, dass "h" um eins
>> zu erhöhen
>
> Du solltest das Programm noch einmal lesen und verstehen was
> Zeigerarithmetik ist.
> Das Programm ist i.O.

Bis auf den RAM-Start bei 0x60 statt 0x100.
Ob das am selbstgebastelten makefile liegt?

[Flo G. (flolix)]

Ja, daran lag es:

Ich hatte -mmcu=avr5 statt atmega328p gesetzt. Wo auch immer ich das 
gelesen habe.


Jetzt gehts!


Vielen herzlichen Dank für die schnelle Hilfe und schönen Abend euch 
allen!

Flolix