Pointer auf Array in struct, aber wie macht man das?

[Steffen H. (avrsteffen)]

Hallo,

Ich komm einfach nicht weiter mit einem Pointer auf ein Array-Eintrag in 
einem struct namens settings_t. Ich habe eine Funktion, die mir die 
Daten aus dem Array im struct per SPI ausgeben soll. Und die erwartet 
einen Pointer auf den Beginn der Daten im Array des Structs.

Hier ist das Struct:

typedef struct {
  double steps_per_mm[3];
  uint8_t microsteps;
  uint8_t pulse_microseconds;
  double default_feed_rate;
  double default_seek_rate;
  uint8_t invert_mask;
  double mm_per_arc_segment;
  double acceleration;
  double junction_deviation;
  long DRVCTRL[3];
  long CHOPCONF[3];
  long SMARTEN[3];
  long SGCSCONF[3];
  long DRVCONF[3];
} settings_t;

extern settings_t settings;

Hier die Funktion welche den Pointer erwartet.

void spi_write_data(char* source, char wr_bytes, char spi_cs)

Und so versuche ich sie momentan aufzurufen:

spi_write_data(&settings.DRVCONF[0],5,CS_EXT);

Ich weiß das da irgendwas noch nicht stimmt, denn es kommt immer die 
Warmung:

../spi.c:68: warning: passing argument 1 of 'spi_write_data' from incompatible pointer type

Wie deklariere/initialisiere ich den Pointer richtig? Bekomm es einfach 
nicht hin. Ich lerne leider erst in "C" zu programmieren.

Wer kann mir weiterhelfen?

Steffen

[geier99 (Gast)]

Steffen H. schrieb:
> void spi_write_data(char* source, char wr_bytes, char spi_cs)

wie wärs mit:
void spi_write_data(long* source, char wr_bytes, char spi_cs)

[Sebasrian L. (Gast)]

DRVCONF ist ein long array deine funktion erwartet aber nen char 
pointer. Typkonflikt?

[Karl H. (kbuchegg)]

Steffen H. schrieb:

> Ich weiß das da irgendwas noch nicht stimmt, denn es kommt immer die
> Warmung:
>

> ../spi.c:68: warning: passing argument 1 of 'spi_write_data' from
> incompatible pointer type
>

spi_write_data will einen Pointer auf char.
Du gibst ihm einen Pointer auf long.

und das passt nun mal nicht. ein char ist kein long. Und auch ein 
Pointer auf char ist daher erst mal nicht kompatibel mit einem Pointer 
auf long.

Jetzt wissen wir an dieser Stelle aber, das dieses 'char' im Grund nur 
ein notwendiges Übel ist für 'ist mir eigentlich wurst was du hast, ich 
nehm einfach die Bytes und schreib sie'.

D.h. im Grunde spielt es keine wirkliche Rolle, dass da ein long im 
Spiel ist. Man darf hier also durchaus Umcasten.
Man muss nur darauf achten, dass spi_write_data auch die tatsächlich 
richtige Anzahl an Bytes schreibt. Und an dieser Stelle kommt dann eine 
zweite Zutat ins Spiel: ein sizeof.

um das komplette Array zu schreiben.

  spi_write_data( (char*)settings.DRVCONF, sizeof( settings.DRVCONF ), CS_EXT);

[Steffen H. (avrsteffen)]

Wow, das geht aber noch schnell um diese Urzeit. Danke schon mal für die 
Antworten.

geier99 schrieb:
> wie wärs mit:
> void spi_write_data(long* source, char wr_bytes, char spi_cs)
Ich will die Funktion schon multikunktionell halten. Deswegen möcht ich 
'char' wenn möglich nicht unbedingt mit 'long' ersetzen.

Karl Heinz Buchegger schrieb:
> um das komplette Array zu schreiben.

spi_write_data( (char*)settings.DRVCONF, sizeof( settings.DRVCONF ), CS_EXT);

Ich werd es sofort ausprobieren. Das mit dem 'sizeof' wollte ich auch 
noch machen wenn es erstmal funktioniert.

Danke euch allen schon mal.

Steffen

[Johann L. (gjlayde)]

Steffen H. schrieb:

> geier99 schrieb:
>> wie wärs mit:
>> void spi_write_data(long* source, char wr_bytes, char spi_cs)
> Ich will die Funktion schon multikunktionell halten. Deswegen möcht ich
> 'char' wenn möglich nicht unbedingt mit 'long' ersetzen.

char* durch long* ist zudem eine gaaanz schlechte Idee wenn bestimmte 
ALignments gegeben sind, etw wenn dein Code auf ARM oder PowerPC oder 
einer anderen strict-Alignment Plattform ausführt.  Zur Hardware steht 
ja nix geschrieben (ausser in deinem Login-Name).

Zudem ist das Layout von settings_t eigenwillig.  Erwarte Probleme beim 
Übertragen zwischen Plattformen mit unterschiedlichen Alignments, 
zumindest wenn du settings_t als ganzes überträgst.

Ohne saubere (De)Serialisierung sind Bit-, Byte-, Word- und 
float-Endianess sowie Darstellung von Flotings weitere Problemfelder.

[Steffen H. (avrsteffen)]

Johann L. schrieb:
> char* durch long* ist zudem eine gaaanz schlechte Idee wenn bestimmte
> ALignments gegeben sind, etw wenn dein Code auf ARM oder PowerPC oder
> einer anderen strict-Alignment Plattform ausführt.  Zur Hardware steht
> ja nix geschrieben (ausser in deinem Login-Name).
>
> Zudem ist das Layout von settings_t eigenwillig.  Erwarte Probleme beim
> Übertragen zwischen Plattformen mit unterschiedlichen Alignments,
> zumindest wenn du settings_t als ganzes überträgst.
>
> Ohne saubere (De)Serialisierung sind Bit-, Byte-, Word- und
> float-Endianess sowie Darstellung von Flotings weitere Problemfelder.

Da hast du wohl Recht und ich noch garnicht drüber nachgedacht. Dann 
werd ich da wohl noch Probleme kriegen.
Danke Johann für die Hinweise.

Karl Heinz Buchegger schrieb:
> um das komplette Array zu schreiben.

spi_write_data( (char*)settings.DRVCONF, sizeof( settings.DRVCONF ), CS_EXT);

Es gibt leider weiterhin eine Warnung und der Pointer ist leider immer 
noch falsch. :(
Vieleicht noch das '&' vergessen und siehe da, so geht es jetzt. Ganz 
ohne WARNING. :)

spi_write_data((char*)&settings.DRVCONF[Motor_X],sizeof(settings.DRVCONF[Motor_X]),CS_EXT);

Danke euch allen denn jetzt kann ich endlich schlafen.

Steffen

[Rolf Magnus (Gast)]

Steffen H. schrieb:
> geier99 schrieb:
>> wie wärs mit:
>> void spi_write_data(long* source, char wr_bytes, char spi_cs)
> Ich will die Funktion schon multikunktionell halten. Deswegen möcht ich
> 'char' wenn möglich nicht unbedingt mit 'long' ersetzen.

Warum nimmst du dann nicht void*? Das ist der in C als generischer 
"Zeiger auf irgendwas, dessen Typ unbekannt ist" vorgesehene Typ. Dann 
kannst du dir auch den Cast sparen.