C String in uint32 und zurück?

[Alex W. (de1m)]

Hallo Leute,

eine Frage an die C Profis

Es geht um die ESP8266, da kann man bestimmte Infos in flash speichern. 
Es sind die 4KB Blöcke und laut Doku sollte es uint32 sein.
So ich möchte z.B "ABCD" speichern und beim nächsten mal wieder lesen.

Ich habe hier was dazu geschrieben, an sich funktioniert es auch, aber 
ich glaube, dass es kein richtiger Weg ist, wenn ich da etwas mehr habe, 
dann wird es umständlich

  uint32 y;
  uint32 x;

  unsigned char buf[4] = "ABCD";
    uint32 frostr = buf[0] + 256U*buf[1] + 65536U*buf[2] + 16777216U*buf[3];

    int i = spi_flash_erase_sector(0x8c);
    os_printf("spi_flash_erase_sector: %d\n", i);
    int o = spi_flash_write(0x8c000, &frostr, sizeof(x));
    os_printf("spi_flash_write: %d\n", o);

    int p = spi_flash_read(0x8c000, &y, sizeof(y));
    os_printf("spi_flash_read: %d\n", p);

    buf[3] = y >> 24;
    buf[2] = y >> 16;
    buf[1] = y >> 8;
    buf[0] = y;

    os_printf("read :%s\n", buf);

[Peter II (Gast)]

ich würde es mit memcpy machen.

unsigned char buf[4] = "ABCD";
uint32 frostr;
memcpy( &frostr, buf, sizeof(frostr) );

//und zurück
unsigned char buf2[4];
memcpy( buf, &frostr, sizeof(frostr) );

bist du dir sicher mit buf[4]? Es passen zwar die 4 Buchstaben rein, 
aber nicht die \0. Darfst also keine Stringfunktionen mit dem buf 
aufrufen.

[Alex W. (de1m)]

Peter II schrieb:
> bist du dir sicher mit buf[4]

mein Fehler, habe ich vergessen.

Noch eine Frage, wie ist es wenn

buf[32]

 ist, sollte ja genau so gehen oder?

[Peter II (Gast)]

Alex W. schrieb:
> ist, sollte ja genau so gehen oder?
dafür müsste man mal wissen, wie die Parameter von spi_flash_write 
definiert sind - hast du ein link auf die Doku?

vermutlich kannst du auch einfach so den String übergeben

spi_flash_write(0x8c000, (uint32_t*)buf, 4));

[Alex W. (de1m)]

danke für die schnelle Hilfe, hat geklappt
So sieht die Funktion aus

  uint32 tosave, toread;
  char ssid[32];
  char ssid2[32];

  os_memcpy(ssid, "CO2", 3);

  int i = spi_flash_erase_sector(0x8c);
  os_printf("spi_flash_erase_sector: %d\n", i);
  int o = spi_flash_write(0x8c000, (uint32*)ssid, sizeof(ssid));
  os_printf("spi_flash_write: %d\n", o);

  int p = spi_flash_read(0x8c000, &toread, sizeof(toread));

  os_memcpy(ssid2, &toread,sizeof(&toread));

  os_printf("Read from flash: %s\n", ssid2);

Aber eine Frage noch )
beim "os_prinf" bekomme ich als Ergebnis "CO2@", hast du vielleicht eine 
Vermutung woher "@" Zeichen kommt?

[Alex W. (de1m)]

Peter II schrieb:
> hast du ein link auf die Doku?

Doku gibt es hier 
https://espressif.com/sites/default/files/documentation/0a-esp8266ex_datasheet_en.pdf

[Peter II (Gast)]

Alex W. schrieb:
> Vermutung woher "@" Zeichen kommt?

du schreibst sizeof(buf) und liest aber sizeof(int32). Also nur 4 
Zeichen.

Also 2 Fehler. Ersten schreibst du das \0 nicht mit raus. Musst dir also 
irgendwo merken wie lang der String wirklich ist. Und beim Lesen musst 
du halt die länge angeben oder so lange lesen bis \0 kommt (die du aber 
vorher auch schreiben musst)

[Peter II (Gast)]

Alex W. schrieb:
> Doku gibt es hier
> 
https://espressif.com/sites/default/files/documentation/0a-esp8266ex_datasheet_en.pdf

das steht nichts von spi_flash_write

[Alex W. (de1m)]

ah ja, hier aber 
https://espressif.com/sites/default/files/documentation/2c-esp8266_non_os_sdk_api_reference_en.pdf

[Peter II (Gast)]

Alex W. schrieb:
> ah ja, hier aber
> 
https://espressif.com/sites/default/files/documentation/2c-esp8266_non_os_sdk_api_reference_en.pdf

ok, dann musst du nur dafür sorgen, das die länge durch 4 Teilbar ist.

spi_flash_write(0x8c000, (uint32*)ssid, sizeof(ssid));
spi_flash_read(0x8c000, (uint32*)ssid, sizeof(ssid));

[Alex W. (de1m)]

okay, danke für die Hilfe ich versuche es morgen weiter.

[Dirk B. (dirkb2)]

Alex W. schrieb:
> Aber eine Frage noch )
> beim "os_prinf" bekomme ich als Ergebnis "CO2@", hast du vielleicht eine
> Vermutung woher "@" Zeichen kommt?

Peter II schrieb:
> aber nicht die \0. Darfst also keine Stringfunktionen mit dem buf
> aufrufen.

printf mit %s ist auch eine Stringfunktion.

[Nop (Gast)]

Peter II schrieb:
> vermutlich kannst du auch einfach so den String übergeben
> spi_flash_write(0x8c000, (uint32_t*)buf, 4));

Nein, das geht nicht besonders gut, weil buf als array of char zwar 
genauso 4 Bytes belegt wie ein uint32_t, aber buf ist ohne Alignment. 
Wenn die Routine einen uint32_t erwartet, wird sie wohl auch ein 
korrektes alignment erwarten.