Struct aus EEPROM lesen als Funktionsparameter

[Mark (Gast)]

Hallo,

ich habe eine Struktur im EEPROM liegen und möchte diese mit einer 
Funktion auslesen. Ich nutze eeprom_read_block aus der <avr/eeprom.h> 
Lib.

Bin mir nicht sicher ob das so gehen könnte.

Zudem wie könnte ich der Funktion einen weiteren Parameter hinzufügen 
mit dem ich das jeweilige Element von Level[..] setzen kann, dass gerade 
ausgelesen soll?

Danke!

typedef struct
{
  uint8_t allowed;
  uint8_t LV;
}Set1_t;

typedef struct
{
  int32_t mot_Pos1Offset;
  int32_t mot_Pos2Offset;
  Set1_t default;
}Set2_t;

typedef struct
{
  Set2_t Level[7];
} Def_Set_t;

Def_Set_t EEMEM EEPROMparameters;
Def_Set_t RAMparameters;

void Factors (volatile Def_Set_t *Lv)
{
  eeprom_read_block( &Lv->Level[3].default.allowed, &Lv->Level[3].default.allowed, sizeof(Lv->Level[3].default.allowed) );
}

[Rufus Τ. F. (rufus)]

Meinst Du vielleicht so etwas ?

void Factors (volatile Def_Set_t *Lv, int index)
{
  eeprom_read_block( &Lv->Level[index].default.allowed,  sizeof(Lv->Level[3].default.allowed) );
}

Du hast eeprom_read_block zweimal den gleichen Parameter übergeben; ich 
nehme an, daß das ein Tippfehler ist, denn was außer Zieladresse und 
Anzahl der zu lesenden Bytes soll der Funktion übergeben werden?

Ist Dir bewusst, daß Du so nur genau ein Byte liest?

sizeof(Lv->Level[3].default.allowed) ist nämlich sizeof uint8_t ...

[Mark (Gast)]

Rufus Τ. F. schrieb:
> Meinst Du vielleicht so etwas ?
>
> void Factors (volatile Def_Set_t *Lv, int index)
> {
>   eeprom_read_block( &Lv->Level[index].default.allowed,
> sizeof(Lv->Level[3].default.allowed) );
> }

Ja genau sowas meinte ich, weil ich vom Index abhängig mache was gelesen 
werden soll aus dem Struct

Rufus Τ. F. schrieb:
> Du hast eeprom_read_block zweimal den gleichen Parameter übergeben; ich
> nehme an, daß das ein Tippfehler ist, denn was außer Zieladresse und
> Anzahl der zu lesenden Bytes soll der Funktion übergeben werden?

Der Prototyp der Funktion lautet

 void eeprom_read_block (void *__dst, const void *__src, size_t __n);

wo die Quelle, das Ziel und die größe angegeben werden muss...

Wieso sind bei dir da nur zwei Parameter angegeben?


Rufus Τ. F. schrieb:
> Ist Dir bewusst, daß Du so nur genau ein Byte liest?
>
> sizeof(Lv->Level[3].default.allowed) ist nämlich sizeof uint8_t ...

Ja ist mir bewusst, aber die Struktur enthält auch andere Datentypen die 
größer uint8_t sind z.B. könnte allowed auch uint32_t sein... die 
Struktur habe ich für die Frage stark vereinfacht

[Jakob (Gast)]

Blöde Frage von einem ASM-bevorzugenden Dino:

Wozu kann "uint8_t allowed" und ".default.allowed" gut sein,
WENN man eben nicht uintDEFAULT_t, sondern ganz POPLIG und REAL
die int32_t lesen will, die man hoffentlich auch so gespeichert
hat?

Bitte keine ideologischen Debatten, sondern einleuchtende (!)
Erklärungen für den Gebrauchs-, oder Portabilitäts-Vorteil
solcher (mir) umständlich erscheinenden Definitions-Rituale!

[Jakob (Gast)]

Kommt BESTIMMT nichts Sachbezogenes!
Werde wohl, wenn überhaupt Reaktionen kommen,
als Troll behandelt... :-(

[Mark (Gast)]

müsste es nicht so heißen?

Def_Set_t EEMEM EEPROMparameters;
Def_Set_t RAMparameters;

void Factors (volatile Def_Set_t *Lv, uint8_t index)
{
  eeprom_read_block( &Lv->Level[index].default.allowed, &EEPROMparameters.Level[index].default.allowed, sizeof(Lv->Level[index].default.allowed) );
}

[Mark (Gast)]

Jakob schrieb:
> Kommt BESTIMMT nichts Sachbezogenes!
> Werde wohl, wenn überhaupt Reaktionen kommen,
> als Troll behandelt... :-(

die Struktur ist für die Frage bewusst vereinfacht gehalten...

Ansich ähnelt die Struktur einem Baumdiagramm

[Axel S. (a-za-z0-9)]

Mark schrieb:
> Jakob schrieb:
>> Kommt BESTIMMT nichts Sachbezogenes!
>> Werde wohl, wenn überhaupt Reaktionen kommen,
>> als Troll behandelt... :-(
>
> die Struktur ist für die Frage bewusst vereinfacht gehalten...

Damit tust du dir keinen Gefallen. Warum glaubst du, eine Antwort auf 
ein Problem, das dem deinigen nur ähnlich ist (wenn überhaupt) würde dir 
besser helfen als eine Antwort auf dein eigentliches Problem?

> Ansich ähnelt die Struktur einem Baumdiagramm

Dann zeigs halt. Oder laß es. Zum Rätselraten ist mir meine Zeit zu 
schade. Ach ja, warum du ein Array in eine struct verpacken mußt, weißt 
auch nur du. Für mich sieht das so aus, als wärest du in die Grube 
gefallen, die du dir selber gegraben hast mit deinen x-fach 
verschachtelten struct Definitionen.

Vielleicht möchtest du mal nachlesen (in einem C-Buch) was der sizeof() 
Operator tut, wenn man ihn auf struct-Member anwendet. Und wie sich das 
davon unterscheidet, wenn man ihn auf die gesamte struct anwendet.

[Rufus Τ. F. (rufus)]

Mark schrieb:
> Der Prototyp der Funktion lautet  void eeprom_read_block (void *__dst,
> const void *__src, size_t __n);
>
> wo die Quelle, das Ziel und die größe angegeben werden muss...

Bei Deinem Aufruf sind Quelle und Ziel identisch:

void Factors (volatile Def_Set_t *Lv)
{
  eeprom_read_block( 
     &Lv->Level[3].default.allowed, 
     &Lv->Level[3].default.allowed, 
     sizeof(Lv->Level[3].default.allowed) );
}

Daß die gelesenen Daten natürlich auch noch irgendwo hinmüssen, das 
hatte ich ... wohl von der Doppelung verwirrt ... übersehen.

Damit verändert sich das hierzu:

void Factors (volatile Def_Set_t *Lv, int index, uint8_t *dest)
{
  eeprom_read_block( 
     &Lv->Level[index].default.allowed,  
     dest, 
     sizeof(Lv->Level[3].default.allowed) );
}