Speicher des Microcontrollers

[Nase1987 (Gast)]

Nabend zusammen,
ich bin gerade dabei mir LED Equipment zum Jonglieren zu programmieren. 
Also eigentlich relativ simpel 2 RGB LED´s und 4 Taster.

Einlesen und Ausgeben klappt alles. Nur nachdem ich jetzt 2h 
Programmiert habe funtzt es plötzlich nicht mehr. Meine Vermutung ist 
das ich den Speicher des ATMega16 voll habe.

Woran kann ich das erkennen das er voll ist?

Gruß
Nase

P.S: Ich porgrammiere mit dem AVR Studio und PonyProg über die Serielle 
Schnittstelle meines PC´s

[Timmo H. (masterfx)]

Bei AVR Studio siehst du nach dem Kompilieren normalerweise eine 
Zusammenfassung wieviel Flash und RAM verwendet wird und dazu noch die 
Prozentuale Angabe bezogen auf den verfügbaren Speicher.
Solange alles unter 100% ist, ist es ok. Jedoch ist die RAM-Nutzung ohne 
Stacknutzung. Das heisst es kann je nach Programm dennoch passieren, 
dass der Stack überläuft und dir andere Werte im RAM zerstört.

[Thomas B. (nichtessbar)]

Welcher Speicher? Programmspeicher? Datenspeicher? Verwendest du 
mallocs?

Beim Compilieren wird dir im Ausgabefenster des AVR-Studio die Größe des 
compilierten Programms angegeben (Sowohl Programmspeicherbedarf als auch 
Datenspeicherbedarf)

[Timmo H. (masterfx)]

Hier auch noch etwas dazu: 
http://www.mikrocontroller.net/articles/AVR-GCC-Codeoptimierung

[Fabian B. (nase1987)]

Besten dank für die Hinweise. Habe es zwar nicht gefunden wo es stehen 
soll, musste aber feststellen das meine Programmierweise sehr 
Speicherschluckend war. Über nacht alles nochmal umgeschrieben und 
mithilfe von 2 Variablen ca. auf 1/10 gebracht, jetzt passt alles drauf 
^^

[Thomas B. (nichtessbar)]

Mit deinem Vorhaben solltest du weit weg davon sein, die 16k Flash 
(Programmspeicher) und das RAM des Mega16 auch nur ansatzweise 
vollzubekommen. Was hast du denn für Optimierung gewählt? 
(Projektoptionen)

Ich glaub irgendwie dass du in deinem Workflow den Hund begraben hast... 
Aufgrund dessen dass du nicht weißt wie die nachschaun kannst wie viel 
Speicher von was benötigt wird denk ich nicht dass du mit Sicherheit auf 
ein Speicherproblem schließen kannst ;)

Ähm, wie meinst du das, mit 2 Variablen?!

[Fabian B. (nase1987)]

So, habe gerade festgestellt das ich noch mit dem AVR Studio4 arbeite.. 
nachdem ich aber bei AVR5.1 nach kopieren meines Codes fehler bekommen 
habe das angeblich die "delay.h" Bibliothek fehler hat, werde ich wohl 
doch bei AVR 4 bleiben ^^

Ok, wie ich jetzt festgestelklt habe, habe ich keine Optimierung 
verwendet. Also O0..

Mit dem programmspeicher voll, denke ich weil man es bei Ponyprog auch 
sehen kann. Je nachdem welchen Mikrokontroller man einstellt entspricht 
ja die ANzahl an zeilen, die dargestellt werden (was ja die einzelnen 
Bytes sind), wenn noch Bytes frei sin ("FF") in der letzten Zeil, dann 
ist das Programm nicht zu lang ^^

Also mit 2 Variablen meine ich folgendes:

Anfangs habe ich die Ports für die RGB einzeln an und ausgeschlatet

bsp:

 PORTB |= 0x09

hier musste ich aber jede Kombination einzeln porgrammieren ^^

jetzt habe ich 2 variablen eingeführt

mode_o für die 1. RGB (die ersten 3 Pins PB0-PB2)
mode_u für die 2. RGB (4-6 Pin also PB3-PB5)

mode_o wurde durch taster1 um +1 erweitern
mode_u durch taster2 um +8 erweitert

a = mode_o +mode_u

und dann halt

 PORTB |= a

somit kurzer Programmierungstext, und trotzdessen beide LED´s unabhängig 
voneinander einstellbar

