Casting: was mache ich falsch?

[Maxim B. (max182)]

Liebes Forum,
ich habe eine Frage:
ich möchte aus Array in Flash Array in RAM füllen. Dabei ist Flash-Array 
als char und Ram-Array als int. Code:

static const signed char ged_16[512] PROGMEM = {
0,2,4,6,8,10,12,14,16,18,20,21,23,25,27,28,
30,31,33,34,36,37,38,39,40,41,42,43,44,45,45,46,
47,47,48,48,48,49,49,49,49,50,50,50,50,50,50,50,
50,50,50,50,50,50,50,50,50,49,49,49,49,49,49,49,
49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,
50,50,50,50,50,50,50,50,50,50,50,50,50,50,49,49,
49,49,49,49,49,49,49,48,48,48,48,48,48,47,47,47,
47,47,47,46,46,46,46,46,46,46,46,45,45,45,45,45,
45,45,45,45,45,45,46,46,46,46,46,46,46,46,47,47,
47,47,47,47,48,48,48,48,48,48,49,49,49,49,49,49,
49,49,49,50,50,50,50,50,50,50,50,50,50,50,50,50,
50,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,
49,49,49,49,49,49,49,49,50,50,50,50,50,50,50,50,
50,50,50,50,50,50,50,50,49,49,49,49,48,48,48,47,
47,46,45,45,44,43,42,41,40,39,38,37,36,34,33,31,
30,28,27,25,23,21,20,18,16,14,12,10,8,6,4,2,
0,-2,-4,-6,-8,-10,-12,-14,-16,-18,-20,-21,-23,-25,-27,-28,
-30,-31,-33,-34,-36,-37,-38,-39,-40,-41,-42,-43,-44,-45,-45,-46,
-47,-47,-48,-48,-48,-49,-49,-49,-49,-50,-50,-50,-50,-50,-50,-50,
-50,-50,-50,-50,-50,-50,-50,-50,-50,-49,-49,-49,-49,-49,-49,-49,
-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,
-50,-50,-50,-50,-50,-50,-50,-50,-50,-50,-50,-50,-50,-50,-49,-49,
-49,-49,-49,-49,-49,-49,-49,-48,-48,-48,-48,-48,-48,-47,-47,-47,
-47,-47,-47,-46,-46,-46,-46,-46,-46,-46,-46,-45,-45,-45,-45,-45,
-45,-45,-45,-45,-45,-45,-46,-46,-46,-46,-46,-46,-46,-46,-47,-47,
-47,-47,-47,-47,-48,-48,-48,-48,-48,-48,-49,-49,-49,-49,-49,-49,
-49,-49,-49,-50,-50,-50,-50,-50,-50,-50,-50,-50,-50,-50,-50,-50,
-50,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,-49,
-49,-49,-49,-49,-49,-49,-49,-49,-50,-50,-50,-50,-50,-50,-50,-50,
-50,-50,-50,-50,-50,-50,-50,-50,-49,-49,-49,-49,-48,-48,-48,-47,
-47,-46,-45,-45,-44,-43,-42,-41,-40,-39,-38,-37,-36,-34,-33,-31,
-30,-28,-27,-25,-23,-21,-20,-18,-16,-14,-12,-10,-8,-6,-4,-2
};

int tabform[512] = {0};

        int *y;
  y = tabform;
  for(int x=0;x<512;x++){
    *y += (int)pgm_read_byte (&ged_16[x]);
    y++;  
  }

Ergebnis: solange ich zu negativen Zahlen komme, steht in Array statt 
wie erwartet -2, -4, -6 ( 0xFFFE, 0xFFFC, 0xFFFA ) einfach 254, 252, 250 
( 0x00FE, 0x00FC, 0x00FA ).
D.h. Problem liegt in casting. Aber wo ich Fehler mache, konnte ich 
nicht capieren. Ich habe schon versucht, casting in 2 Schritten zu 
machen, zuerst eine Zwischenvariable als signed char, dann eine 
Zwischenvariable als int, erst danach in Array - immer gleiche 
Ergebnis...

AVR Studio 4.18 und gcc-5.4.0.

Vielen Dank für die Erklärung im voraus!

[Bernd K. (prof7bit)]

caste das vorzeichenlose byte zuerst mal in ein signed char. Erst dann 
hast Du eine negative Zahl.

