Union Struct Fehler

[Gerhard (Gast)]

Liebe Kollegen!

Kann mir jemand sagen wo der Fehler liegt?

union fnc_lat
{
  struct {
  unsigned int cb      :2;    // Control Bit
  unsigned int cr      :1;    // Counter Reset
  unsigned int pd1    :1;    // Power Down 1
  unsigned int mux    :3;    // MUX Control
  unsigned int pcp    :1;    // PD Polarity
  unsigned int cdt    :1;    // CP Threestate
  unsigned int fe      :1;    // Fastlock Enable
  unsigned int fm      :1;   // Fastlock Mode
  unsigned int tcc    :4;    // Timer Counter Control
  unsigned int cs1    :3;   // Current Setting 1
  unsigned int cs2    :3;    // Current Setting 2
  unsigned int pd2    :1;    // Power Down 2
  unsigned int pv      :2;    // Prescaler Value
} BITS;
  uint32_t RAW;
}

//Init

union fnc_lat i_lat_init = {3,0,0,0,0,0,0,0,0,0,0,0,0};


union fnc_lat i_lat_save = i_lat_init; //FEHLER: expression must have a constant value

[(prx) A. K. (prx)]

i_lat_init ist keine Konstante. Globale und statische Variablen kann man 
aber nur mit Konstanten initialisieren.

[DirkZ (Gast)]

Gerhard schrieb:
> } BITS;
>   uint32_t RAW;
> }

zu

  uint32_t RAW;
}bob;

semi, keine anonyme Union

>
> //Init
>
> union fnc_lat i_lat_init = {3,0,0,0,0,0,0,0,0,0,0,0,0};
>

union fnc_lat i_lat_init = {{3,0,0,0,0,0,0,0,0,0,0,0,0}};


> union fnc_lat i_lat_save = i_lat_init; //FEHLER: expression must have a
> constant value

funktioniert.

i_lat_save.BITS.cb hat den Wert 3.

[Gerhard (Gast)]

wenn ich const union ... schreibe geht es auch nicht.
Darf ich nur Werte nehmen??

union fnc_lat
{
  struct {
  unsigned int cb      :2;    // Control Bit
  unsigned int cr      :1;    // Counter Reset
  unsigned int pd1    :1;    // Power Down 1
  unsigned int mux    :3;    // MUX Control
  unsigned int pcp    :1;    // PD Polarity
  unsigned int cdt    :1;    // CP Threestate
  unsigned int fe      :1;    // Fastlock Enable
  unsigned int fm      :1;   // Fastlock Mode
  unsigned int tcc    :4;    // Timer Counter Control
  unsigned int cs1    :3;   // Current Setting 1
  unsigned int cs2    :3;    // Current Setting 2
  unsigned int pd2    :1;    // Power Down 2
  unsigned int pv      :2;    // Prescaler Value
} BITS;
  uint32_t RAW;
}

//Init

const union fnc_lat i_lat_init = {3,0,0,0,0,0,0,0,0,0,0,0,0};


union fnc_lat i_lat_save = i_lat_init; //FEHLER: expression must have a constant value

[Gerhard (Gast)]

Danke! Hab die zweite { schon dazugeschrieben.

[(prx) A. K. (prx)]

Gerhard schrieb:
> wenn ich const union ... schreibe geht es auch nicht.

Weil ein "const" keine Konstante markiert, sondern eine konstante 
Variable. Das klingt zwar bescheuert, aber so haben die C Götter das mal 
eingetütet. Erst C++ hat da (ein wenig) ein Einsehen.

[Kindergärtner (Gast)]

Gerhard schrieb:
> } BITS;
>   uint32_t RAW;
> };
1. fehlt da bei dir das Semikolon
> const union fnc_lat i_lat_init = {{3,0,0,0,0,0,0,0,0,0,0,0,0}};
2. Müssen da doppelte {{ und }} hin

3.: Kompiliere einfach mit C++, da geht sowas. Schreibe dann auch 
"constexpr" vor die letzte Zeile um sicherzustellen, dass die "statisch" 
initialisiert wird.

[(prx) A. K. (prx)]

Was auch in C geht:
#define i_lat_init {{3,0,0,0,0,0,0,0,0,0,0,0,0}}
union fnc_lat i_lat_save = i_lat_init;

[Kindergärtner (Gast)]

A. K. schrieb:
> Was auch in C geht:
Es sei denn, er versucht danach intuitiv so etwas:

uint32_t X = i_lat_init.RAW;

Was natürlich fürchterlich in die Hose geht.

[Gerhard (Gast)]

Das fehlende Semikolon ist beim kopieren verschwunden. Im Code steht es.
Cool, dass dir das aufgefallen ist.

Ich habe es so gelöst:

union fnc_lat
{
  struct {
  unsigned int cb      :2;    // Control Bit
  unsigned int cr      :1;    // Counter Reset
  unsigned int pd1    :1;    // Power Down 1
  unsigned int mux    :3;    // MUX Control
  unsigned int pcp    :1;    // PD Polarity
  unsigned int cdt    :1;    // CP Threestate
  unsigned int fe      :1;    // Fastlock Enable
  unsigned int fm      :1;   // Fastlock Mode
  unsigned int tcc    :4;    // Timer Counter Control
  unsigned int cs1    :3;   // Current Setting 1
  unsigned int cs2    :3;    // Current Setting 2
  unsigned int pd2    :1;    // Power Down 2
  unsigned int pv      :2;    // Prescaler Value
} BITS;
  uint32_t RAW;
} ;

//Init


#define init_i{{3,0,0,0,0,0,0,0,0,0,0,0,0}}

const union fnc_lat i_lat_init = init_i;

union fnc_lat i_lat_save = init_i;

DANKE! für Eure Hilfe!

[(prx) A. K. (prx)]

Kindergärtner schrieb:
> Was natürlich fürchterlich in die Hose geht.

Klar, aber das sagt ihm dann schon der Compiler. Und so wie er es jetzt 
gemacht hat geht auch dies.

[Kindergärtner (Gast)]

A. K. schrieb:
> Klar, aber das sagt ihm dann schon der Compiler.
Ja, durch unverständliche kryptische Error-Messages...
> Und so wie er es jetzt gemacht hat geht auch dies.
Ja, mit mehr Makros geht alles...

[DirkZ (Gast)]

Gerhard schrieb:
> } BITS;
>   uint32_t RAW;
> }

ich schreib's nochmal: keine anonyme Union, Semikolon fehlt.

    uint32_t RAW;
 } beliebiger_name;

[Kindergärtner (Gast)]

DirkZ schrieb:
> ich schreib's nochmal: keine anonyme Union, Semikolon fehlt.
Die union ist nicht anonym, sie heißt "fnc_lat". Dein vorschlag würde 
gleich eine Instanz anlegen; das will er an der Stelle aber gar nicht.

DirkZ schrieb:
> ich schreib's nochmal:
Naja, dein 1. Post war nicht gerade... verständlich