EDIT: Gerade mit mit Os Optimiert ausgeführt... ca die hälfte des 
vorrigen Speicher (laut Ponyprog)

[Thomas B. (nichtessbar)]

wie viel Speicher brauchst du denn jetzt? Hab so das Gefühl du weißt 
nicht so wirklich was du da mit was tust und welches Programm was 
erzeugt.

In 16k Programmspeicher solltest du einige tausend Zeilen Code 
unterbringen. So ein PORTB |= 0x01 braucht ein paar Byte im Speicher, du 
hast aber 16kB davon.... Poste mal deinen Code... oder den Output 
(unterstes Fenster im AVR-Studio, da wo sich was rührt wenn du auf 
übersetzen klickst) anschaun.

Und die delay.h ist nicht kaputt sondern arbeitet ohne Optimierung und 
definiertes F_CPU Symbol nicht (richtig)

Was verwendest du für einen Programmer (Hardware)?

[Thomas B. (nichtessbar)]

Noch mehr würd dein Mapfile evtl. Listfile helfen, aber ich denke du 
weißt nicht was das ist oder? Könntest bei den Projektoptionen 
einstellen dass das erzeugt wird.

Denke jetzt - ohne dass ich deinen Fehler weiß - du solltest dich ein 
bisschen mit den Grundlagen von Mikroprozessoren, Entwicklungstools und 
Compilation Toolchain auseinandersetzen... (AVR-Tutorial hier ist gut 
geeignet dafür..)

[Frank M. (ukw) (Moderator)]

Fabian Bade schrieb:

> Ok, wie ich jetzt festgestelklt habe, habe ich keine Optimierung
> verwendet. Also O0..

Benutze -Os. Dann klappt das auch mit delay.h. Ausserdem wird der vom 
Compiler erzeugte Code auf ca. die Hälfte schrumpfen.

[Fabian B. (nase1987)]

Ich benutze das Atmel Evaluationboard V2.01

Der Code

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


int mode_o = 1; //Variable fuer obere LED - Farbmodus
int mode_u = 8; //Variable fuer untere LED - Farbmodus

int blink = 0; //Variable fuer den Blinkmodus

int fest = 0;  //Variable für Zuweisung welche LED Blinkt

int n1 =10; //Variable fuer An1
int m1 =0; //Variable fuer Pause1

int n2 =10; //Variable fuer An2
int m2 =0; //Variable fuer Pause2

int b= 9;  //Durchlaufvariable für Farbverlauf

int a;

