Array Deklaration Problem

[Nino K. (lnino)]

Hallo an alle,

wenn ich versuche Werte in ein Array zu schreiben, habe ich das Problem, 
dass nicht immer die Werte auch im Array stehen.

Funktioniert NICHT:

char level1[8] = {0, 10, 17, 21, 17, 10, 0, 0};

Funktioniert NICHT:

char level1[] = {0, 10, 17, 21, 17, 10, 0, 0};

Funktioniert:

char level1[8];
 
level1[0]=0; 
level1[1]=10; 
level1[2]=17; 
level1[3]=21; 
level1[4]=17; 
level1[5]=10; 
level1[6]=0; 
level1[7]=0;

Nur bei der letzten Variante befinden sich die Werte auch im Array wenn 
ich diese zum Beispiel am Display ausgebe. Woran könnte das liegen? Die 
anderen Varianten müssten doch auch gehen, oder?

lg Nino

[Timmo H. (masterfx)]

Welcher Compiler? Was bekommst du wenn du dir in einer schleife alle 
Werte ausgibst?

[Nino K. (lnino)]

Ich verwende den GCC. Es steht immer null drin.

[Karl H. (kbuchegg)]

Compilierbares komplettes Beispiel.
Da wir davon ausgehen können, dass jeder C Compiler das Initialisieren 
eines Arrays beherrscht, muss das Problem im Umfeld dieser 
Initialisierung liegen.

[Ingo (Gast)]

An deinem Code ist nichts falsch, der Fehler muss woanders sein.

[Timmo H. (masterfx)]

Also mit GCC 4.7.1 macht folgender Code:

int main()
{
    int i;

    int level1[8] = {20,30,40,50,60,70,80,90};
    int level2[] = {21,31,41,51,61,71,81,91};
    int level3[8];

    level3[0] = 22;
    level3[1] = 32;
    level3[2] = 42;
    level3[3] = 52;
    level3[4] = 62;
    level3[5] = 72;
    level3[6] = 82;
    level3[7] = 92;


    for(i = 0; i < 8; i++)
        printf("Level1[%d]: %d\n", i , level1[i]);
    for(i = 0; i < 8; i++)
        printf("Level2[%d]: %d\n", i , level2[i]);
    for(i = 0; i < 8; i++)
        printf("Level3[%d]: %d\n", i , level3[i]);

    return 0;
}

wie erwartet folgende Ausgabe

Level1[0]: 20
Level1[1]: 30
Level1[2]: 40
Level1[3]: 50
Level1[4]: 60
Level1[5]: 70
Level1[6]: 80
Level1[7]: 90
Level2[0]: 21
Level2[1]: 31
Level2[2]: 41
Level2[3]: 51
Level2[4]: 61
Level2[5]: 71
Level2[6]: 81
Level2[7]: 91
Level3[0]: 22
Level3[1]: 32
Level3[2]: 42
Level3[3]: 52
Level3[4]: 62
Level3[5]: 72
Level3[6]: 82
Level3[7]: 92

[Cyblord -. (cyblord)]

Ingo schrieb:
> An deinem Code ist nichts falsch, der Fehler muss woanders sein.

PEBKAC

[Nino K. (lnino)]

Ich habe nun den Fehler gefunden. War echt gut versteckt.
Im Makefile meines projektes und der libraries hatte ich -Os stehen und 
wie es scheint hat das nach dem Update der AVR Toolchain und WinAVR 
etwas zu viel beschnitten.

Nachdem ich die makefiles auf -O0 geändert habe, funktioniert diese 
Zuweisung wieder.

Echt seltsam. Habe es auch nur durch Zufall gefunden, da ich zeitgleich 
nach dem Update Probleme mit dem Display hatte.

[Karl H. (kbuchegg)]

Nino K. schrieb:

> Nachdem ich die makefiles auf -O0 geändert habe, funktioniert diese
> Zuweisung wieder.

Das ist keine Zuweisung, sondern eine Intialisierung.

Und mit dem Abschalten des Optimizers hast du nur den eigentlichen 
Fehler versteckt aber nicht beseitigt. Drinn ist er immer noch, nur gibt 
es jetzt keinen Optimizer mehr, der ihn gnadenlos ausnutzen würde.

[Nino K. (lnino)]

Hi Karl Heinz,

