struct als array an funktion übergeben.

[Mathias G. (motze)]

hallo liebe Gemeinde,

ich möchte gerne ein struct an einer Funktion übergeben. Leider 
funktioniert das nicht so wie geplant

struct LED
{
  uint8_t gruen;
  uint8_t rot;
  uint8_t blau;
};

struct LED led[60];

void output_send_led(struct LED _led[], uint8_t _anzahl_led)
{
  uint8_t i;
  uint8_t a;
  
  for (a = 0; a < _anzahl_led; a++)
  {
    for (i = 0; i < 8; i++)
    {
      if(_led[a].gruen & 0x80)
      {
        output_eins();
      }
      else
      {
        output_null();
      }
      _led[a].gruen<<=1;
    }
    
    for (i = 0; i < 8; i++)
    {
      if(_led[a].rot & 0x80)
      {
        output_eins();
      }
      else
      {
        output_null();
      }
      _led[a].rot<<=1;
    }
    
    for (i = 0; i < 8; i++)
    {
      if(_led[a].blau & 0x80)
      {
        output_eins();
      }
      else
      {
        output_null();
      }
      _led[a].blau<<=1;
    }
  }
}

int main(void)
{
  DDRC |= (1<<PC0);
  
  led[2].blau = 0xFF;
  led[2].rot = 0xFF;
  led[2].gruen = 0xFF;
  
    while(1)
    {
    output_send_led(led, 10);
    output_reset();
    }
}

vielen dank für eure Hilfe.

[Falk B. (falk)]

@ Mathias G. (motze)

>ich möchte gerne ein struct an einer Funktion übergeben. Leider
>funktioniert das nicht so wie geplant.

Kann man machen, ist aber eher unüblich. Solche großen Massen an Daten 
will man nicht kopieren, sondern übergibt Zeiger.

Da fehlt wohl ein typedef. Eher so.

typedef struct
{
  uint8_t gruen;
  uint8_t rot;
  uint8_t blau;
} LED_t;

LED_t led[60];

void output_send_led(LED_t *_led, uint8_t _anzahl_led)
{
  uint8_t i;
  uint8_t a;
  
  for (a = 0; a < _anzahl_led; a++)
  {
    for (i = 0; i < 8; i++)
    {
      if(_led[a].gruen & 0x80)
      {
        output_eins();
      }
      else
      {
        output_null();
      }
      _led[a].gruen<<=1;
    }
    
    for (i = 0; i < 8; i++)
    {
      if(_led[a].rot & 0x80)
      {
        output_eins();
      }
      else
      {
        output_null();
      }
      _led[a].rot<<=1;
    }
    
    for (i = 0; i < 8; i++)
    {
      if(_led[a].blau & 0x80)
      {
        output_eins();
      }
      else
      {
        output_null();
      }
      _led[a].blau<<=1;
    }
  }
}

int main(void)
{
  DDRC |= (1<<PC0);
  
  led[2].blau = 0xFF;
  led[2].rot = 0xFF;
  led[2].gruen = 0xFF;
  
    while(1)
    {
    output_send_led(led, 10);
    output_reset();
    }
}

[Mathias G. (motze)]

erstmal ganz herzlichen dank für deine Hilfe leider bekomme ich noch 
eine Fehlermeldung:

array type has incomplete element type

was möchte mir den der Compiler damit sagen?

[Falk B. (falk)]

Möglicherweise fehlt da ein

#include <stdint.h>

[Mathias G. (motze)]

nein leider nicht die Fehlermeldung bleibt die gleiche.

[Falk B. (falk)]

Dann sind die Dinge oben im Quelltext nicht so in deinem ECHTEN 
Quelltext, sondern in anderer Reihenfolge oder verteilt auf Dateien?

[Patrick D. (oldbug)]

Irgendwas in diese Richtung wird es aber sein!

OT: wie kommt man eigentlich auf die Idee, irgend einen Namen mit 
Unterstrich[en] am Anfang zu versehen? Lass das...

[Mathias G. (motze)]

erstmal danke für eure Antwort. Ihr hattet recht es war nicht richtig in 
Quelltext. Jetzt wird es ohne Fehler Compiliert.

vielen dank dafür

[Mathias G. (motze)]

jetzt habe ich leider ein anderes Problem der folgende Code Funktioniert 
genau einmal danach nicht mehr woran kann das den liegen?

void output_reset()
{
  _delay_us(60);
}

void output_alle(uint8_t _led[])
{
  uint8_t i;
  uint8_t k;
  
  for (i = 0; i < 180; i++)
  {  
    for (k = 0; k < 8; k++)
    {
      if(_led[i] & 0x80)
      {
        output_eins();
      }
      else
      {
        output_null();
      }
      _led[i]<<=1;  
    }
  }
}

