Atmega Flash -> PROGMEM -> ARRAY

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

Hier fehlt das "&" vor image ;)

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?

1

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

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

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

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

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?

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

1

2

  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

1

2

  pgm_read_byte(&image[column])

Jan H. schrieb:

1

...

2

lcd_gotoxy(page++, 0);

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

1

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

2

{

3

  uint16_t page  = 0;

4

  uint16_t column  = 0;

5

6

  lcd_gotoxy(0,0);

7

  for (column=0;column<sizeofimage;column++)

8

  {

9

    lcd_senddata(swapBits(pgm_read_byte(image+column)));

10

    if ((column%LCD_WIDTH_SIZE)==0)

11

    {

12

      lcd_gotoxy(page++, 0);

13

      if (page==LCD_PAGE_SIZE) page=0;

14

    }

15

  }

16

}

Und der Aufrufer..

1

lcd_printImage(pgm_read_ptr(hesch),1024);

Nach wie vor das selbe..

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

1

> void lcd_printImage (const unsigned char* PROGMEM image,

2

>                      uint16_t sizeofimage)

3

>

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

Das mag er nicht.

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

Compilers.

Musste mir gerade das lachen verkneifen ;)

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.

1

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

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

1

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

2

3

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

4

{

5

    ...

6

7

        lcd_senddata(swapBits(image[column]));

8

    ...

?

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

1

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

2

> 

3

> void lcd_printImage(const __flash unsigned char image[], uint16_t 

4

> sizeofimage)

5

> {

6

>     ...

7

> 

8

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

9

>     ...

10

>

> ?

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

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

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.