Fehler in For-Schleife mit I2C Kommunikation

[uffi]

Ja, super Idee. Allerdings sagt der Compiler:

Size after:
allrounder.elf :
section size addr
.text 4236 0
.data 570 8388704
.bss 14 8389274
.eeprom 512 8454144
.stab 1440 0
.stabstr 183 0
.debug_aranges 64 0
.debug_pubnames 531 0
.debug_info 2184 0
.debug_abbrev 838 0
.debug_line 3084 0
.debug_frame 432 0
.debug_str 801 0
.debug_loc 1141 0
Total 16030

Sehe ich das richtig, dass der Compiler von den 1kByte SRAM 570 Bytes für die Variablen reserviert?
Dann müßte der Stack aber schon sehr lang werden, damit das die Ursache für den Fehler sein kann...

Oder liege ich da falsch?

Danke und Gruß, uffi.

[sternst]

Sehe ich das richtig, dass der Compiler von den 1kByte SRAM 570 Bytes für die Variablen reserviert?

Nein, 584 Bytes. Und das sind nur die statischen Variablen.

Dann müßte der Stack aber schon sehr lang werden, damit das die Ursache für den Fehler sein kann...

Ja, wird er ja aber auch. Allein mit der Variable "distRAM" belegst du 512 Bytes auf dem Stack. Und schon bist du über den 1024 Bytes.


Da du ja wohl kaum die Funktion main rekursiv aufrufen willst, solltest du als erstes mal die lokalen Variablen dort static machen. Dann sind sie vom Stack runter und tauchen mit in der Auflistung vom Compiler auf. Das gibt dir etwas mehr Überblick. Dann reduzierst du den RAM-Verbrauch, indem du alle konstanten Strings ins Flash verbannst. Und dann sehen wir weiter.

Und mal so nebenbei:
Dieses SREG-Gefummel in den ISRs ist völlig überflüssig. Darum kümmert sich der Compiler.

[uffi]

Vielen Dank sternst!!!

Thema ist damit gelöst.

Ich habe mit dem kleinen Program avrstackview und meiner .elf Datei den Stack-Verbrauch berechnen lassen. Hier das Ergebnis:

Analysis complete: 0 errors, 0 warnings
flash usage (words): 2118
global RAM usage (bytes): 1096
maximum func. subroutine levels: 6
maximum func. stack (bytes): 554
maximum int. subroutine levels: 1
maximum int. stack (bytes): 9
nested interrupts: 0
maximum nest. int. stack (bytes): 0
maximum total stack (bytes): 563
maximum total subroutine levels: 7
maximum total RAM usage (bytes): 1659

Ich benutze also 1659 Bytes RAM und habe nur 1024.

Vielen, vielen Dank nochmal, das ist die Lösung meines Problems!!!

[uffi]

Uups, leider zu früh gefreut.

Hab jetzt den Code nach Stefan's Hinweisen modifiziert und das Stack-Problem beseitigt. Die Simulation mit avrstackview bringt nun folgendes Ergebnis:

Analysis complete: 0 errors, 0 warnings
flash usage (words): 2155
global RAM usage (bytes): 609
maximum func. subroutine levels: 6
maximum func. stack (bytes): 35
maximum int. subroutine levels: 1
maximum int. stack (bytes): 8
nested interrupts: 0
maximum nest. int. stack (bytes): 0
maximum total stack (bytes): 43
maximum total subroutine levels: 7
maximum total RAM usage (bytes): 652

Also o.k.

Jedoch zeigt sich nun folgendes Fehlerbild:
Nur noch der erste Sensor (Adresse E0) wird korrekt gelesen. Der zweite mit Adresse E2 wird auch noch gelesen, aber der Wert 02 kann nicht sein (so nahe Werte kann der US Sensor nicht ausgeben), auf dem LCD wird dieser Wert auch nicht mehr ausgegeben, sondern abrupt die FOR-Schleife vorzeitig beendet.
Dann wird das andere Sensorboard korrekt ausgelesen (i2c Adresse 0xD0).

Hat jemand noch eine Idee?
Anbei nochmal alle Sourcen, Daten und ein Bild vom Logic Analyzer gesammelt im File "Problem.zip".

Danke, Dirk.

P.S. Frage noch an Stefan: welches sind string Konstanten in meinem Code?
P.P.S. Wenn ich die Schleife auflöse und alle 3 US-Sensoren einzeln abfrage, funktioniert alles!!! Es muß also an dieser verflixten FOR-Schleife liegen, so sieht sie jetzt aus:

  // select display
  lcd_select(0);     
  
  for ( i = 0 ; i < 3 ; i++ )
  {
  // check if ultrasonic sensor is busy
   busy_US[i] = i2c_start(US_left + (2*i) + I2C_READ);
   
   // address I2C device ultrasonic sensor with read access
   if (busy_US[i] == 0)
   {
    distance[i] = i2c_readNak(); // read one byte
    i2c_stop(); // release I2C bus
    
    utoa(distance[i], buffer, 10);
    
    // move cursor to position on line 0
    lcd_gotoxy(10+(4*i),0);  
    // put string to display (line 0)
    lcd_puts(buffer);
    lcd_puts("  ");
    
    if (i==2)
    {
     i2c_start(US_general + I2C_WRITE);
     // address I2C device ultrasonic sensor with write access
     i2c_write(Start_US); //start new ultrasonic measurement
     i2c_stop(); // release I2C bus
    }
    
   }
   
   else
   {
    i2c_stop(); // release I2C bus
   }
  }

[sternst]

Wie groß können denn die gelieferten Werte werden? Bestimmt doch auch größer als 99, oder? In dem Fall ist buffer zu klein. Für eine dreistellige Zahl muss buffer die Größe 4 haben (drei Ziffern plus Null-Terminierung).

P.S. Frage noch an Stefan: welches sind string Konstanten in meinem Code?

All die Strings in Anführungszeichen, z.B.

lcd_puts("Sensor Board 2 Results: ");

Die belegen aktuell auch Platz im RAM, was aber nicht nötig wäre.

[uffi]

ja, Du hast Recht, bei buffer kommen Zahlen bis 3-stellig vor:

es muß heißen static char buffer(4);

Komischerweise hat das aber bisher keinen Fehler gegeben, die LCD Anzeige ist korrekt mit 3 Ziffern. Kann das für den Schleifenabbruch verantwortlich sein?

Vielen Dank für Deine Hilfe!

[sternst]

Komischerweise hat das aber bisher keinen Fehler gegeben, die LCD Anzeige ist korrekt mit 3 Ziffern.

Klar, für die Ausgabe ist ja primär auch nur wichtig an der entsprechenden Adresse die entsprechenden Daten zu finden. Dass bei der Erstellung der Daten andere Variablen zerstört wurden interessiert da ja erst mal nicht.

Kann das für den Schleifenabbruch verantwortlich sein?

Du überschreibst dort eine andere Variable (außer du hast das Glück, dass buffer die letzte Variable im Speicher ist). Das kann alle möglichen Auswirkungen haben. Wie wäre es einfach mit Ausprobieren?