uint8_t Led_60[180] =
{
  0x00, 0xFF, 0xFF, //1
  0x00, 0xFF, 0xFF, //2
  0x00, 0xFF, 0xFF, //3
  0x00, 0xFF, 0xFF, //4
  0x00, 0xFF, 0xFF, //5
  0x00, 0xFF, 0xFF, //6
  0x00, 0xFF, 0xFF, //7
  0x00, 0xFF, 0xFF, //8
  0x00, 0xFF, 0xFF, //9
  0x00, 0xFF, 0xFF, //10
  
  0xFF, 0xFF, 0x00, //11
  0xFF, 0xFF, 0x00, //12
  0xFF, 0xFF, 0x00, //13
  0xFF, 0xFF, 0x00, //14
  0xFF, 0xFF, 0x00, //15
  0xFF, 0xFF, 0x00, //16
  0xFF, 0xFF, 0x00, //17
  0xFF, 0xFF, 0x00, //18
  0xFF, 0xFF, 0x00, //19
  0xFF, 0xFF, 0x00, //20
  
  0xFF, 0xFF, 0xFF, //21
  0xFF, 0xFF, 0xFF, //22
  0xFF, 0xFF, 0xFF, //23
  0xFF, 0xFF, 0xFF, //24
  0xFF, 0xFF, 0xFF, //25
  0xFF, 0xFF, 0xFF, //26
  0xFF, 0xFF, 0xFF, //27
  0xFF, 0xFF, 0xFF, //28
  0xFF, 0xFF, 0xFF, //29
  0xFF, 0xFF, 0xFF, //30
    
  0xFF, 0xFF, 0xFF, //31
  0xFF, 0xFF, 0xFF, //32
  0xFF, 0xFF, 0xFF, //33
  0xFF, 0xFF, 0xFF, //34
  0xFF, 0xFF, 0xFF, //35
  0xFF, 0xFF, 0xFF, //36
  0xFF, 0xFF, 0xFF, //37
  0xFF, 0xFF, 0xFF, //38
  0xFF, 0xFF, 0xFF, //39
  0xFF, 0xFF, 0xFF, //40
  
  0xFF, 0xFF, 0xFF, //41
  0xFF, 0xFF, 0xFF, //42
  0xFF, 0xFF, 0xFF, //43
  0xFF, 0xFF, 0xFF, //44
  0xFF, 0xFF, 0xFF, //45
  0xFF, 0xFF, 0xFF, //46
  0xFF, 0xFF, 0xFF, //47
  0xFF, 0xFF, 0xFF, //48
  0xFF, 0xFF, 0xFF, //49
  0xFF, 0xFF, 0xFF, //50
  
  0xFF, 0xFF, 0xFF, //51
  0xFF, 0xFF, 0xFF, //52
  0xFF, 0xFF, 0xFF, //53
  0xFF, 0xFF, 0xFF, //54
  0xFF, 0xFF, 0xFF, //55
  0xFF, 0xFF, 0xFF, //56
  0xFF, 0xFF, 0xFF, //57
  0xFF, 0xFF, 0xFF, //58
  0xFF, 0xFF, 0xFF, //59
  0x00, 0xFF, 0x00 //60  
};

int main(void)
{  
  DDRC |= (1<<PC0);
  
    while(1)
    {
    output_alle(Led_60);
    output_reset();
    _delay_ms(5000);
    }
}

ich weiß das mit den unterstrichen ist doof ich werde es mir auch 
abgewöhnen.

Also die Leds leuchten genau 5 Sekunden auf und danach geht nix mehr.
Output_reset hat bei anderen Sachen immer hingehauen mit Timing.

vielen dank schonmal

[foobar (Gast)]

> Also die Leds leuchten genau 5 Sekunden auf und danach geht nix mehr.

>     _led[i]<<=1;

Kein Wunder, nach einem Durchlauf von output_alle sind alle Werte in 
deinem led-Array 0.

Wenn du die Werte erhalten möchtest:

void output_alle(uint8_t led[])
{
    uint8_t i, k;

    for (i = 0; i < 180; i++)
        for (k = 0x80; k; k >>= 1)
            if(led[i] & k)
                output_eins();
            else
                output_null();
}

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

Falk Brunner schrieb:
> Solche großen Massen an Daten
> will man nicht kopieren, sondern übergibt Zeiger.

Genau das tut er. Es gibt in C keine Array-Übergabe per Wert, nur per 
Referenz. Sein

> void output_send_led(LED_t _led[], uint8_t _anzahl_led)

ist nur eine andere Schreibweise von deinem

> void output_send_led(LED_t *_led, uint8_t _anzahl_led)

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

Falk Brunner schrieb:
> Da fehlt wohl ein typedef. Eher so.

Ob man
  struct LED { ... }
schreibt und im weiteren Code stets
  struct LED
verwendet, oder erst
  typedef struct { ... } LED
und dann dafür
  LED
ist gleich. Seine Variante wirkt nur etwas altbacken. Man kann auch
  typedef struct LED { ... } LED
schreiben, und dann beides verwenden.

[Falk B. (falk)]

@ A. K. (prx)


>ist nur eine andere Schreibweise von deinem

>> void output_send_led(LED_t *_led, uint8_t _anzahl_led)

Hmm, stimmt. Es war schon zu spät.

[Mathias G. (motze)]

Vielen dank für die Hilfe jetzt geht es so wie ich es will. Manchmal ist 
man echt blind. Klar wenn ich die alle Rausschifte das nur noch nullen 
übrig geblieben sind.

vielen dank nochmal.