Funktion wird nicht aufgerufen

[Christian (Gast)]

Guten Abend,
ich zerbreche mir seit etwa drei Stunden den Kopf über einen Problem, 
das ich mir nicht erklären kann.
Ich habe mir eine Platine erstellt um mit zwei PWM Kanälen MOSFETs zu 
steuern. Die Eingestellt Leistung soll dann über zwei Bargraphen 
dargestellt werden.
Die Platine ist soweit fertig und ich habe mir ein Programm geschrieben 
um die Funktion zu prüfen. Soweit zum Hintergrund...
In dem Testprogramm wollte ich die Ausgänge blinken lassen um eben mit 
meinen Oszilloskop zu prüfen ob die Platine i.O. ist. Das Programm sieht 
wie folgt aus:

#ifndef F_CPU
#define F_CPU 8000000L
#endif

#include <avr/io.h>
#include <util/delay.h>


void susi(void){
  DDRD |= (1<<PD1)|(1<<PD2)|(1<<PD3)|(1<<PD4)|(1<<PD5)|(1<<PD6)|(1<<PD7);
  DDRC |= (1<<DDC2)|(1<<DDC3)|(1<<DDC4)|(1<<DDC5);
  DDRB |= (1<<DDB1)|(1<<DDB2);
  
}

void test(void){
  _delay_ms(200);
  PORTD |= (1<<PD7);
  _delay_ms(10);
  PORTD &= ~(1<<PD7);
}

int main(void)
{
    //Wenn die folgende Zeile nicht auskommentiert ist, funktionieren die
    //Pins an PORTD
    //DDRD |= (1<<PD1)|(1<<PD2)|(1<<PD3)|(1<<PD4)|(1<<PD5)|(1<<PD6)|(1<<PD7);
    
    susi();  // <-- Dieser Aufruf funktioniert nicht
    while(1)
    {
    test();
        
    }
}

Wird der Funktionsaufruf "susi();" auskommentiert und alle Anweisungen 
die in der Funktion enthalten sind direkt vor die while-Schleife 
geschrieben, funktioniert alles wie geplant.
Weil ich mir das überhaupt nicht erklären kann, habe ich die Funktion 
"test();" erstellt und den Inhalt der while-Schleife darin 
untergebracht. Dieser Aufruf funktioniert jedoch wie geplant.

Ich bedanke mich schonmal freundlichst für die Hilfe, vielleicht sehe 
ich einfach den Wald vor lauter Bäumen nicht.

[Peter II (Gast)]

Christian schrieb:
> Ich bedanke mich schonmal freundlichst für die Hilfe, vielleicht sehe
> ich einfach den Wald vor lauter Bäumen nicht.

welche CPU für den Compiler gewählt?

Um welchen µC geht es überhaupt und wie sind die fuse gesetzt?

[ork (Gast)]

Du benutzt einmal P... und einmal DD... Warum ist das nicht einheitlich?

Beim Init in main wird nur ein Port initialisiert. In der Funktion drei. 
Eventuell hast du einen Kurzschluss an einem der anderen Ports???

[Lenny D. (le-do)]

Höchstinteressantes Verhalten... ;)
Hätte Tippfehler oder so was dummes vermutet, sieht aber alles OK aus.
Probier doch mal die Ausgänge direkt in der Routine zu schalten, dann 
siehst du ob sie wenigstens aufgerufen wird. Susi umbennen hast du 
bestimmt schon probiert und sollte eigentlich nicht das Problem sein 
(susi in ner header schon leer definiert?)

Kann es sein dass die Variablenänderungen auf magische Art und Weise 
lokal bleiben oder sonst wie irgendwie wegoptimiert werden?

[Christian (Gast)]

In den "PDn" Makros steht ja das gleiche wie in den "DDn" Makros, das 
hab ich schon getestet. Daher stehen hier die "PDn" Makros. Ursprünglich 
standen die "DDn" Makros drin.

Verwendet wird ein ATMega8-AU. Die Fuses sind auf internen 8MHz Takt 
64ms Delay gestellt, ansonsten auf Auslieferungszustand.

Compiliert wurde mit dem AVR-Studio 6. Zuvor hatte ich mit Xcode in 
Verbindung mit AVR-GCC aus dem AVR-Crosspack gearbeitet. Das Resultat 
war aber identisch.

Gebrannt wurde mit dem AVRISP mkII.
Soweit die Daten zum Projekt.

Wie gesagt, mich wundert, dass die Funktion "susi()" nicht innerhalb der 
"main" aufgerufen wird. Wenn jedoch alle Anweisungen aus "susi()" in der 
"main" stehen, klappt alles wie gehabt.
Merkwürdigerweise wird die Funktion "test()" aus der while-Schleife 
innerhalb der "main" aufgerufen.