da muss ich die leider recht geben. Denn in einem anderen Zusammenhang 
funktioniert es wieder nicht. :-(

Dieser Code funktioniert. Da habe ich wieder zeilenweise das Array 
befüllt. Es wird am Display ein Ball angezeigt.

#define F_CPU 14745600
 
#include <stdio.h>
 
#include <avr/io.h>
#include <avr/pgmspace.h>
 
#include <inttypes.h>
#include <string.h>
 
#include <util/delay.h>
#include "../libnerdkits/delay.h"
#include "../libnerdkits/lcd.h"

int main(void) {

  // Local Variables  (The Arrays haven been set to a size of 12 to increase afterwards)

  char level1[8];
  
  level1[0]=0;
  level1[1]=10;
  level1[2]=17;
  level1[3]=21;
  level1[4]=17;
  level1[5]=10;
  level1[6]=0;
  level1[7]=0;

  int i = 0;

  // fire up the LCD
  lcd_init();
  lcd_home();

lcd_set_type_command();     // Change into command mode
lcd_write_byte(0x40);       // Change to CGRAM to store custom character - at 0x00
lcd_set_type_data();        // Change into data mode

for (i=0; i<8; i++)
{
    lcd_write_byte(level1[i]);
}

lcd_set_type_command();     // Change into command mode
lcd_write_byte(0x80);       // Change to DDRAM

lcd_set_type_data();
lcd_write_byte(0x00);       // Show stored custom character on lcd screen

  // busy loop
  while(1) {

  // do nothing
  }

  return 0;
}

Aber der folgende Code funktioniert nicht. Es wird am Display ein 
Kästchen komplett ausgefüllt. Ich habe lediglich die Befüllung des 
Arrays geändert.

#define F_CPU 14745600
 
#include <stdio.h>
 
#include <avr/io.h>
#include <avr/pgmspace.h>
#include <inttypes.h>
 
#include <string.h>
 
#include <util/delay.h>
#include "../libnerdkits/delay.h"
#include "../libnerdkits/lcd.h"

int main(void) {

  // Local Variables  (The Arrays haven been set to a size of 12 to increase afterwards)

  int level1[8] = {0,10,17,21,17,10,0,0};

  int i = 0;

  // fire up the LCD
  lcd_init();
  lcd_home();

lcd_set_type_command();     // Change into command mode
lcd_write_byte(0x40);       // Change to CGRAM to store custom character - at 0x00
lcd_set_type_data();        // Change into data mode

for (i=0; i<8; i++)
{
    lcd_write_byte(level1[i]);
}

lcd_set_type_command();     // Change into command mode
lcd_write_byte(0x80);       // Change to DDRAM

lcd_set_type_data();
lcd_write_byte(0x00);       // Show stored custom character on lcd screen

  // busy loop
  while(1) {

  // do nothing
  }

  return 0;
}

Woran könnte das liegen?

[Stefan E. (sternst)]

Nino K. schrieb:
> Woran könnte das liegen?

An einem Fehler im Build-Process.
Poste das Makefile.

[Dirk B. (dirkbilland)]

>Ich habe lediglich die Befüllung des
Arrays geändert.

Nur ein Hinweis: Du hast mehr geändert: Du hast den Typ von level1 von 
char nach int geändert. Das kann der entscheidene Unterschied sein.

Schau Dir mal eventuelle Warnings an. Auch ein Blick ins Mapping-File 
könnte sachdienliche Hinweise liefern (welche Variablen liegen 
vor/hinter dem betroffenen Array, welcher Code greift auf diese zu, 
usw.)

[Nino K. (lnino)]

Stefan Ernst schrieb:
> An einem Fehler im Build-Process.
> Poste das Makefile.

Anbei mein makefile:

GCCFLAGS=-g -Os -Wall -mmcu=atmega168 
LINKFLAGS=-Wl,-u,vfprintf -lprintf_flt -Wl,-u,vfscanf -lscanf_flt -lm
AVRDUDEFLAGS=-c avr109 -p m168 -b 115200 -P COM8
LINKOBJECTS=../libnerdkits/delay.o ../libnerdkits/lcd.o ../libnerdkits/uart.o

all:  lnino_eval_board-upload

lnino_eval_board.hex:  lnino_eval_board.c
  make -C ../libnerdkits
  avr-gcc ${GCCFLAGS} ${LINKFLAGS} -o lnino_eval_board.o lnino_eval_board.c ${LINKOBJECTS}
  avr-objcopy -j .text -O ihex lnino_eval_board.o lnino_eval_board.hex
  
lnino_eval_board.ass:  lnino_eval_board.hex
  avr-objdump -S -d lnino_eval_board.o > lnino_eval_board.ass
  
lnino_eval_board-upload:  lnino_eval_board.hex
  avrdude ${AVRDUDEFLAGS} -U flash:w:lnino_eval_board.hex:a

[Stefan E. (sternst)]

  avr-objcopy -j .text -O ihex lnino_eval_board.o lnino_eval_board.hex

Da ist die Ursache deines Problems.
Keine data-Section => keine Initialisierungsdaten für das Array

[Nino K. (lnino)]

Stefan Ernst schrieb:
> Keine data-Section => keine Initialisierungsdaten für das Array

Vielen Dank Stefan, das war die Lösung.

Habe nun die Zeile im Makefile angepasst, nun passen die Werte vom 
Array.

avr-objcopy -j .text -j .data -O ihex lnino_eval_board.o lnino_eval_board.hex