Problem mit __flash, struct und (vmtl) Pointern.

[Moritz A. (moritz_a)]

Hi,

ich versuche gerade eine kleine Ansteuerung für LEDs, eigentlich nichts 
wildes. Da sich aber recht viele Muster ergeben, sollen diese alle mit 
__flash abgelegt werden.

Die Struktur an sich habe ich mir folgendermaßen gedacht:

light_patterns[] = [
    {
        light_pattern.nr_elements => int,
        light_pattern.rotate      => int,
        light_pattern.lp_elements => [
            &pattern_element_1
            &pattern_element_2,
            &pattern_element_3
        ]
    },
    {
        light_pattern.nr_elements => int,
        light_pattern.rotate      => int,
        light_pattern.lp_elements => [
            &pattern_element_2,
            &pattern_element_4,
            &pattern_element_2
            …
        ]
    }
    …
]

pattern_element_1 = …
pattern_element_2 = …
…

Also grob gesagt ein Array, welches die verschiedenen Muster enthält. 
Diese bestehen jeweils aus dem Parameter "rotate" sowie einem Array aus 
Pointern auf die einzelnen "Leuchtzustände" (da sich diese in 
verschiedenen Pattern wiederholen können) sowie der Anzahl der 
pattern_element, da diese nicht gleich sein müssen.

Nun habe ich versucht das ganze von innen nach außen aufzubauen, aber 
ich bekomme die einzelnen pattern_element nicht sauber ins Array im 
light_pattern. Nach diversem ratlosen rumprobieren stehe ich gerade da 
mit:

#ifndef _PATTERNS_H
#define _PATTERNS_H

typedef const __flash struct {
    const uint8_t l12;
    const uint8_t l34;
    const uint8_t duration;
} pattern_element;

typedef const __flash struct {
    const uint8_t nr_elements;
    const uint8_t rotate;
    const __flash pattern_element *lp_elements[];
} light_pattern;

const __flash pattern_element lpe_4444_30 = { 0x44, 0x44, 30 }; 
const __flash pattern_element lpe_0000_30 = { 0x00, 0x00, 30 };
const __flash pattern_element lpe_0000_200 = { 0x00, 0x00, 200 };

const __flash pattern_element * const __flash lp_elements_dualflash[] = { 
    &lpe_4444_30, &lpe_0000_30, 
    &lpe_4444_30, &lpe_0000_30,
    &lpe_0000_200
};

const __flash light_pattern lp_dualflash = { 5, 1, &lp_elements_dualflash };

#endif //_PATTERNS_H

Und dem Fehler:

Compiling C: main.c
avr-gcc -c -mmcu=attiny2313 -I. -gdwarf-2 -DF_CPU=8000000UL -DUART_RX_BUFFER_SIZE=32 -DUART_TX_BUFFER_SIZE=32 -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -Wall -Wextra -Wpedantic -Wstrict-prototypes -Wa,-adhlns=./main.lst  -std=gnu99 -Wundef -MMD -MP -MF .dep/main.o.d main.c -o main.o 
In file included from main.c:6:0:
patterns.h:26:1: warning: missing braces around initializer [-Wmissing-braces]
 const __flash light_pattern lp_dualflash = { 5, 1, &lp_elements_dualflash };
 ^
patterns.h:26:1: warning: (near initialization for ‘lp_dualflash.lp_elements’) [-Wmissing-braces]
patterns.h:26:1: warning: initialization from incompatible pointer type [enabled by default]
patterns.h:26:1: warning: (near initialization for ‘lp_dualflash.lp_elements[0]’) [enabled by default]
patterns.h:26:1: warning: initialization of a flexible array member [-Wpedantic]
patterns.h:26:1: warning: (near initialization for ‘lp_dualflash.lp_elements’) [-Wpedantic]

Grüße
Moritz

[Stefan E. (sternst)]

    const __flash pattern_element *lp_elements[];

Hier ist das [] falsch. Lasse das weg, und die Fehler sollten 
verschwinden.

[Moritz A. (moritz_a)]

Hatte ich auch schon getestet, aber so ganz mag er den Pointer immer 
noch nicht:

In file included from main.c:6:0:
patterns.h:26:1: warning: initialization from incompatible pointer type [enabled by default]
 const __flash light_pattern lp_dualflash = { 5, 1, &lp_elements_dualflash };
 ^
patterns.h:26:1: warning: (near initialization for ‘lp_dualflash.lp_elements’) [enabled by default]

Daher war mein erster Verdacht, dass da irgendeine Typüberprüfung 
zuschlägt bezüglich Pointer auf einzelnes Element oder Arrayanfang, 
weshalb ich die [] im struct ergänzte.

[Stefan E. (sternst)]

    const __flash pattern_element **lp_elements;

[Moritz A. (moritz_a)]

 const __flash light_pattern lp_dualflash = { 5, 1, &lp_elements_dualflash };
 ^
patterns.h:26:1: warning: (near initialization for ‘lp_dualflash.lp_elements’) [enabled by default]
patterns.h:26:1: error: initializer element is not computable at load time
patterns.h:26:1: error: (near initialization for ‘lp_dualflash.lp_elements’)
patterns.h:26:1: warning: missing initializer for field ‘lp_elements’ of ‘light_pattern’ [-Wmissing-field-initializers]
patterns.h:13:37: note: ‘lp_elements’ declared here
     const __flash pattern_element **lp_elements;
                                     ^

[Stefan E. (sternst)]

 const __flash pattern_element * const __flash *lp_elements;

[Moritz A. (moritz_a)]

kompiliert wieder durch, aber:

In file included from main.c:6:0:
patterns.h:26:1: warning: initialization from incompatible pointer type [enabled by default]
 const __flash light_pattern lp_dualflash = { 5, 1, &lp_elements_dualflash };
 ^
patterns.h:26:1: warning: (near initialization for ‘lp_dualflash.lp_elements’) [enabled by default]

Aber behoben hat es dann ein

const __flash pattern_element * const __flash (*lp_elements)[];

Insofern schonmal ein großes Danke für deine Hilfe!


Wo ich es so betrachte, eigentlich müsste der Schritt über
lp_elements_dualflash[]
doch auch umgehbar sein, dass ich direkt im Struct das Pointerarray 
anstatt dem Pointer aufs Pointerarray habe.

EDIT: Quatsch, das hat ja keine feste Länge.