avr-gcc: Format für Flash-String-Ausgabe?

[Taucher (Gast)]

Gibt es bei arv-gcc ein Format für die printf-Familie, mit dem man 
Flash-Strings  ausgeben kann?

[Oliver S. (oliverso)]

https://www.nongnu.org/avr-libc/user-manual/group__avr__stdio.html

Oliver

[Taucher (Gast)]

Das hilft leider nicht weiter.

Was ich suche, ist ein Format-Präfix für %s, das dafür sorgt dass 
*printf den betreffenden Parameter aus dem Flash holt – ich will mir 
damit einen zweiten Puffer ersparen, in den ich den String aus dem Flash 
kopieren müsste, wenn er mit dem normalen %s ausgegeben wird.

Die *printf_P-Funktionen leisten das nicht.

[foobar (Gast)]

Großes S holt aus dem Flash.

[Hmmm (Gast)]

foobar schrieb:
> Großes S holt aus dem Flash.

Steht übrigens auch in der oben verlinkten Dokumentation:

"S Similar to the s format, except the pointer is expected to point to a 
program-memory (ROM) string instead of a RAM string."

[Taucher (Gast)]

Vielen Dank!

[Taucher (Gast)]

Hm, so einfach scheint es doch nicht zu sein:

const char __flash *getstring();

sprintf(pBuf, "%S", getstring());
               ^
warning: format ‘%S’ expects argument of type ‘wchar_t *’, but argument 9 has type ‘const __flash char *’ [-Wformat=]

wchar_t auf einem 328?

[Falk B. (falk)]

Taucher schrieb:
> Hm, so einfach scheint es doch nicht zu sein:
> const char __flash *getstring();
> sprintf(pBuf, "%S", getstring());

Was macht denn deine Funktion getstring()? Liefert die WIRKLICH einen 
Pointer auf einen String im Flash? Ich glaube außerdem, daß __flash 
zwingend vor char stehen muss.

const __flash char *getstring();

https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Flash_mit_flash_und_Embedded-C

Ich bin mir nicht sicher, ob das mit __flash funktioniert. Probier mal 
lieber so.

const char *getstring();

Oder teste mal einfach so.

sprintf(pBuf, "%S", PSTR("Hello World"));

Außerdem bietet sich sprintf_P an, damit sparst du RAM, denn der 
Formatstring ist ja konstant.

> warning: format ‘%S’ expects argument of type ‘wchar_t *’, but argument
> 9 has type ‘const __flash char *’ [-Wformat=]
>
> wchar_t auf einem 328?

Was für einen IDE bzw. Compiler nutzt du?

[foobar (Gast)]

Das ist ein Problem beim gcc - der hat fest eingebaute Regeln für die 
Formatstringüberprüfung (%S ist alias für %ls ist wchar-string) und die 
stimmen nicht mit denen der avr-libc überein.  Ignorieren oder 
abschalten (-Wno-format).

[Taucher (Gast)]

Ja, ich habs gerade getestet – die Warnung kann man ignorieren. sprintf 
tut mit dem %S wie erhofft.

[Taucher (Gast)]

@Falk: hier eine lauffähige Version der getstring. Die heißt hier 
getTaskName_P und hat einen Parameter, aber der ändert am Prinzip 
nichts:

    typedef struct {
        const void                  *taskAddr;
        const char const __flash    *name;
    } Name;

    #define N(name)       { name, FSTR(#name) }
    static const Name __flash NameTable[] = {
        N(onClockTick),
        N(readSensors),
        N(initSensors),
        N(powerBankTriggered),
        N(powerBankKeepAlive)
    };
    #undef N


    const char __flash *getTaskName_P(void *pTask) {
        for (uint8_t i = 0; i < ELEMENTS(NameTable); i++)
            if (pTask == NameTable[i].taskAddr)
                return NameTable[i].name;

        return PSTR("***");
    }

[Falk B. (falk)]

Taucher schrieb:
> const char __flash *getTaskName_P(void *pTask) {
>         for (uint8_t i = 0; i < ELEMENTS(NameTable); i++)
>             if (pTask == NameTable[i].taskAddr)
>                 return NameTable[i].name;
>         return PSTR("***");
>     }

Tststs, solche Sachen würde ich GANZ schnell bleiben lassen! for() 
Schleifen ohne Klammern sind nicht gut, sondern eine SEHR gute Chance, 
sich ins Knie zu schießen. Spätestesn dann, wenn man mal eas erweitern 
will. Das Gleiche bei if(). Die Klammern lasse ich nur weg, wenn es 
WIRKLICH eine einfach Anweisung ist, die in die gleiche Zeile wie das 
if() paßt, ansonsten IMMER mit Klammern.

Just my 2 cents.

[Taucher (Gast)]

Falk B. schrieb:
> Die Klammern lasse ich nur weg, wenn es
> WIRKLICH eine einfach Anweisung ist, die in die gleiche Zeile wie das
> if() paßt, ansonsten IMMER mit Klammern.

Der Turm macht aber genau das, was er soll und das auf Anhieb… Er stammt 
aus der Bibliothek im Kopf, da muss ich nicht mehr viel nachdenken.

Es gibt schwierigere Konstrukte in C.

Viel lästiger finde ich dass der Compiler Konstrukte, wie die NameTable 
in meinem Beispiel nicht in Funktionen erlaubt. Hält man sich nicht 
daran, kommt eine völlig irreführenden Fehlermeldung.

[Johann L. (gjlayde)]

Falk B. schrieb:
> Ich glaube außerdem, daß __flash zwingend vor char stehen muss.

Die Reihenfolge der Qualifier spielt keine Rolle.

Taucher schrieb:
>  const char const __flash    *name;

Ein "const" kann weg; conster als const geht nicht ;-)