[Thomas E. (thomase)]

Christian schrieb:
> Verwendet wird ein ATMega8-AU.

Wird das Programm auch für diesen Controller kompiliert und das richtige 
File geflasht? Etwas anderes kommt kaum in Frage.

mfg.

[Lumbart (Gast)]

Poste bitte die Listdatei.

[Frank (Gast)]

Hallo Christian

In den Fuses den hacken bei Comp raus dann geht's.
Das ist ein Kompatibilitäts Modus für ältere Varianten.
Hatte ich auch.

Das Programm hängt sich beim Rücksprung auf da die Stack bereinigung 
nicht stimmt.

Frank

[Thomas E. (thomase)]

Frank schrieb:
> In den Fuses den hacken bei Comp raus dann geht's.
Was ist ein "hacken"?

> Das ist ein Kompatibilitäts Modus für ältere Varianten.
> Hatte ich auch.
Aber nicht bei einem Atmega8.

mfg.

[Oliver S. (oliverso)]

Frank schrieb:
> In den Fuses den hacken bei Comp raus dann geht's.
> Das ist ein Kompatibilitäts Modus für ältere Varianten.
> Hatte ich auch.

1. Fuses haben keine Hacken
2. Der Mega8 hat keinen Kompatibilitätsmodus für "ältere Varianten". 
Sowas gibt z.B. beim Mega128
3. Ja nee, is klar...

Oliver

[Christian (Gast)]

Ok, hier mal das Listing.

Builds/main.elf:     file format elf32-avr

Sections:
Idx Name          Size      VMA       LMA       File off  Algn
  0 .text         0000007a  00000000  00000000  00000074  2**1
                  CONTENTS, ALLOC, LOAD, READONLY, CODE
  1 .data         00000000  00800060  0000007a  000000ee  2**0
                  CONTENTS, ALLOC, LOAD, DATA
  2 .stab         0000054c  00000000  00000000  000000f0  2**2
                  CONTENTS, READONLY, DEBUGGING
  3 .stabstr      00000c3a  00000000  00000000  0000063c  2**0
                  CONTENTS, READONLY, DEBUGGING
  4 .comment      00000011  00000000  00000000  00001276  2**0
                  CONTENTS, READONLY
  5 .debug_aranges 00000020  00000000  00000000  00001288  2**3
                  CONTENTS, READONLY, DEBUGGING
  6 .debug_info   000000be  00000000  00000000  000012a8  2**0
                  CONTENTS, READONLY, DEBUGGING
  7 .debug_abbrev 00000014  00000000  00000000  00001366  2**0
                  CONTENTS, READONLY, DEBUGGING
  8 .debug_line   00000058  00000000  00000000  0000137a  2**0
                  CONTENTS, READONLY, DEBUGGING

Disassembly of section .text:

00000000 <__vectors>:
   0:  12 c0         rjmp  .+36       ; 0x26 <__ctors_end>
   2:  19 c0         rjmp  .+50       ; 0x36 <__bad_interrupt>
   4:  18 c0         rjmp  .+48       ; 0x36 <__bad_interrupt>
   6:  17 c0         rjmp  .+46       ; 0x36 <__bad_interrupt>
   8:  16 c0         rjmp  .+44       ; 0x36 <__bad_interrupt>
   a:  15 c0         rjmp  .+42       ; 0x36 <__bad_interrupt>
   c:  14 c0         rjmp  .+40       ; 0x36 <__bad_interrupt>
   e:  13 c0         rjmp  .+38       ; 0x36 <__bad_interrupt>
  10:  12 c0         rjmp  .+36       ; 0x36 <__bad_interrupt>
  12:  11 c0         rjmp  .+34       ; 0x36 <__bad_interrupt>
  14:  10 c0         rjmp  .+32       ; 0x36 <__bad_interrupt>
  16:  0f c0         rjmp  .+30       ; 0x36 <__bad_interrupt>
  18:  0e c0         rjmp  .+28       ; 0x36 <__bad_interrupt>
  1a:  0d c0         rjmp  .+26       ; 0x36 <__bad_interrupt>
  1c:  0c c0         rjmp  .+24       ; 0x36 <__bad_interrupt>
  1e:  0b c0         rjmp  .+22       ; 0x36 <__bad_interrupt>
  20:  0a c0         rjmp  .+20       ; 0x36 <__bad_interrupt>
  22:  09 c0         rjmp  .+18       ; 0x36 <__bad_interrupt>
  24:  08 c0         rjmp  .+16       ; 0x36 <__bad_interrupt>

