STM32 Struct ins Flash

[Ma B. (drumstick)]

Guten Morgen!

Ich arbeite mit dem STM32F103VDT6 und dem uVision von Keil.

Ich habe festgestellt, dass meine Funktion die Daten ins Flash speichert 
doch nicht ganz funktioniert.

So sieht mein  Struct aus:

typedef struct  {
                  uint8_t             CntCal;
                  int                 ShttrueIdx[CNTSHT];
                  uint8_t             Nmbr_of_Shttr;
                  uint8_t             Nmbr_of_ShttrPart;
                  uint8_t             CntsofWings;
                  int                 MotDir;
                  int                 MotDistCntVal;
                  int                 MotDistCal_PulsesMax;
                  int                 MotDistShttrPoints[6];
                  BOOL                cal_OK;
                }
                tSetup2;

BOOL ist als unsigned int definiert!

So sieht meine Funktion aus:

void set_in_Flash(tSetup2 *setup)
{
  uint32_t *pData = (uint32_t *)setup;
  uint8_t i;
  int x = (sizeof(tSetup2) / sizeof(int));
  
  FLASH_Unlock();
  FLASH_ClearFlag(FLASH_FLAG_BSY | FLASH_FLAG_EOP | FLASH_FLAG_PGERR |  FLASH_FLAG_WRPRTERR);
  FLASH_ErasePage(FLSHBSEADRPG192);
   for(i=0;i<x;i++)
  {
     FLASH_ProgramWord(FLSHBSEADRPG192 + i*sizeof(uint32_t), pData[i]);
  }
  FLASH_Lock();
  
}

Kann mir jemand diese Zeile erklären?:

FLASH_ProgramWord(FLSHBSEADRPG192 + i*sizeof(uint32_t), pData[i]);

Warum pData[i] und nicht pData? bzw. pData ist ja ein Pointer der auf 
die Adresse von setup zeigt.

Ich wüsste im Moment nicht, wo der Fehler sonst noch sein könnte?!


Vielen Dank und Grüsse

M.B.

[Steel (Gast)]

Das hat schon seine Richtigkeit, das i-te 32-Bit Wort aus dem 
Datenbereich mit der Anfangsadresse pData wird in den Flashbereich mit 
Anfangsadresse FLSHBSEADRPG192 plus i*4Byte kopiert.

Sehe da keinen Fehler.

Den struct sollte man "packed" machen, damit der Compiler keine 
Füllbytes einfügt und man sollte darauf achten, dass er vielfache von 32 
Bit lang ist.

[Markus M. (Firma: EleLa - www.elela.de) (mmvisual)]

Ich würde die Struct eher so anordnen (weiß grad die korrekte Syntax 
nicht auswendig, aber so ähnlich):

typedef struct  {
union {
int W;
struct{
                  uint8_t             CntCal;
                  uint8_t             Nmbr_of_Shttr;
                  uint8_t             Nmbr_of_ShttrPart;
                  uint8_t             CntsofWings: 7;
                  uint8_t             cal_OK: 1;
};
};
                  int                 MotDir;
                  int                 MotDistCntVal;
                  int                 MotDistCal_PulsesMax;
                  int                 MotDistShttrPoints[6];
                  int                 ShttrueIdx[CNTSHT];
                }
                tSetup2;

[Ma B. (drumstick)]

ok, Du hast die uint8_t in eine union verfasst, aber was bedeutet das 
int W; ??

Danke und Gruss

[Markus M. (Firma: EleLa - www.elela.de) (mmvisual)]

Da könnte man die ganzen Zähler alle auf einmal auslesen und schneller 
weiter verwenden, z.B. bei Übertragung über CAN oder USB, da die immer 
32-Bit weise gefüttert werden.