Kleines Problem beim Auslesen mit Progmem

[Stefan S. (sschultewolter)]

Hallo,

ich habe ein kleines Problem mit PROGMEM und dem Auslesen innerhalb 
einer Funktion. Bekomme da einfach nicht die richtigen Werte. Kompiler 
meckert an der Stelle leider nicht.

#include <avr/pgmspace.h>
const uint8_t PROGMEM colorGreenBlue[NUM_LEDS][3] =
{
  {0x00, 0xff, 0x00},
  {0x00, 0xff, 0x48},
  {0x00, 0xff, 0x90},
  {0x00, 0xff, 0xd8},
  {0x00, 0xd8, 0xff},
  {0x00, 0x90, 0xff},
  {0x00, 0x48, 0xff},
  {0x00, 0x00, 0xff}
};

void leds_loadPattern(struct cRGB *newPattern, const uint8_t *pattern, const uint8_t SIZE_OF_PATTERN)
{
  for(uint8_t i = 0; i < SIZE_OF_PATTERN; i++) {
    newPattern[i].r = pgm_read_byte(pattern[i][0]);
    newPattern[i].g = pgm_read_byte(pattern[i][1]);
    newPattern[i].b = pgm_read_byte(pattern[i][2]);
  }
}

// Aufruf innerhalb einer Funktion
struct cRGB color[NUM_LEDS];
leds_loadPattern(color, (uint8_t*)colorWheel, NUM_LEDS);

[Peter II (Gast)]

wenn ich die Doku richtig lesen fehlt ein &

http://www.nongnu.org/avr-libc/user-manual/pgmspace.html

newPattern[i].r = pgm_read_byte(&pattern[i][0]);

[Stefan S. (sschultewolter)]

void leds_loadPattern(struct cRGB *newPattern, const uint8_t *pattern, const uint8_t SIZE_OF_PATTERN)
{
  for(uint8_t i = 0; i < SIZE_OF_PATTERN; i++) {
    newPattern[i].r = pgm_read_byte(&(pattern[i][0]));
    newPattern[i].g = pgm_read_byte(&(pattern[i][1]));
    newPattern[i].b = pgm_read_byte(&(pattern[i][2]));
  }
}

// Aufruf
leds_loadPattern(color, (uint8_t*)colorWheel, numLeds);

Bekomme hier noch den Fehler, welches sich auf die 3 Zeilen in der 
for-Schleife beziehen
subscripted value is neither array nor pointer nor vector


Edit: So gehts

    newPattern[i].r = pgm_read_byte(&(pattern[i*3+0]));

[Carl D. (jcw2)]

Du hast ja auch beim Aufruf der Funktion explizit aus einem 
2-dimensionalen ein 1-dimensionales gemacht.

[Johann L. (gjlayde)]

#define NUM_LEDS 8

struct cRGB { uint8_t r, g, b; };

typedef struct
{
    struct cRGB rgb[NUM_LEDS];
} pattern_t;

const __flash pattern_t colorGreenBlue =
{
    {
        { 0x00, 0xff, 0x00 },
        { 0x00, 0xff, 0x48 },
        { 0x00, 0xff, 0x90 },
        { 0x00, 0xff, 0xd8 },
        { 0x00, 0xd8, 0xff },
        { 0x00, 0x90, 0xff },
        { 0x00, 0x48, 0xff },
        { 0x00, 0x00, 0xff }
    }
};

void leds_loadPattern (pattern_t *pattern, const __flash pattern_t *pattern0)
{
    *pattern = *pattern0;
}

void Aufruf_innerhalb_einer_Funktion (void)
{
    pattern_t color;
    leds_loadPattern (&color, &colorGreenBlue);
}

[Max B. (theeye)]

Klappt jetzt alles?
Für neue Projekte empfehle ich __flash 
https://gcc.gnu.org/onlinedocs/gcc/Named-Address-Spaces.html

Gruß Max