[Maxim B. (max182)]

Danke!
Ich habe nun verstanden, wo mein Fehler lag...
Ich habe auch die Variante mit Zwischenvariable char ausprobiert. Aber 
ich ging davon aus, daß char signed ist.

int *y;
  char xx;
  y = tabform;
  for(int x=0;x<512;x++){
    xx = (char)pgm_read_byte (&ged_16[x]);
    *y += (int)xx;
    y++;  
  }

Das war mein Fehler!

Die Variante

  int *y;
  signed char xx;
  y = tabform;
  for(int x=0;x<512;x++){
    xx = (signed char)pgm_read_byte (&ged_16[x]);
    *y += (int)xx;
    y++;  
  }

 arbeitet wie erwartet.

Vielen Dank!

[Bernd K. (prof7bit)]

Maxim B. schrieb:
> int *y;
>     *y += (int)xx;

Du brauchst das xx übrigens danach nicht mehr nochmal explizit nach int 
zu casten. Wenn Du ein signed char und ein int addierst wird die ganze 
Rechnung automatisch mit int ausgeführt, beide Operanden werden zuvor 
automatich nach int promoted oder falls größer als int dann zum größten 
Datentyp der in dem Ausdruck vorkommt, mindestens jedoch zu int oder 
falls alle unsigned sind dann zu unsigned int. Stichwort zum Googeln: 
Integer-Promotion.

Und am Rande, falls Du mal drüberstolperst, behalte auch im Hinterkopf 
daß auf dem AVR ein int 16 Bit lang ist, auf größeren Platformen meist 
32 Bit.

[Johann L. (gjlayde)]

Maxim B. schrieb:

 static const __flash int8_t ged_16[512] = { ... };
  int tabform[512] = {0};
  int *y = tabform;
  for (int x = 0; x < 512; x++)
    {
      *y += ged_16[x];
      y++;
    }

[Besetzungsexperte (Gast)]

> Casting: was mache ich falsch?
Du hast nicht die Couch genutzt.

[Maxim B. (max182)]

Vielen Dank Bernd! Ich muß noch vieles lernen...

[Rolf M. (rmagnus)]

Maxim B. schrieb:
> Danke!
> Ich habe nun verstanden, wo mein Fehler lag...
> Ich habe auch die Variante mit Zwischenvariable char ausprobiert. Aber
> ich ging davon aus, daß char signed ist.

Leider ist es in C nicht wie bei allen anderen Integer-Typen so, dass es 
automatisch signed ist, wenn nichts explizit davor steht. Es ist vom 
Compiler und dessen Einstellungen abhängig. Es gibt dann drei als 
verschieden zu betrachtende Typen: char, signed char und unsigned char. 
Ersteres nimmt man für Text, während man für Binärdaten immer die beiden 
letzteren verwendet.

[fop (Gast)]

Und da unter C noch nicht mal die Anzahl der Bits eines Variablentyps 
festgelegt sind, bindet man noch besser die Datei stdint.h ein und 
benutzt deren Typen.
Bei uint8_t ist dann alles klar.

[Maxim B. (max182)]

fop schrieb:

> Bei uint8_t ist dann alles klar.
Nur bei AVR Studio 4 weniger bequem, weil diese Typen nicht automatisch 
von Programm beim Schreiben blau gemacht werden. Das bringt mehr Gefahr, 
Fehler zu machen. Nur aus diesem Grund bleibe ich bei gewöhnlichen 
Typen.

AVR Studio 4 ist weit von Vollkommenheit. Z.B. hier ist gar nicht 
möglich, in Watch-Fenster die Variablen uint24_t zu beobachten, die bei 
heutigem GCC längst möglich sind. Aber einfache IDE, für AVR 
ausreichend, aus IDE kann man Programm direkt in Mikrocontroller 
schreiben... Bequem...

[Bernd K. (prof7bit)]

fop schrieb:
> Und da unter C noch nicht mal die Anzahl der Bits eines Variablentyps
> festgelegt sind, bindet man noch besser die Datei stdint.h ein und
> benutzt deren Typen.
> Bei uint8_t ist dann alles klar.

Und wenn man schreibfaul ist wie ich macht man es wie die ehrwürdigen 
Kernelhacker und macht sich typedefs nach dem Vorbild in linux/types.h

u8
s8
u16
s16
u32
s32
u64
s64