int main(void)
{  
  DDRD = 0x00;
    DDRB = (1<<DDB0) | (1<< DDB1) | (1 << DDB2) | (1<<DDB3) | (1<< DDB4) | (1 << DDB5);  

  PORTB &= 0x00;
  
  while (1)
    {

      //loop_until_bit_is_set(PIND, PD4);

      if((PIND & 0x20)) //Mode_o - taster 4
      {
        PORTB &= 0x00;
        _delay_ms(500);
        mode_o += 1;
        if (mode_o >= 9)  
        {
          mode_o = 1;
        }
  
    
      }

      if ((PIND & 0x04)) //Mode_u - taster 1
      { 
          PORTB &= 0x00;
          _delay_ms(500);
          mode_u += 8;
          if(mode_u > 64)
          {
            mode_u = 8;
          }
      }

      if((PIND & 0x08))  //Festlegung Blink -Taster2
      {
        PORTB &= 0x00;
        _delay_ms(500);
        blink += 1;
        if(blink >= 6)
        {  blink = 0;    }

        if ( blink ==0)
        {  n1 =6;
          m1 =3;
          n2 =6;
          m2 =3;
                }
        if (blink ==1)
        {  n1=100;
          m1=50;
          n2 =100;
          m2 =50;
                }
        if (blink ==2)
        {  n1=200;
          m1=200;
          n2 =200;
          m2 =200;
                }
        if (blink ==3)
        {  n1=200;
          m1=300;
          n2 =100;
          m2 =300;
                }    
        if (blink ==4)
        {  n1=500;
          m1=200;
          n2 =200;
          m2 =500;
                }
        if (blink ==5)
        {  n1=1000;
          m1=500;
          n2 =1000;
          m2 =500;
                }
      }


      if((PIND & 0x10))  //Festlegung Fest - Taster 3
      {
          PORTB &= 0x00;
          _delay_ms(400);
          fest += 1;
          if(fest >= 3)
          {
            fest = 0;
          }
      }
    
      a = mode_o +mode_u;
  
//_______________________________________________________________
    if( mode_u <=63)
    {  
      if (mode_o <=7)
      {
        if (fest ==0) //beide das gleiche Blinken
        {   PORTB |= a;
          _delay_ms(n1);
          PORTB &= 0;
          _delay_ms(m1);
          PORTB |= a;
          _delay_ms(n2);
          PORTB &= 0;
          _delay_ms(m2);
        }

        if (fest ==1) //obere blinkt
        {  PORTB |= a;
          _delay_ms(n1);
          PORTB &= mode_o;
          _delay_ms(m1);
            PORTB |= a;
          _delay_ms(n2);
          PORTB &= mode_o;
          _delay_ms(m2);
        }

        if (fest ==2) //untere blinkt
        {  PORTB |= a;
          _delay_ms(n1);
          PORTB &= mode_u;
          _delay_ms(m1);
          PORTB |= a;
          _delay_ms(n2);
          PORTB &= mode_u;
          _delay_ms(m2);
        }
      }
    }
//___________________________________________________________________

    if (mode_o >=8) //obere durchlaufen
    {  if (mode_u <=56) // obere durchlaufen /untere dauerfarbe
      {
        b = 1 + mode_u;
        while (b<=6+mode_u)
        {
          PORTB |= b;
          _delay_ms(n1);
          PORTB &= 0;
          _delay_ms (m1);
          b += 1;
  
          PORTB |= b;
          _delay_ms(n2);
          PORTB &= 0;
          _delay_ms (m2);
          b += 1;
        }
      }
    }

    if (mode_u == 64) //untere durchlaufend
    {
      if (mode_o >= 8) // beide durchlaufend
      {
        b = 9;
        while (b<= mode_o+ 56)
        {   
          PORTB |= b;
          _delay_ms(n1);
          PORTB &= 0;
          _delay_ms (m1);
          b += 9;
      
          PORTB |= b;
          _delay_ms(n2);
          PORTB &= 0;
          _delay_ms (m2);
          b += 9;
        }
      }
      if (mode_o <=7)    //untere durchlaufen / obere dauerfarbe
      {
        b = mode_o + 8;
        while (b<=mode_o + 56)
        {
          PORTB |= b;
          _delay_ms(n1);
          PORTB &= 0;
          PORTB |= mode_o;
          _delay_ms (m1);
          b += 8;
      
          PORTB |= b;
          _delay_ms(n2);
          PORTB &= 0;
          PORTB |= mode_o;
          _delay_ms (m2);
          b += 8;
        }
      }

      
    }
  }
  
while (1);  

return 0;
}

und ja ihr habt beide recht ^^ ... habe mapefile und listfile mit 
erstellt... wo finde ich die denn jetzt??? bzw. welche endung haben 
sie...

habe halt erst angefangen mit microcontroller programmieren... hatte 
gehofft das mit mein 1 Semester einführung in C++ etwas weiterhilft, 
aber das war eher ein trugschluss :D

Gruß
Fabian

P.S: Danke für die Hilfe

[oldmax (Gast)]

Hi
Also, wenn das bisschen Code es schafft, den Speicher von einem Atmega16 
vollzurammeln, dann werd ich sofort vergessen, das ich mit dem Gedanken 
gespielt hab, ein wenig C zu lernen.....
Da läuft etwas anderes schief. Könnte es sein, das hier Bibliotheken 
eingebunden werden, die nicht erforderlich sind? Selbst dann ist es mir 
ein Rätsel, wie mit so wenig Code ein Speicher voll werden kann.
Gruß oldmax

[Thomas E. (thomase)]

Fabian Bade schrieb:
> _delay_ms (m2);
Das geht gar nicht. _delay_ms darf nur mit Konstanten benutzt werden.

Und dann poste deinen Code mal als Anhang.
Kompilierbar, 0 Errors, 0 Warnings.

mfg.

[Oliver (Gast)]

>c:\programme\atmel\avr tools\avr 
>toolchain\bin\../lib/gcc/avr/4.4.3/../../../../avr/include/util/delay.h :12>0: 
error: __builtin_avr_delay_cycles expects an integer constant.

Man sollte die Fehlermeldungen des Compilers schon ernst nehmen.