00000026 <__ctors_end>:
  26:  11 24         eor  r1, r1
  28:  1f be         out  0x3f, r1  ; 63
  2a:  cf e5         ldi  r28, 0x5F  ; 95
  2c:  d4 e0         ldi  r29, 0x04  ; 4
  2e:  de bf         out  0x3e, r29  ; 62
  30:  cd bf         out  0x3d, r28  ; 61
  32:  1e d0         rcall  .+60       ; 0x70 <main>
  34:  20 c0         rjmp  .+64       ; 0x76 <_exit>

00000036 <__bad_interrupt>:
  36:  e4 cf         rjmp  .-56       ; 0x0 <__vectors>

00000038 <susi>:
#include <avr/io.h>
#include <util/delay.h>


void susi(void){
    DDRD |= (1<<PD1)|(1<<PD2)|(1<<PD3)|(1<<PD4)|(1<<PD5)|(1<<PD6)|(1<<PD7);
  38:  81 b3         in  r24, 0x11  ; 17
  3a:  8e 6f         ori  r24, 0xFE  ; 254
  3c:  81 bb         out  0x11, r24  ; 17
    DDRC |= (1<<DDC2)|(1<<DDC3)|(1<<DDC4)|(1<<DDC5);
  3e:  84 b3         in  r24, 0x14  ; 20
  40:  8c 63         ori  r24, 0x3C  ; 60
  42:  84 bb         out  0x14, r24  ; 20
    DDRB |= (1<<DDB1)|(1<<DDB2);
  44:  87 b3         in  r24, 0x17  ; 23
  46:  86 60         ori  r24, 0x06  ; 6
  48:  87 bb         out  0x17, r24  ; 23
  4a:  08 95         ret

0000004c <test>:
  #else
    //round up by default
    __ticks_dc = (uint32_t)(ceil(fabs(__tmp)));
  #endif

  __builtin_avr_delay_cycles(__ticks_dc);
  4c:  2f ef         ldi  r18, 0xFF  ; 255
  4e:  81 ee         ldi  r24, 0xE1  ; 225
  50:  94 e0         ldi  r25, 0x04  ; 4
  52:  21 50         subi  r18, 0x01  ; 1
  54:  80 40         sbci  r24, 0x00  ; 0
  56:  90 40         sbci  r25, 0x00  ; 0
  58:  e1 f7         brne  .-8        ; 0x52 <test+0x6>
  5a:  00 c0         rjmp  .+0        ; 0x5c <test+0x10>
  5c:  00 00         nop
    
}

void test(void){
    _delay_ms(200);
    PORTD |= (1<<PD7);
  5e:  97 9a         sbi  0x12, 7  ; 18
  60:  8f e1         ldi  r24, 0x1F  ; 31
  62:  9e e4         ldi  r25, 0x4E  ; 78
  64:  01 97         sbiw  r24, 0x01  ; 1
  66:  f1 f7         brne  .-4        ; 0x64 <test+0x18>
  68:  00 c0         rjmp  .+0        ; 0x6a <test+0x1e>
  6a:  00 00         nop
    _delay_ms(10);
    PORTD &= ~(1<<PD7);
  6c:  97 98         cbi  0x12, 7  ; 18
  6e:  08 95         ret

00000070 <main>:
{
    //Wenn die folgende Zeile nicht auskommentiert ist, funktionieren die
    //Pins an PORTD
    //DDRD |= (1<<PD1)|(1<<PD2)|(1<<PD3)|(1<<PD4)|(1<<PD5)|(1<<PD6)|(1<<PD7);
    
    susi();  // <-- Dieser Aufruf funktioniert nicht
  70:  e3 df         rcall  .-58       ; 0x38 <susi>
    while(1)
    {
        test();
  72:  ec df         rcall  .-40       ; 0x4c <test>
  74:  fe cf         rjmp  .-4        ; 0x72 <main+0x2>

00000076 <_exit>:
  76:  f8 94         cli

00000078 <__stop_program>:
  78:  ff cf         rjmp  .-2        ; 0x78 <__stop_program>

[(prx) A. K. (prx)]

Christian schrieb:
> void susi(void){
>   DDRD |=
> (1<<PD1)|(1<<PD2)|(1<<PD3)|(1<<PD4)|(1<<PD5)|(1<<PD6)|(1<<PD7);
>   DDRC |= (1<<DDC2)|(1<<DDC3)|(1<<DDC4)|(1<<DDC5);
>   DDRB |= (1<<DDB1)|(1<<DDB2);
>
> }

Reduziere das mal auf genau den einen für den Test interessanten Port 
und Pin, also PD7. Dein Ersatzcode in main() enthält nur einen der 
Ports, die Susi alle. Wenn die übrigen Ports mit der Umschaltung auf 
Output einen Kurzen erzeugen, dann resetted dir u.U. der Controller weg.