Atmega Flash -> PROGMEM -> ARRAY

[Jan H. (janiiix3)]

Moin.

Ich möchte ein Array in den Flash parken.
Das klappt soweit auch. Nur beim auslesen, mache ich was falsch.

const unsigned char hesch [] PROGMEM = {..};

Und hier will ich das Array ausgeben. Leider passt da was überhaupt 
nicht?! Es wird nur Mist angezeigt.
Was mache ich bei der Übergabe falsch?

void lcd_printImage(const unsigned char *image, uint16_t sizeofimage)
{
  uint16_t page  = 0;
  uint16_t column  = 0;
  
  lcd_gotoxy(0,0);
  for (column=0;column<sizeofimage;column++)
  {
    lcd_senddata(swapBits(pgm_read_byte(image[column])));
    if ((column%LCD_WIDTH_SIZE)==0)
    {
      lcd_gotoxy(page++, 0);
      if (page==LCD_PAGE_SIZE) page=0;
    }
  }
}

[M. K. (sylaina)]

Jan H. schrieb:
> lcd_senddata(swapBits(pgm_read_byte(image[column])));

Hier fehlt das "&" vor image ;)

[Jan H. (janiiix3)]

M. K. schrieb:
> Jan H. schrieb:
>> lcd_senddata(swapBits(pgm_read_byte(image[column])));
>
> Hier fehlt das "&" vor image ;)

Und dann normal den Pointer übergeben?

lcd_printImage(pgm_read_ptr(&hesch),sizeof(hesch));

[M. K. (sylaina)]

Ach ne, das ist ja schon nen Pointer. Ich hab vorhin irgendwie den * 
übersehen...wie sieht denn swapBits() aus? Vielleicht passt da was 
nicht.

[Oliver S. (oliverso)]

M. K. schrieb:
> Ach ne, das ist ja schon nen Pointer.

Ist zwar einer, aber du hattest trotzdem recht.
Wenn da image[] gestanden hätte, wärs zwar das gleich, aber klarer.

Oliver

[Jan H. (janiiix3)]

Also wenn ich ohne das PROGMEM arbeite, klappt es. Es muss was an der 
Übergabe sein..

[Arduinoquäler (Gast)]

Jan H. schrieb:
> void lcd_printImage(const unsigned char *image, uint16_t sizeofimage)

Ich vermute mal dass der Pointer auf das Image ein
PROGMEM Typ sein muss.

[Jan H. (janiiix3)]

Arduinoquäler schrieb:
> Jan H. schrieb:
>> void lcd_printImage(const unsigned char *image, uint16_t sizeofimage)
>
> Ich vermute mal dass der Pointer auf das Image ein
> PROGMEM Typ sein muss.

Wie würde das explizit aussehen?

[Rufus Τ. F. (rufus)]

Jan H. schrieb:
> pgm_read_byte(image[column])

Hier wird der Pointer "image" vor dem Aufruf von "pgm_read_byte" 
dereferenziert, und somit der Funktion der (vermeintliche) Inhalt 
übergeben. Es muss aber weiterhin ein Pointer übergeben werden.

Mit

 
  pgm_read_byte(image + column)

bleibt der übergebene Wert ein Pointer, es wird nur der Offset "column" 
dazugerechnet.

Das ist funktional das gleiche wie die Verwendung des Adressoperators

 
  pgm_read_byte(&image[column])

[Axel S. (a-za-z0-9)]

Jan H. schrieb:

...
lcd_gotoxy(page++, 0);

Warum heißt die Funktion lcd_gotoxy() und nicht lcd_gotoyx()?

[Jan H. (janiiix3)]

void lcd_printImage(const unsigned char *image, uint16_t sizeofimage)
{
  uint16_t page  = 0;
  uint16_t column  = 0;
  
  lcd_gotoxy(0,0);
  for (column=0;column<sizeofimage;column++)
  {
    lcd_senddata(swapBits(pgm_read_byte(image+column)));
    if ((column%LCD_WIDTH_SIZE)==0)
    {
      lcd_gotoxy(page++, 0);
      if (page==LCD_PAGE_SIZE) page=0;
    }
  }
}

