großes Programm funktioniert nicht

[Hebel Zieher (Gast)]

Hi,

Ich habe ein Problem mit meinem Programm.

µC: PIC 18F4523
Programmsprache: C
compiler: MPLAB C18

Ich habe mir eine 6x6x6 LED Cube gebaut welche auch Funktionstüchtig ist 
(http://www.youtube.com/watch?v=_YT1HWFXGgo).
Mein Programm bestet im prenzip aus einer Rendern funktion welche einen 
Array auswertet. In diesen Array stehn die Bildinformationen.

So nun zum eigentlichem Problem:
Solange das Programm klein bleibt (Laut MPLAB 2700 Zeilen bzw laut 
Memory Usage Gauge: 4036 von 16384 Words) geht alles Wunderbar. Sobalt 
das Programm aber über eine gewisse Größe (solte bei 4000-4500 words 
leigen) geht, steigt das Programm quasi aus. Die Leds machen was sie 
wollen, der µC macht meiner meinung garnichts mehr.



Code (hauptprogramm):

/** I N C L U D E S **********************************************************/
#include <p18cxxx.h>
#include "delays.h"    // für die Warteschleife
//#include "stdlib.h"

/** Configuration ********************************************************/
#pragma config OSC = HS   //CPU=14.7456 MHz
#pragma config PWRT = OFF
//#pragma config BOR = OFF
#pragma config WDT = OFF  //Watchdog Timer
#pragma config LVP = OFF  //Low Voltage ICSP
//#pragma config STVREN = OFF  //Steck Full/underflow reset
 

/** Genutzte PORTS des PICs ***********************************************

PORTA:  A0 - S_Latch 1 AN
    A1 - S_Latch 2 AN
    A2 - S_Latch 3 AN

PORTB  B0 - S_Latch LED 1
    B1 - S_Latch LED 2
    B2 - S_Latch LED 3
    B3 - S_Latch LED 4
    B4 - S_Latch LED 5
    B0 - S_Latch LED 6

PORTC  C0 - E_Latch Ebene 1
    C1 - E_Latch Ebene 2
    C2 - E_Latch Ebene 3
    C3 - E_Latch Ebene 4
    C4 - E_Latch Ebene 5
    C5 - E_Latch Ebene 6
    C6 - TX MAX232
    C7 - RX MAX232

PORTD  D0 - E_Latch 1 AN
    D1 - E_Latch 2 AN

*/



#pragma code
  unsigned char LED[37]; 

  unsigned char s, e, x, i;
  unsigned int time, t;

// Init()
// Stellt die Ein/Ausgänge ein
// Schaltet alle Ausgänge auf 0

void init(void) {
  
  LATB  = 0x00;    //Setzt alle seulen auf 0
  TRISB  = 0x00;    //Stellt Port B auf Output
  
//  LATA  = 0x00;    //Setzt alle Seulen Latches auf 0
  TRISA  = 0x00;    //Stellt PORT A auf Output
  LATA  = 0xFF;    //Setzt alle Seulen Latches auf 1 um sie zu leeren

  LATC  = 0x00;    //Setzt alle Ebenen auf 0
  TRISC  = 0x00;    //Stellt PORT C auf Output

//  LATD  = 0x00;    //Setzt alle Ebenen Latches auf 0
  TRISD  = 0x00;    //Stellt Port D auf Output
  LATD  = 0xFF;    //Setzt alle Ebene Latches auf 1 um sie zu leeren

  Delay1TCY();    //Wartet kurz um sicherzu gehn das wirklich alle Signale gelöscht wurden
  
  LATA  = 0x00;    //Schaltet die Seulen Latches ab
  LATD  = 0x00;    //Schaltet die Ebenen Latches ab

//Konstanten *****************************************************************

#define  Seule_1   LATBbits.LATB0    //Seule 1      
#define  Seule_2   LATBbits.LATB1    //Seule 2
#define  Seule_3   LATBbits.LATB2    //Seule 3
#define  Seule_4   LATBbits.LATB3    //Seule 4
#define  Seule_5   LATBbits.LATB4    //Seule 5
#define  Seule_6   LATBbits.LATB5    //Seule 6

#define  S_Latch_1 LATAbits.LATA0    //Seulen Latch für Seule 1-6   & 19-24  
#define  S_Latch_2 LATAbits.LATA1    //Seulen Latch für Seule 7-12  & 25-30
#define  S_Latch_3 LATAbits.LATA2    //Seulen Latch für Seule 13-18 & 31-36

#define  Ebene_1   LATCbits.LATC0    //Ebene 1
#define  Ebene_2   LATCbits.LATC1    //Ebene 2
#define  Ebene_3   LATCbits.LATC2    //Ebene 3
#define  Ebene_4   LATCbits.LATC3    //Ebene 4
#define  Ebene_5   LATCbits.LATC4    //Ebene 5
#define  Ebene_6   LATCbits.LATC5    //Ebene 6

#define  E_Latch_1 LATDbits.LATD0    //Ebenen Latch vorne
#define  E_Latch_2 LATDbits.LATD1    //Ebenen Latch hinten
//*****************************************************************************
}//Init() ENDE



//Löscht alle LED Signale
void clear_all(void) {
  LATB  = 0x00;    //Seulen Bus auf 0
  LATC  = 0x00;    //Ebenen Bus auf 0
  LATA  = 0xFF;    //Seulen Löschen
  LATD  = 0xFF;    //Ebenen Löschen
  
  Delay1TCY();    //Wartet kurz um sicherzu gehn das wirklich alle Signale gelöscht wurden
  
  LATA  = 0x00;    //Schaltet die Seulen Latches ab
  LATD  = 0x00;    //Schaltet die Ebenen Latches ab
}//clear_all() ENDE


// 1 durchlauf entspricht 1/370s
void Rendern(unsigned char LED[37]) {
  
  clear_all();      //Resettet alle Signale

  //unsigned char s, e, x, i;

  for (e=0; e<12; e++) {        // 12 druchgänge für die 12 Ebenen
    for (s=1; s<4; s++) {      //  3 durchgänge für die  3 Seulen 
      //Schaltet die Seulen Latches ab
      S_Latch_1 = 0;    
      S_Latch_2 = 0;
         S_Latch_3 = 0;

      x = e*3+s;          //Errechnet welche index im Array benutzt werden soll
      LATB = LED[x];
      switch (s) {        //Switch: Schaltet je nach dem welche Seulen beutzt werden die entsprechende durch
        case 1:          //Die vorder Reihe
             S_Latch_1 = 1;
          break;
        case 2:          //Die mittlere Reihe
             S_Latch_2 = 1;
          break;
        case 3:          //Die hintere Reihe
             S_Latch_3 = 1;
          break;
      }//Switch S ENDE
    }//Seulen schleife ENDE (3 durchläufe)
    

    switch (e) {
      case 0:
        Ebene_1 = 1;    //Ebenen BUS
        E_Latch_1 = 1;    //Ebenen Latch AN/AUS
          break;
      case 1:
        Ebene_2 = 1;    //Ebenen BUS
         E_Latch_1 = 1;    //Ebenen Latch AN/AUS
          break;
      case 2:
        Ebene_3 = 1;    //Ebenen BUS
         E_Latch_1 = 1;    //Ebenen Latch AN/AUS
          break;
      case 3:
        Ebene_4  = 1;    //Ebenen BUS
         E_Latch_1 = 1;    //Ebenen Latch AN/AUS
          break;
      case 4:
        Ebene_5 = 1;    //Ebenen BUS
         E_Latch_1 = 1;    //Ebenen Latch AN/AUS
          break;
      case 5:
        Ebene_6 = 1;    //Ebenen BUS
         E_Latch_1 = 1;    //Ebenen Latch AN/AUS
          break;
      case 6:
        Ebene_1 = 1;    //Ebenen BUS
        E_Latch_2 = 1;    //Ebenen Latch AN/AUS
          break;
      case 7:
        Ebene_2 = 1;    //Ebenen BUS
        E_Latch_2 = 1;    //Ebenen Latch AN/AUS
          break;
      case 8:
        Ebene_3 = 1;    //Ebenen BUS
        E_Latch_2 = 1;    //Ebenen Latch AN/AUS
          break;
      case 9:
        Ebene_4 = 1;    //Ebenen BUS
        E_Latch_2 = 1;    //Ebenen Latch AN/AUS
          break;
      case 10:
        Ebene_5 = 1;    //Ebenen BUS
        E_Latch_2 = 1;    //Ebenen Latch AN/AUS
          break;
      case 11:
        Ebene_6 = 1;    //Ebenen BUS
        E_Latch_2 = 1;    //Ebenen Latch AN/AUS
          break;
    }//Switch E ENDE
    
    //Bildanzeige dauer (ebene dauer 20µS
    for (i=0; i<75; i++) {
          Delay1TCY();
    }
  clear_all();      //Resettet alle Signale  
  }//Ebene Schleife ENDE (12 durchläufe

//clear_all();      //Resettet alle Signale  
return;
}//end Rendern


//Bild datein
#include "Muster/schlange.inc"  

void main(void)
{

  init();    //Stellt alle Ports ein und Löscht die ausgänge



//Programm schleife mit den Bilddatein
while(1){




schlange();    //Muster aus den Bilddatein


}
}//end main

Code Bildinhalt (ausschnitt):

void schlange(void) {


time = 50; // 0,13sec
  //Bild
  //Seulen vorne
  LED[1]  = 0x0;  //ebene 1
  LED[2]  = 0x0;
  LED[3]  = 0x0;
  LED[4]  = 0x0;  //Ebene 2
  LED[5]  = 0x0;
  LED[6]  = 0x0;
  LED[7]  = 0x0;  //Ebene 3
  LED[8]  = 0x0;
  LED[9]  = 0x0;
  LED[10]  = 0x0;  //Ebene 4
  LED[11]  = 0x0;
  LED[12]  = 0x0;
  LED[13]  = 0x0;  //Ebene 5
  LED[14]  = 0x0;
  LED[15]  = 0x0;
  LED[16]  = 0x1;  //Ebene 6
  LED[17]  = 0x0;
  LED[18]  = 0x0;

  //Seulen hinten
  LED[19]  = 0x0;  //Ebene 1
  LED[20]  = 0x0;
  LED[21]  = 0x0;
  LED[22]  = 0x0;  //Ebene 2
  LED[23]  = 0x0;
  LED[24]  = 0x0;
  LED[25]  = 0x0;  //Ebene 3
  LED[26]  = 0x0;
  LED[27]  = 0x0;
  LED[28]  = 0x0;  //Ebene 4
  LED[29]  = 0x0;
  LED[30]  = 0x0;
  LED[31]  = 0x0;  //Ebene 5
  LED[32]  = 0x0;
  LED[33]  = 0x0;
  LED[34]  = 0x0;  //Ebene 6
  LED[35]  = 0x0;
  LED[36]  = 0x0;

  for (t=0; t<time; t++) {
  Rendern(LED);
  }
  

  //Bild
  //Seulen vorne
  LED[1]  = 0x0;  //ebene 1
  LED[2]  = 0x0;
  LED[3]  = 0x0;
  LED[4]  = 0x0;  //Ebene 2
  LED[5]  = 0x0;
  LED[6]  = 0x0;
  LED[7]  = 0x0;  //Ebene 3
  LED[8]  = 0x0;
  LED[9]  = 0x0;
  LED[10]  = 0x0;  //Ebene 4
  .
        .
        .
//Der code wiederholt sich ab hier immer wieder mit anderen werten für die Arrays
//bis ca zeile 2500 läuft das alles hier problemlos ab zeile 2500-3000 leider nicht mehr

Da ich ein Anfänger bin was C-Programierung und µC allgemein angeht 
hoffe ich das ihr mir helfen könnt.
Bis ez ist mir nur der gedanke gekommen das die vielen Arrays probleme 
machen.

vielen Dank im vorraus
Der Hebel Zieher

[Christian H. (christian_h)]

Wie kompilierst du denn?  Und du bist dir auch ganz sicher, dass das 
Endprodukt noch auf den Chip passt?

[Hebel Zieher (Gast)]

Christian H. schrieb:
> Wie kompilierst du denn?  Und du bist dir auch ganz sicher, dass das
> Endprodukt noch auf den Chip passt?

Ich kompiliere über MPLAB mit dem c18 Compiler, hier is mal noch der 
Build bericht:

Executing: "C:\Program Files (x86)\Microchip\mplabc18\v3.38\bin\mcc18.exe" -p=18F4523 "main.c" -fo="main.o" -Ou- -Ot- -Ob- -Op- -Or- -Od- -Opa-
MPLAB C18 v3.38 (evaluation)
Copyright 2000-2011 Microchip Technology Inc.
Days remaining until evaluation becomes feature limited:  54
Executing: "C:\Program Files (x86)\Microchip\mplabc18\v3.38\bin\mplink.exe" /p18F4523 /l"C:\Program Files (x86)\Microchip\mplabc18\v3.38\lib" "C:\Program Files (x86)\Microchip\mplabc18\v3.38\bin\LKR\18f4523_g.lkr" "main.o" /u_CRUNTIME /z__MPLAB_BUILD=1 /m"Led Cube 6³.map" /w /o"Led Cube 6³.cof"
MPLINK 4.39, Linker
Device Database Version 1.1
Copyright (c) 1998-2011 Microchip Technology Inc.
Errors    : 0

MP2HEX 4.39, COFF to HEX File Converter
Copyright (c) 1998-2011 Microchip Technology Inc.
Errors    : 0

Loaded L:\LED Cube 6x6x6\Pic prog\Led Cube 6³.cof.

Wegen der größe des programms ich bin mir absolut sicher das es nicht zu 
groß ist.
MPLAB sag das nur 4000 von 16 000 Words belegt sind und wenn ich mir die 
HEX file anschaue sind da auch mehr wie genug adressen frei.

[Lehrmann M. (ubimbo)]

Schau dir doch mal die Simulation an, ob da die LATx die von dir 
gewünschten Werte annimmt.

Dem Compiler-Report entnehme ich, dass du ein Evaluationsversion von C18 
hast. Richtig? Kein gecracktes Programm ?! ....

Hebel Zieher schrieb:
> Die Leds machen was sie
> wollen, der µC macht meiner meinung garnichts mehr.

Erzähl mal mehr über deine Schaltung.

Könnte es sein, dass deine Stromversorgung nicht ausreichend ist?

Außerdem solltest du deine orthographischen Fähigkeiten mal überdenken. 
Säule nicht Seule, Prinzip nicht prenzip, Adjektive beginnen mit 
Kleinbuchstaben, Nomen immer mit Großbuchstaben. Ich sag das nicht um 
dich zu ärgern od. zu demütigen, sondern das hat auch viel mit 
Programmierkunst zu tun. Gerade wenn man sich für C entscheidet 
(zugegeben es gibt außer Assembler keine Alternative), dann muss man auf 
Präzision achten, ansonsten verrennt man sich gerade bei größeren 
Programmen verdammt schnell. Ganz nebenbei ist es recht peinlich, dass 
man in seinem Programmcode mit grausamer Konsequenz 'Seule' und nicht 
'Säule' schreibt.

[holger (Gast)]

>Ganz nebenbei ist es recht peinlich, dass
>man in seinem Programmcode mit grausamer Konsequenz 'Seule' und nicht
>'Säule' schreibt.

Ob der Compiler das 'ä' im define wohl mag?
Ich glaube nicht.
Du bist echt nen Bimbo;)

[M. J. (manfred-64)]

mal so ne frage neben bei,
ich programmiere nur (aäöo... bisher) in Assembler!?
ist das obige Programm in C wirklich als "Groß" zu bezeichnen???
in Assembler is das eher als "Popelig" zu bezeichnen!

[Purzel H. (hacky)]

>ist das obige Programm in C wirklich als "Groß" zu bezeichnen???

Natuerlich nicht. Das obige Programm ist pipifax

[P. S. (Gast)]

Manfred John schrieb:

> ist das obige Programm in C wirklich als "Groß" zu bezeichnen???

Noe.

> in Assembler is das eher als "Popelig" zu bezeichnen!

In C auch.

[M. J. (manfred-64)]

Peter Stegemann schrieb:
> Manfred John schrieb:
>
>> ist das obige Programm in C wirklich als "Groß" zu bezeichnen???
>
> Noe.
>
>> in Assembler is das eher als "Popelig" zu bezeichnen!
>
> In C auch.

wie beruhigend :)