delay geht NUR mit einer Konstante, ansonsten explodiert dein 
Speicher...

Oliver

[MWS (Gast)]

Thomas Eckmann schrieb:
> Das geht gar nicht. _delay_ms darf nur mit Konstanten benutzt werden.

Das ist falsch. Richtig wäre "sollte".

[Thomas E. (thomase)]

MWS schrieb:
> Das ist falsch. Richtig wäre "sollte".
Richtig ist, daß dein Kommentar vollkommen überflüssig ist.

mfg.

[MWS (Gast)]

Thomas Eckmann schrieb:
> Einzig richtig ist, daß dein Kommentar vollkommen überflüssig ist.

Nö, ist er nicht. Woher sollst Du sonst wissen, wie's richtig ist ? :D
Ja, stimmt, lesen der Doku würd' helfen.

[Fabian B. (nase1987)]

Also liegt der ausgenutzte Speicher nur daran das ich im _delay_ms eine 
variable verwende, anstatt eine Konstante ?!? Weil laufen tut das 
Programm ja, nur das ich ( vor dem anschalten der optimierung) zu wenig 
speicherplatz hatte... was sich jetzt ja auch erledigt hat  dank eurer 
hilfe

[Peter II (Gast)]

Fabian Bade schrieb:
> Weil laufen tut das
> Programm ja

aber vermutlich nicht mit den richtigen Zeiten. Die wartenzeiten werden 
damit nicht eingehalten.

schreibt dir selber eine funktion wie:

void delay_ms( ms unsigned int ) {
   while ( ms-- )
      _delay_ms( 1 )
}

dann kann du sie mit einer variable aufrufen, und dein Programm wird 
noch mal viel kleiner. Der Nachteil der dadurch ensteht ist das das 
delay_ms immer etwas (ein paar µs ) länger derauert als vorgegeben, 
sollte aber bei dir egal sein.

[Thomas E. (thomase)]

MWS schrieb:
> Ja, stimmt, lesen der Doku würd' helfen.
Deine Kommentare werden leider immer überflüssiger.

Hilfreich wäre es aber, die aktuellen Tools zu benutzen, die bei 
Verwendung einer Varaiablen eine Fehlermeldung ausgeben:

:28: error: __builtin_avr_delay_cycles expects an integer constant.

mfg.

[MWS (Gast)]

Thomas Eckmann schrieb:
> Hilfreich wäre es aber die aktuellen Tools zu benutzen, die bei
> Verwendung einer Varaiablen eine Fehlermeldung ausgeben:

Ach, wenn Du etwas weniger eingebildet wärst und dem TO erklärt hättest, 
warum der von ihm compilierbare Code nicht gehen soll, das wäre 
hilfreich gewesen.

Und wenn Du schon meinst, er solle aktuelle Tools benutzen, warum 
schreibst Du ihm das dann nicht einfach ?

Dicke Hose machen und nix dahinter.

Tatsächlich war der Code für den TO compilierbar und die variable 
Zeitangabe hat ihm die Fließkommalibrary mit reingewurstelt und seinen 
Code aufgebläht. Und dass der TO jetzt nicht der Profi ist, ohne das 
abwertend zu meinen, sieht man doch bereits an der Codegestaltung. Wie 
soll er da die Problematik des als Variablen übergebenen Parameters in 
der _delay_ms kennen ?

Nimm Dir 'n Beispiel an Peter II, der hat wenigstens was konstruktives 
gepostet.

[Thomas E. (thomase)]

MWS schrieb:
> Dicke Hose machen und nix dahinter.
Du läufst ja richtig zur Hochform auf.

MWS schrieb:
> Das ist falsch. Richtig wäre "sollte".
Vergiss' aber bitte nicht, daß du mit dieser kleinkarierten Kacke 
angefangen hast.

mfg.

[MWS (Gast)]

Thomas Eckmann schrieb:
> Vergiss' aber bitte nicht, daß du mit dieser kleinkarierten Kacke
> angefangen hast.

Nö, ist doch nicht kleinkariert.
Kleinkariert ist's dem TO zu sagen "darf nicht" und zu verschweigen 
warum.

Denn in der Version des TO ging's offensichtlich, da würd' ich mich an 
dessen Stelle auch fragen, "Wieso? Geht doch"

Also wurdest Du doch nur so behandelt, wie Du andere behandelt hast. Und 
das ist ok so.