Variablen im Flash (PROGMEM) übergeben

[Matthias F. (frank91)]

Hallo alle miteinander :P

Ich arbeite gerade zum ersten mal mit PROGMEM.
Hier komme ich an einer stelle nicht weiter.

Bild_Fahrrad.c

const uint8_t cYCLO_Bitmaps[] PROGMEM=
{
0b11111111, 0b11111111... //wurde zum Verständnis gekürzt.
}

const uint8_t cYCLO_WidthPixels PROGMEM= 128;
const uint8_t cYCLO_HeightPixels PROGMEM= 64;

Bild_Fahrrad.h

extern const uint8_t cYCLO_Bitmaps[];
extern const uint8_t cYCLO_WidthPixels;
extern const uint8_t cYCLO_HeightPixels;

main.c

grafik_LCD_bild(0, 0, cYCLO_Bitmaps, cYCLO_WidthPixels, cYCLO_HeightPixels);

grafik_lcd.c

void grafik_LCD_bild (uint8_t x, uint8_t y, const uint8_t bitmap[], const uint8_t widthPixels, const uint8_t heightPixels)
{
lcd_string(dtostrf(pgm_read_byte(&cYCLO_WidthPixels), 3, 0, wert)); //128. Also Richtig.
  
  lcd_string(dtostrf(pgm_read_byte(&cYCLO_HeightPixels), 3, 0, wert)); //64. Also Richtig.


  lcd_string(dtostrf(pgm_read_byte(&widthPixels), 3, 0, wert)); //ergibt 255. Also falsch.
  lcd_string(dtostrf(pgm_read_byte(&heightPixels), 3, 0, wert)); //ergibt 255. Also falsch.
}

Wieso kann ich in "grafik_LCD_bild" nicht widthPixels und heightPixels 
auslesen?
Verwende ich im selben Unterprogramm "pgm_read_byte (&bitmap[zahler])" 
kann ich das Array erfolgreich auslesen.

Mir ist bewusst, dass es nicht viel Sinn macht zwei einzelne kleinen 
Variablen im Flash abzulegen. Da sie aber zu "cYCLO_Bitmaps[] PROGMEM" 
gehören hätte ich aber gerne die 2 zugehörigen Variablen ebenfalls im 
Flash.

Was mache ich hier falsch?

[Falk B. (falk)]

@ Matthias Frank (frank91)

>void grafik_LCD_bild (uint8_t x, uint8_t y, const uint8_t bitmap[], const 
>uint8_t widthPixels, const uint8_t heightPixels)


>{
>lcd_string(dtostrf(pgm_read_byte(&cYCLO_WidthPixels), 3, 0, wert)); >//128. Also 
Richtig.

>  lcd_string(dtostrf(pgm_read_byte(&cYCLO_HeightPixels), 3, 0, wert)); >//64. 
Also Richtig.

Direkter Zugriff. Passt.


>  lcd_string(dtostrf(pgm_read_byte(&widthPixels), 3, 0, wert)); //ergibt >255. 
Also falsch.
>  lcd_string(dtostrf(pgm_read_byte(&heightPixels), 3, 0, wert)); //ergibt >255. 
Also falsch.
>}

Geht so nicht. Das muss man anders machen. Entweder mit der Erweiterung 
__flash oder Old school mit PGM_P pointern.

https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Flash_mit_flash_und_Embedded-C

progmem Version


main.c

grafik_LCD_bild(0, 0, cYCLO_Bitmaps, &cYCLO_WidthPixels, &cYCLO_HeightPixels);

grafik_lcd.c

void grafik_LCD_bild (uint8_t x, uint8_t y, PGM_P bitmap, PGM_P widthPixels, PGM_P heightPixels)
{
  lcd_string(dtostrf(pgm_read_byte(widthPixels), 3, 0, wert));
  lcd_string(dtostrf(pgm_read_byte(heightPixels), 3, 0, wert));
}

__flash Version


Bild_Fahrrad.c

