initializer element is not constant

[Sepp (Gast)]

Hallo zusammen.

Ich hab da ein kleines Problem.
Hier der Codeausschnitt:

//enc.h
unsigned char GetMac1();

//enc.c
unsigned char GetMac1()
{
  MAC_DDR = 0x00;
  return MAC1;
}

//uipopt.h
#define UIP_FIXEDETHADDR 1
//hier mein stümperhafter Eintrag
#define UIP_ETHADDR0    GetMac1()
//ENCJ28J60_MACx existieren
#define UIP_ETHADDR1  ENC28J60_MAC2
#define UIP_ETHADDR2  ENC28J60_MAC3
#define UIP_ETHADDR3  ENC28J60_MAC4
#define UIP_ETHADDR4  ENC28J60_MAC5
#define UIP_ETHADDR5  ENC28J60_MAC6

//uip.c
const struct uip_eth_addr uip_ethaddr = {{UIP_ETHADDR0,
            UIP_ETHADDR1,
            UIP_ETHADDR2,
            UIP_ETHADDR3,
            UIP_ETHADDR4,
            UIP_ETHADDR5}};

Die Fehlermeldung:
../Net/uip/uip.c:132: error: initializer element is not constant
../Net/uip/uip.c:132: error: (near initialization for 
'uip_ethaddr.addr[0]')

Wie kann ich mein Ausgabe dort einfügen?

[Rolf Magnus (Gast)]

Wie der Compiler schon sagt: Der Initialisierer muß eine 
Compilezeit-Konstante sein.

> Wie kann ich mein Ausgabe dort einfügen?

#define GetMac1() (MAC_DDR = 0x00, MAC1)

[Karl H. (kbuchegg)]

Da hast du Pech, das geht in C nicht.
Du kannst zur Initialisierung einer globalen Variablen
keinen Funktionsaufruf verwenden.

Die erste Funktion die in C ausgeführt werden kann, ist main()

[Rolf Magnus (Gast)]

Einen Funktionsaufruf als Initialisierer geht bei C++, nicht aber bei C. 
Und mein define ist natürlich Blödsinn.

[Sepp (Gast)]

Puhh. Was mach ich da, bzw wo mach ich das. Sowas hab ich ja noch nie 
gesehen.

[Karl H. (kbuchegg)]

Sepp wrote:
> Puhh. Was mach ich da, bzw wo mach ich das. Sowas hab ich ja noch nie
> gesehen.

Ne andere Frage:
Wer oder was ist eigentlich MAC1? Ich vermute mal, das
ist ein Makro

Hintergrund: Gibt es einen Grund, warum du das nicht
so schreiben willst:

#define UIP_ETHADDR0    MAC1

[Karl H. (kbuchegg)]

Sepp wrote:
> Puhh. Was mach ich da, bzw wo mach ich das. Sowas hab ich ja noch nie
> gesehen.


Ich hoffe Rolf stimmt mir zu wenn ich sage: Musst du auch
nicht verstehen. Der Komma-Operator ist, nun ja, eine Waffe.

Im Übrigen hätte dir das Makro auch bei deinem Problem nicht
geholfen, da eine Zuweisung (und eine solche ist ja im Makro
enthalten) nun mal in einer Funktion stehen muss.

[Sepp (Gast)]

Hallo

also MAC1 sind Eingänge. Also hier soll MAC1 mit PINF gleichgesetzt 
sein. Ich will damit den Hexwert zweier Hexswitcher auslesen.
somit sollte #define UIP_ETHADDR0 MAC1 auch nicht funktionieren (,oder?)

[Karl H. (kbuchegg)]

Sepp wrote:
> Hallo
>
> also MAC1 sind Eingänge. Also hier soll MAC1 mit PINF gleichgesetzt
> sein. Ich will damit den Hexwert zweier Hexswitcher auslesen.
> somit sollte #define UIP_ETHADDR0 MAC1 auch nicht funktionieren (,oder?)

Wenn MAC1 etwas darstellt, was zur Compilezeit zu einer Konstanten
evaluiert, dann sollte das schon gehen.

[Sepp (Gast)]

Also die Funktion ist raus. Die Zuweisung ist jetzt direkt, so wie von 
kbuchegg beschrieben. Leider wieder gleicher Fehler.
../Net/uip/uip.c:131: error: initializer element is not constant
../Net/uip/uip.c:131: error: (near initialization for 
'uip_ethaddr.addr[0]')


Das würde leider erklären warum jeder, der einen Webserver baut eine 
fest Mac vergibt.

[Benedikt K. (benedikt)]

Sepp wrote:

> Das würde leider erklären warum jeder, der einen Webserver baut eine
> fest Mac vergibt.

Ich würde sagen, das liegt eher daran, dass die Mac Adresse eigentlich 
fest sein sollte.

Mach einfach das const weg, und weise den Wert am Anfang von main zu. 
Dann sollte es gehen.

[Sepp (Gast)]

Das war auch mein Gedanke. Leider wirft mir dann das Die uip tausende 
von Fehlermeldungen an verschiedensten stellen.
Die Lösung wird sein dem Makefile das generieren eines Defines zu 
überlassen und das änderen des Makefiles (also der MAC-Adressen) einem 
Script.

[Sepp (Gast)]

Verdammt waren da viele Rechtschreibfehler drin in meinem Thread :)

[Simon K. (simon)]

Du kannst die MAC Adresse des uip-Stacks auch zur Laufzeit 
initialisieren.. Siehe conf-Datei.