[AVR] __flash conversion warning

[Nico W. (nico_w)]

Moin zusammen,

Pointer, __flash und co sind noch recht neu für mich und ich bin schon 
eine ganze Zeit am lesen. Manches klappt schon ganz gut, aber hier weiß 
ich nicht mehr weiter.

Angefangen habe ich mal hiermit: 
https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Flash_mit_flash_und_Embedded-C

Mein Code sind in etwa so aus:

void writestr_F(const __flash char *data_F) {
  uint8_t r, i=0;
  while ((r = data_F[i])) {
    do_something(r);
    i++;
  }
}

#define FSTR(X) ((const __flash char *) { X })

main() {
  writestr_F(FSTR("print a string"));
}

Ich bekomme jetzt aber diese Warnung:

 warning: conversion from address space 'generic' to address space '__flash' [-Waddr-space-convert]

Grüße
Nico

[Peter II (Gast)]

bin mir nicht sicher, aber die {} sehen mir falsch aus.

#define FSTR(X) ((const __flash char *) ( X) )

[Nico W. (nico_w)]

Leider nicht. Egal ob (X),{X} oder X.

[Oliver S. (oliverso)]

"Print a String" ist nunmal ein normaler Sting, den legt der Compiler im 
SRam an. Da ändert auch der davor geschrieben Cast in einen Flash 
Pointer nichts.

Das Makro FSTR dient nur und ausschließlich dazu, Flash-Pointer zu 
initialisieren. Was du brauchst, ist PSTR. Das legt eine Stringkonstante 
direkt im Flash an.

Dazu solltest du dir nochmals den Unterschied zwischen printf und 
printf_P anschauen.

Oliver

[Nico W. (nico_w)]

Oliver S. schrieb:
> "Print a String" ist nunmal ein normaler Sting, den legt der Compiler im
> SRam an. Da ändert auch der davor geschrieben Cast in einen Flash
> Pointer nichts.

Ja, was du schreibst macht durchaus Sinn. Ich habe mir mal nach deinem 
Tipp printf_P und printf angeguckt. Kam da jetzt aber nicht wirklich 
weiter. Allerdings habe ich mir dann mal PSTR genauer angesehen.

Letzendlich will ich mein Zeugs ja in den Flash schreiben. Aber nicht 
mehr über PSTR und dem PROGMEM sondern über __flash.

Über Beitrag "Struct mit String im Flash Zugriff"
und PSTR bin ich jetzt auf folgendes hinaus:

#define XSTR(X) ({static const __flash char __s[] = (X); &__s[0];})

[Oliver S. (oliverso)]

Nico W. schrieb:
> Letzendlich will ich mein Zeugs ja in den Flash schreiben. Aber nicht
> mehr über PSTR und dem PROGMEM sondern über __flash.

Je nun, des Menschen Wille ist sein Himmelreich...

Oliver

[Nico W. (nico_w)]

Oliver S. schrieb:
> Je nun, des Menschen Wille ist sein Himmelreich...

Jepp, und der Wille macht sogar den Code deutlich einfacher kompatibel 
zu nem ARM.

Anstatt:

#ifdef __AVR__

  #include <avr/pgmspace.h>

#else

  #define PROGMEM
  #define PGM_P const char *
  #define PSTR(s) ((const PROGMEM char *)(s))
  #define pgm_read_byte(x) (*((uint8_t *)(x)))
  #define pgm_read_word(x) (*((uint16_t *)(x)))
  #define pgm_read_dword(x) (*((uint32_t *)(x)))

#endif /* __AVR__, ! __AVR__ */

einfach:

#ifndef __AVR__
  #define __flash
#endif /* ! __AVR__ */

#define FSTR(X) ((const __flash char[]) { X })
#define XSTR(X) ({static const __flash char __s[] = X; &__s[0];})