const __flash uint8_t cYCLO_Bitmaps[] PROGMEM=
{
0b11111111, 0b11111111... //wurde zum Verständnis gekürzt.
}

const __flash uint8_t cYCLO_WidthPixels PROGMEM= 128;
const __flash uint8_t cYCLO_HeightPixels PROGMEM= 64;

Bild_Fahrrad.h

extern const __flash uint8_t cYCLO_Bitmaps[];
extern const __flash uint8_t cYCLO_WidthPixels;
extern const __flash uint8_t cYCLO_HeightPixels;

main.c

grafik_LCD_bild(0, 0, cYCLO_Bitmaps, cYCLO_WidthPixels, cYCLO_HeightPixels);

grafik_lcd.c

void grafik_LCD_bild (uint8_t x, uint8_t y, const __flash uint8_t *bitmap, uint8_t widthPixels, uint8_t heightPixels)
{
  lcd_string(dtostrf(widthPixels), 3, 0, wert));
  lcd_string(dtostrf(heightPixels), 3, 0, wert));
}

[Johann L. (gjlayde)]

Falk B. schrieb:
> __flash Version
> const __flash uint8_t cYCLO_Bitmaps[] PROGMEM=

PROGMEM ist da überflüssig und verwirrend.

[Falk B. (falk)]

Mist, Copy&Paste Fehler

[Falk B. (falk)]

Hier nochmal korrigiert.


__flash Version


Bild_Fahrrad.c

const __flash uint8_t cYCLO_Bitmaps[]=
{
0b11111111, 0b11111111... //wurde zum Verständnis gekürzt.
}

const __flash uint8_t cYCLO_WidthPixels = 128;
const __flash uint8_t cYCLO_HeightPixels = 64;

Bild_Fahrrad.h

extern const __flash uint8_t cYCLO_Bitmaps[];
extern const __flash uint8_t cYCLO_WidthPixels;
extern const __flash uint8_t cYCLO_HeightPixels;

main.c

grafik_LCD_bild(0, 0, cYCLO_Bitmaps, cYCLO_WidthPixels, cYCLO_HeightPixels);

grafik_lcd.c

void grafik_LCD_bild (uint8_t x, uint8_t y, const __flash uint8_t *bitmap, uint8_t widthPixels, uint8_t heightPixels)
{
  lcd_string(dtostrf(widthPixels), 3, 0, wert));
  lcd_string(dtostrf(heightPixels), 3, 0, wert));
  lcd_string(dtostrf(bitmap[0]), 3, 0, wert));
}

[Matthias F. (frank91)]

Ich hab das mit dem flash jetzt ausprobiert, weil ich dann das 
pgm_read_byte nicht mehr benötige und es das ganze übersichtlicher 
macht.

Das ganze hat dank deiner Hilfe sofort funktioniert :P
Danke :-D

Bei:

void grafik_LCD_bild (uint8_t x, uint8_t y, const __flash uint8_t *bitmap, uint8_t widthPixels, uint8_t heightPixels)

hast du das __flash nur bei dem Array mit angegeben. Brauche ich das bei 
widthPixels und heightPixels nicht?

[Falk B. (falk)]

@ Matthias Frank (frank91)

>Bei:

>void grafik_LCD_bild (uint8_t x, uint8_t y, const __flash uint8_t *bitmap, 
>uint8_t widthPixels, uint8_t heightPixels)

>hast du das __flash nur bei dem Array mit angegeben. Brauche ich das bei
>widthPixels und heightPixels nicht?

Nein, weil beim Funktionsaufruf die __flash Variablen gelesen und als 
normale Parameter übergeben werden. Die Arbeit des Flash-Zugriffs 
geschieht dort. Beim Pointer ist das anders, dort muss ja auch innerhalb 
der Funktion bekannt sein, dass es kein normaler Pointer ist sondern ein 
__flash Pointer.

[Matthias F. (frank91)]

ah ok, ich hab es verstanden :P

Danke :-D