Typenkonflikte in KeiluV5

Rufus Τ. F. schrieb:
> Mach doch mal den Textersatz, den der Präprozessor macht:
>
> configureGPIO(TSC_LS2_SAMPLE.pin, TSC_LS2_SAMPLE.port, ...)
>
> // wird im ersten Schritt zu
>
> configureGPIO({TSC_GROUP_PORT.tscGroup1,GPIO_PIN_2}.pin,
> {TSC_GROUP_PORT.tscGroup1,GPIO_PIN_2}.port, ...)
>
> Das schluckt kein C-Compiler.

Ok, und weshalb funktioniert Folgendes? :

1

#define TOUCHKEY2_LED        (pin_t)             {GPIOB, GPIO_PIN_2}

2

configureGPIO(TOUCHKEY2_LED.pin, TOUCHKEY2_LED.port,

Steht da doch:
a value of type "GPIO_TypeDef *" cannot be used to initialize an entity 
of type "volatile uint32_t"

Frank M. schrieb:
> Fragender schrieb:
>> typedef struct tscGroupPort{
>> GPIO_TypeDef* tscGroup1,
>> tscGroup2,
>> tscGroup3,
>> tscGroup4,
>> tscGroup5,
>> tscGroup6;
>> }tscGroupPorts_t;
>
> Du hast 2 Möglichkeiten, das zu fixen:
>
> A) Du schreibst vor jeden Struct Member explizit den Typ hin und
> schließt die Zeilen mit Semikolon statt Komma ab.
>
> B) Du wiederholst den Stern in jeder Zeile, damit tscGroup2 usw. auch
> wirklich Pointer sind - so wie Du es auch für tscGroup1 vorgesehen hast.
>
> Ein oft gemachter Fehler, wie z.B.int main ()
> {
>    int * a, b, c;
> }
>
> a ist dann ein Pointer, b und c sind aber keine.
>
> So etwas passiert, wenn man alles möglichst knapp formulieren will. Das
> rächt sich dann.
>
> P.S.
>
> Ich vermeide prinzipiell, das Sternchen an den Typ zu kleben, sondern
> umgebe ihn immer mit Leerzeichen links und rechts. Denn es gehört nicht
> direkt zum Typ. Zum anderen habe ich mir angewöhnt, den Typ explizit vor
> jeden Identifier wie z.B. einer Variablen zu schreiben. Ist zwar mehr
> Schreibarbeit, ist aber lesbarer. Aus diesem Grund ist der Fehler in der
> obigen Definition auch extrem schlecht zu erkennen.
>
> Ein Compiler sieht
>    int* a, b, c;
>
> als Kurzform von:
>    int * a;
>    int b;
>    int c;
>
> Ein Mensch sieht das auf den ersten Blick unter Umständen anders.



Hi Frank

So funktionierts.

Rufus Τ. F. schrieb:
> Mach doch mal den Textersatz, den der Präprozessor macht:
>
> configureGPIO(TSC_LS2_SAMPLE.pin, TSC_LS2_SAMPLE.port, ...)
>
> // wird im ersten Schritt zu
>
> configureGPIO({TSC_GROUP_PORT.tscGroup1,GPIO_PIN_2}.pin,
> {TSC_GROUP_PORT.tscGroup1,GPIO_PIN_2}.port, ...)
>
> Das schluckt kein C-Compiler.

Vonwegen es schluckt kein Compiler...

Wirklich top von dir Frank, danke für deine Hilfe.
Ich hatte gemeint, wenn der Stern am Typ hängt, gälte das für alle 
members. Wieder etwas neues gelernt.

Ich habe mich für deine Variante entschieden.

1

typedef struct tscGroupPort{

2

GPIO_TypeDef *tscGroup1;

3

GPIO_TypeDef *tscGroup2;

4

GPIO_TypeDef *tscGroup3;

5

GPIO_TypeDef *tscGroup4;

6

GPIO_TypeDef *tscGroup5;

7

GPIO_TypeDef *tscGroup6;

8

}tscGroupPorts_t;

NichtWichtig schrieb:
> Steht da doch:
> a value of type "GPIO_TypeDef *" cannot be used to initialize an entity
> of type "volatile uint32_t"

Woww! Suppper Tipp.

Fragender schrieb:
> error:  #144: a value of type "GPIO_TypeDef *" cannot be used to
> initialize an entity of type "volatile uint32_t"

Warum verstehst du die Fehlermeldung eigentlich nicht?

du hast eine Variable

1

volatile unsigned long blabla;

und willst selbige mit einem Zeiger, nämlich GPIO_TypeDef* belegen. 
Jetzt müßtest du bloß mal nachschauen, wie dein Typ "GPIO_TypeDef" denn 
überhaupt definiert ist. Ich vermute mal, daß sich dahinter ein dicker 
fetter struct befindet, der sämtliche Register eines der GPIO's enthält. 
Aber das kannst du ja selber herauskriegen.

Warum mußt du eigentlich alles so unsäglich verkomplizieren, bis keiner 
mehr durchsieht, was du da eigentlich gemacht hast (im Gegensatz zu 
gemeint hast)

W.S.