da werd' ich mir doch mal ANSI-C und ??? µC-C reinzien (µC-C???)
oda was is für PIC's für lau zu bekommen is. (nur für "kleine Projekte")
(und wenn ich nur im "Notfall" nur Fix-Point-Operationen verwende)
hab mom nur mir 16F... zu tun
was wäre da zu empfehlen (da mach ich noch'n gesonderten Thread auf, 
wenn ich sonst nix finde)

[Hebel Zieher (Gast)]

Lehrmann Michael schrieb:
> Schau dir doch mal die Simulation an, ob da die LATx die von dir
> gewünschten Werte annimmt.

Werde ich morgen mal angehn.


Lehrmann Michael schrieb:
> Erzähl mal mehr über deine Schaltung.
>
> Könnte es sein, dass deine Stromversorgung nicht ausreichend ist?

Alle ICs sind mit Kondensatoren gepuffert außerdem läuft das 
Programm/Hardware einwandfrei bis zu einer gewissen Programmgröße.

Mein Programm besteht wie gesagt aus 2 Teilen:
1. Das 'kleine' Hauptprogramm (ca 200 Zeilen lecherlich ich weiß)
2. Der Bildinformation (groß)- Ich benötige bei meinem jetzigen 
Programmaufbau einen Array mit 36 6-bit informationen pro Bild, welches 
angezeit werden soll. Diesen Code lasse ich mir mit einem 
HTML/PHP-Ürogramm erzeugen. Wird dieser Code nun zulang (ab ca 4250 von 
16000 words) steigt das programm aus.