Und der Aufrufer..

lcd_printImage(pgm_read_ptr(hesch),1024);

Nach wie vor das selbe..

[Arduinoquäler (Gast)]

Arduinoquäler schrieb:
> Ich vermute mal dass der Pointer auf das Image ein
> PROGMEM Typ sein muss.

void lcd_printImage (const unsigned char* PROGMEM image,
                     uint16_t sizeofimage)

Kann es gerade nicht konkret ausprobieren ....

[Jan H. (janiiix3)]

Arduinoquäler schrieb:
> Arduinoquäler schrieb:
>> Ich vermute mal dass der Pointer auf das Image ein
>> PROGMEM Typ sein muss.
>
>

> void lcd_printImage (const unsigned char* PROGMEM image,
>                      uint16_t sizeofimage)
>

>
> Kann es gerade nicht konkret ausprobieren ....

Das mag er nicht.

[Arduinoquäler (Gast)]

Jan H. schrieb:
> Das mag er nicht.

Ist das die Fehlermeldung des Programms oder des Compilers?  ;-)

[Jan H. (janiiix3)]

Arduinoquäler schrieb:
> Jan H. schrieb:
>> Das mag er nicht.
>
> Ist das die Fehlermeldung des Programms oder des Compilers?  ;-)

Compilers.

[Arduinoquäler (Gast)]

Jan H. schrieb:
> Compilers.

... und ich dachte der Compiler gibt nur englisch-
sprachige Fehlermeldungen aus.

[Jan H. (janiiix3)]

Musste mir gerade das lachen verkneifen ;)

[Arduinoquäler (Gast)]

Jan H. schrieb:
> Musste mir gerade das lachen verkneifen

Statt zu lachen könntest du ja auch was Sinnvolleres tun.

z.B. eine reale Fehlermeldung posten.

[Jan H. (janiiix3)]

Arduinoquäler schrieb:
> Jan H. schrieb:
>> Musste mir gerade das lachen verkneifen
>
> Statt zu lachen könntest du ja auch was Sinnvolleres tun.
>
> z.B. eine reale Fehlermeldung posten.

Error    expected ';', ',' or ')'

[Leo C. (rapid)]

Warum sich immer noch mit PROGMEM abrackern? Wie wärs mit

const __flash unsigned char hesch [] = {...};

void lcd_printImage(const __flash unsigned char image[], uint16_t sizeofimage)
{
    ...

        lcd_senddata(swapBits(image[column]));
    ...

?

[Arduinoquäler (Gast)]

Leo C. schrieb:
> Warum sich immer noch mit PROGMEM abrackern?

Weil das die alten Compiler-Versionen des AVR Studios
nicht kennen / können.

Unserem freundlichen TO muss man ja jeden Popel einzeln aus der
Nase ziehen, und er äussert sich ja weder zu Compiler Versionen
noch zum Typ seiner Entwicklungsumgebung.

[Jan H. (janiiix3)]

Leo C. schrieb:
> Warum sich immer noch mit PROGMEM abrackern? Wie wärs mit
>

> const __flash unsigned char hesch [] = {...};
> 
> void lcd_printImage(const __flash unsigned char image[], uint16_t 
> sizeofimage)
> {
>     ...
> 
>         lcd_senddata(swapBits(image[column]));
>     ...
>

> ?

Hey Super. Damit klappt das.
Nur wird doch das ganze Array übergeben und nicht die Adresse oder?

[Oliver S. (oliverso)]

Jan H. schrieb:
> Nur wird doch das ganze Array übergeben und nicht die Adresse oder?

Nur Chuck Norris kann in C ganze Arrays als Funktionsparameter 
übergeben...

Oliver

[Jan H. (janiiix3)]

Oliver S. schrieb:
> Jan H. schrieb:
>> Nur wird doch das ganze Array übergeben und nicht die Adresse oder?
>
> Nur Chuck Norris kann in C ganze Arrays als Funktionsparameter
> übergeben...
>
> Oliver

Danke für den tollen Hinweis ;)
Hatte nur den * Operator vergessen.