Pointer String mit PROGMEM?!

[Jan H. (janiiix3)]

Hallo,

Gibt es eine einfache Möglichkeit die ganzen Pointer Strings mit PROGMEM 
zu versehen?

/*
*  Strings for LCD Display
*/
typedef struct
{
  char *PrjName;
  char *SWV;
  char *HWV;
  char *SetTime;
  char *SetDate;
  char *SetIllumination;
  char *Time[10];  
  char *SwitchTimeEnable;
  char *ToggleTriac;
  char *ManuelMode;
  char *TriacOn;
  char *TriacOff;  
  char *SubmenueTime[5];
  char *SubmenueExit;
  char *hours;
  char *minutes;
  char *seconds;
}AppStrings_t;

[fop (Gast)]

Die Suchen & Ersetzen Funktion Deines Editors ? Suche "char", ersetze 
durch ...

[Jan H. (janiiix3)]

fop schrieb:
> Die Suchen & Ersetzen Funktion Deines Editors ? Suche "char", ersetze
> durch ...

So meinte ich das nicht.

Ich möchte gerne die Variablen mit im Programmspeicher einbetten (sind 
Konstante Pointer Strings), aktuell sind sie ja im SRAM.

AppStrings_t const Strings PROGMEM; // das bewirkt nichts...

[Georg G. (df2au)]

Ein schneller Blick in die libc-Hilfe zeigt beim Thema pgmspace, dass es 
z.B. den Typ PGM_p gibt.

[Jan H. (janiiix3)]

Georg G. schrieb:
> Ein schneller Blick in die libc-Hilfe zeigt beim Thema pgmspace, dass es
> z.B. den Typ PGM_p gibt.


Habe ich bereits gesehen. Das funktioniert aber nur mit char Array´s?!

[Frank M. (ukw) (Moderator)]

Das müsste eigentlich so gehen:

const char PrjName[] PROGMEM = "Hallo";
const char SWV[]     PROGMEM = "Welt";
...

typedef struct
{
   const char * const PROGMEM PrjName;
   const char * const PROGMEM SWV;
   ...
}AppStrings_t;

AppStrings_t mystrings =
{
    PrjName,
    SWV,
    ...
};

[Johann L. (gjlayde)]

Frank M. schrieb:
>    const char * const PROGMEM PrjName;

Das sollte einen Fehler geben.

const __flash char PrjName[] = "Hallo";
const __flash char SWV[] = "Welt";
...

typedef struct
{
   const __flash char *PrjName;
   const __flash char *SWV;
   ...
}AppStrings_t;

AppStrings_t mystrings = // mystrings im RAM
{
    PrjName,
    SWV,
    ...
};

const __flash AppStrings_t mystrings = // mystrings im Flash
{
    PrjName,
    SWV,
    ...
};

Analog mit PROGMEM, nur dass man dann noch #include avr/pgmspace.h und 
pgm_read_xxx braucht (mit __flash genügt "normales" C).

[Johann L. (gjlayde)]

Siehe auch:

AVR-GCC-Tutorial: Flash mit flash und Embedded-C
AVR-GCC-Tutorial: Flash mit PROGMEM und pgm read

[Frank M. (ukw) (Moderator)]

Johann L. schrieb:
>>    const char * const PROGMEM PrjName;
>
> Das sollte einen Fehler geben.

Nö. Ich hatte das vorher getestet. Der avr-gcc nimmt das anstandslos.

> const __flash AppStrings_t mystrings = // mystrings im Flash
> {

Die adäquate Version mit PROGMEM - falls eine ältere avr-gcc-Version 
benutzt wird, die noch kein __flash versteht:

const AppStrings_t mystrings PROGMEM =
{

[Peter D. (peda)]

Eigentlich macht man ja eine Struct, damit man nicht für jedes Element 
einen extra Pointer braucht, sondern nur einen Pointer auf die Struct, 
um schnell und codesparend mit Displacement auf die einzelnen Elemente 
zugreifen zu können.

Insbesondere die Pointer in den Flash zu legen, hieße ja, die Adressen 
sind zur Compilezeit bekannt. Wozu also die Umstände mit Pointern?

[Wilhelm M. (wimalopaan)]

Und bei der PROGMEM Variante aufpassen, dass man die PROGMEM C-Strings 
bei der Ausgabe anders behandeln muss als "normale" Stringliterale oder 
char-Arrays.

In C++ habe ich entsprechende Datentypen dafür und muss den Unterschied 
nicht selbst beachten.

[Eric B. (beric)]

Peter D. schrieb:
> Eigentlich macht man ja eine Struct, damit man nicht für jedes Element
> einen extra Pointer braucht, sondern nur einen Pointer auf die Struct,
> um schnell und codesparend mit Displacement auf die einzelnen Elemente
> zugreifen zu können.
>
> Insbesondere die Pointer in den Flash zu legen, hieße ja, die Adressen
> sind zur Compilezeit bekannt. Wozu also die Umstände mit Pointern?

Ich gebe dir Recht, aber wie baut man sich diese Struktur zusammen? Wenn 
ich nicht die Pointer sondern die Texte direkt im struct ablegen will, 
muss ich ja die Arraygrößen bzw die länder der Texten mitangeben:

const __flash struct {
  char helloWorld[14];
  char swVersion[6];
} flashTexts = {
  "Hello, World!",
  "v0.01"
};

Geht, ist aber ziemlich fehleranfällig.

[Eric B. (beric)]

hmm...

1) Definiere alle Texte mit einem Makro

// flashtexts.inc

// define all flash texts here
// TEXT(name, value)
TEXT(helloWorld, "Hello, World!")
TEXT(swVersion,  "0.01beta")
TEXT(breakfast, "Continental")

2) Header-file zum inkludieren wo man die Text(e) benutzen möchte:

// flashtexts.h
#ifndef FLASHTEXTS_H
#define FLASHTEXTS_H

#undef  TEXT
#define TEXT(NAME,VALUE) NAME[ sizeof(VALUE) ];

extern const __flash struct {
#include "flashtexts.inc"
} texts;

#endif

3) C Modul worin die Texte angelegt werden:

// flashtexts.c
#include "flashtexts.h"


#undef  TEXT
#define TEXT(NAME,VALUE) VALUE,

texts = {
#include "flashtexts.inc"
};

4) Benutzen:

#include "flashtexts.h"

int main()
{
  printf(texts.helloWorld);
}

[Johann L. (gjlayde)]

Frank M. schrieb:
> Johann L. schrieb:
>>>    const char * const PROGMEM PrjName;
>>
>> Das sollte einen Fehler geben.
>
> Nö. Ich hatte das vorher getestet. Der avr-gcc nimmt das anstandslos.

Sonderlich sinnvoll ist so ein Code aber nicht.  Es ist ungefähr so 
sinnvoll, wie unterschiedliche Komponenten eines Struct in 
unterschiedliche Sections legen zu wollen.

Immerhin bekomme ich eine Warnung, dass progmem ignoriert wird -- 
vorausgesetzt, es steht nach dem "PrjName".

[Rabe (Gast)]

Eric B. schrieb:

>

> const __flash struct {
>   char helloWorld[14];
>   char swVersion[6];
> } flashTexts = {
>   "Hello, World!",
>   "v0.01"
> };
>

> Geht, ist aber ziemlich fehleranfällig.

Was ist daran fehleranfällig?

[Peter D. (peda)]

Ich benutze die pgmspace.h möglichst nicht und "verschwende" RAM mit der 
C-konformen Schreibweise.
Meistens lohnt sich die Mühe auch nicht.
Beim ATMega1284 (16KB RAM) muß mit RAM nicht so geizen.

[Jan H. (janiiix3)]

Also in den meisten Quelltexten sehe ich nur Strings mit Arrays.

char Str[] = "Test_String";

Wie macht ihr das denn?

[Arduinoquäler (Gast)]

Peter D. schrieb:
> Beim ATMega1284 (16KB RAM) muß mit RAM nicht so geizen.

Beim Arduino Uno und Consorten (alle kleineren AVRs) ist man
häufig sehr dankbar wenn man das RAM zu Lasten des Flash von
den vielen Nicht-Variablen befreien kann.

[Eric B. (beric)]

Rabe schrieb:

> Was ist daran fehleranfällig?

Dass ich aufpassen muss, dass die Längenangaben und tatsächliche 
Textlängen passen, und die Reihenfolge der Texte und deren Definitionen 
die gleiche ist.

const __flash struct {
  char helloWorld[200];
  char swVersion[2];
} flashTexts = {
  "v0.01",
  "Hello, World!" 
};

[Peter D. (peda)]

Ich hab auch festgestellt, daß die pgmspace.h leider nicht abwärts 
kompatibel ist.
Wenn man ältere Projekte aus WINAVR 2010 mit neueren AVR-GCC compiliert, 
gibt es Fehlermeldungen (und umgekehrt).
Daher fasse ich die nur noch mit der Kneifzange an.

[Johann L. (gjlayde)]

Peter D. schrieb:
> Ich hab auch festgestellt, daß die pgmspace.h leider nicht abwärts
> kompatibel ist.
> Wenn man ältere Projekte aus WINAVR 2010 mit neueren AVR-GCC compiliert,
> gibt es Fehlermeldungen (und umgekehrt).

Es gibt Fehler, wenn man versucht, nicht-const Objekte ins Flash zu 
legen.  Das ist dann aber ein Problem des Project-Codes, nicht von 
pgmspace.h.

Dass avr-gcc nicht-const Daten im Flash ablehnt, ist auch den Release 
Notes entnehmbar; wenn man also asbach uralt Projekte übersetzten will, 
sollte man auch eine asbach uralt Toolchain verwenden.

Und hast du ein Beispiel, dass Code, der mit einer aktuellen Toolchain 
übersetzt, dass mit einem WinAVR-20100110 nicht mehr tut (abgesehen von 
damals noch nicht unterstützten Devices, Optionen, Qualifiern, etc).

[Frank M. (ukw) (Moderator)]

Johann L. schrieb:
> Dass avr-gcc nicht-const Daten im Flash ablehnt, ist auch den Release
> Notes entnehmbar; wenn man also asbach uralt Projekte übersetzten will,
> sollte man auch eine asbach uralt Toolchain verwenden.

Braucht man nicht. Dann schreibt man const davor und schon läuft es mit 
der alten und mit der neuen Toolchain.

[Johann L. (gjlayde)]

Frank M. schrieb:
> Johann L. schrieb:
>> Dass avr-gcc nicht-const Daten im Flash ablehnt, ist auch den Release
>> Notes entnehmbar; wenn man also asbach uralt Projekte übersetzten will,
>> sollte man auch eine asbach uralt Toolchain verwenden.
>
> Braucht man nicht. Dann schreibt man const davor und schon läuft es mit
> der alten und mit der neuen Toolchain.

Ja, ich weiß.  Hilft aber zunächst nicht weiter, wenn jemand ein Projekt 
mit einer neuen Toolchain generieren will und dann Fehler bekommt. 
Beispiel ist USBasp

http://www.fischl.de/usbasp/

Da kannst dann erst mal auf die Suche gehen, wo das const hin muss...