Das ich irgendwo ein tipfehler gemacht habe ist ausgeschlossen da  der 
Code ja erzeugt wurde.


Noch ein Hinweiß zum fehlerablauf:
Solange das Programm wie besagt unter 4250 words bleibt startet es ganz 
normal, die I/O-Pins werden als Ausgänge gesetzt und das Programm läuft 
die Bildinformationen ab.
Überschreite ich jezt die tolle 4250er Marke, macht der PIC einfach 
garnichts mehr keine I/O-Pin belegung einfach garnichts, es wäre als 
würde ich einen leeren PIC in den sockel stecken (habe den PIC 
ausgelesen er ist beschrieben).

Mir kommt das einfach Spanisch vor wenn alles geht und eine Zeile Code 
alles futsch macht.

mfg Hebel Zieher

[Purzel H. (hacky)]

Das Bild ist im flash ? Hat's denn ueberhaupt Platz ? Aendert der 
zugriff ?

[Julian B. (julinho)]

Reicht der RAM aus?

[Peter D. (peda)]

Hebel Zieher schrieb:
> unsigned char LED[37];

Hebel Zieher schrieb:
> void Rendern(unsigned char LED[37]) {

Auch wenn es prinzipiell nicht verboten ist, man nennt globale und 
lokale Variablen nicht gleich.
Die lokale Definition versteckt die globale.

Und Großschreibung interpretiert der erfahrene Programmierer nicht als 
Variable, sondern als Macro.


Peter