Problem mit pgm_read_byte_far

Gast #237045
Hi,

bitte kann mir jemand helfen?

Ich habe in meinem Mega128 sehr viel statische Daten die im Flash
liegen. Mittlerweile sind es weit über 100k. So viel ich weis, kann ich
die nicht mehr mit der Funktion pgm_read_byte auslesen, da diese
Funktion nur eine short Adresse hat.

Jetzt bin ich gezwungen auf die Funktion pgm_read_byte_far
umszusteigen, die eine long Adresse hat. Doch leider bringt der
Compiler Fehlermeldungen z.B. warning: cast from pointer to integer of
diffrent size oder error: SFR_IO_ADDR undeclared (fist use this
function).
Was jemand was da schief läuft.

Hier mein Code (pseudo)

long pos;
unsigned char temp;

const unsigned char Array [xxx] PROGMEM = { "Datenbytes" };

temp = pgm_read_byte_far (&Array[pos]);
Gast
Zuerst: ist keine "Fehlermeldung", sondern "nur" eine Warnung. Aber
das hilft nicht wirklich. Soweit erinnert, verwaltet avr-gcc alle
Pointer mit 16bit, so auch die auf Flash/Progmem. Die *_far Funktionen
erwartet eine 32bit-Angabe (vgl. pgmspace.h). Ein Cast duerfte die
Warnung unterdruecken. Aber das ist wohl nur die "halbe Miete":
Soweit ich weiss, koennen mit PROGMEM nur Konstanten in den ersten 64kB
des Flash-Speichers angelegt werden, duerfte also bei 100kB noch von
anderer Stelle zu Problemen kommen.
Gast
"Ich habe in meinem Mega128 sehr viel statische Daten die im Flash
liegen. Mittlerweile sind es weit über 100k."

Eine andere µC-Familie (z.B. ARM7) mag in der Grössenordnung sinnvoller
sein. Grosse Daten/Programme sind nicht die Stärke von 8-Bit
Prozessoren.
Gast
Hallo Tom,

ist zwar ne Weile her bei dir, aber ich habe gerade genau das gleiche
Problem. Hattest du damals eine Lösung gefunden?

Besten Dank,
Peter
"pgm_read_byte_far" will einen uint32_t-Wert als Parameter, daher
diese Warnung. Der normale Pointer-Mechanismus funktioniert hier nicht.
Man muß die Adresse in einen uint32_t packen und damit dann
"pgm_read_byte_far" aufrufen.
Gast
Hierzu braucht's folgendes Makro, welches einen 32 Bit Pointer auf die
Progmem Daten zurück liefert:

//----------------------------------------------------------
// Macros to access data defined in PROGMEM above 64kB
//----------------------------------------------------------
#define FAR(var)                                      \
({                                                    \
    uint_farptr_t tmp;                                \
                                                      \
    _asm_ __volatile__(                             \
                                                      \
            "ldi    %A0, lo8(%1)"           "\n\t"    \
            "ldi    %B0, hi8(%1)"           "\n\t"    \
            "ldi    %C0, hh8(%1)"           "\n\t"    \
            "clr    %D0"                    "\n\t"    \
        :                                             \
            "=d" (tmp)                                \
        :                                             \
            "p"  (&(var))                             \
    );                                                \
    tmp;                                              \
})
//----------------------------------------------------------

Dann kannst Du z.B. auf die folgende Weise auf die Daten zugreifen:

MyByte = pgm_read_byte_far(FAR(MyProgMemPointer));


@jörg (falls Du zufällig hier reinschauen solltest)

Ich sehe immer wieder Threads mit Leuten, die über dieses Problem
stolpern. Wäre es nicht sinnvoll, dieses Makro in <pgmspace.h>
aufzunehmen?

MfG Peter
Gast
Besten Dank euch beiden!

In der Zwischenzeit hatte ich mir eine proprietäre Lösung gebastelt,
aber der Ansatz über das FAR Makro mit der Direktive hh8(...) ist
natürlich schöner.

Macht's gut,
Peter
Gast
Hi,

ich versuche nun schon eine ganze Weile das oben genannte Makro

//----------------------------------------------------------
// Macros to access data defined in PROGMEM above 64kB
//----------------------------------------------------------
#define FAR(var)                                      \
({                                                    \
    uint_farptr_t tmp;                                \
                                                      \
    asm volatile(                           \
                                                      \
            "ldi    %A0, lo8(%1)"           "\n\t"    \
            "ldi    %B0, hi8(%1)"           "\n\t"    \
            "ldi    %C0, hh8(%1)"           "\n\t"    \
            "clr    %D0"                    "\n\t"    \
        :                                             \
            "=d" (tmp)                                \
        :                                             \
            "p"  (&(var))                             \
    );                                                \
    tmp;                                              \
})
//----------------------------------------------------------

zu verwenden um auf Konstanten mittels pgm_read_byte_far zuzugreifen.


Ich bekomme jedoch in dieser Zeile

uint8_t b = pgm_read_byte_far(FAR(&array[X]));

immer die Fehlermeldung

error: invalid lvalue in unary '&'

Was ist falsch an dieser Zeile ? Ich habe testweise auch mal

uint8_t b = pgm_read_byte_far(FAR(array[X]));

probiert, was aber nur zur dreifachen Ausgabe von

undefined reference to `r30'
undefined reference to `r30'
undefined reference to `r30'

führt. Wäre schön wenn mir hier jemand helfen könnte.

Gruss, Tobias
Gast
Wie hast du das Array deklariert, bzw. liegen die Array-Daten wirklich 
im FLASH? Das ist nämlich nicht ganz so trivial. Wenn man nicht gut 
aufpasst liegt zwar der Array-Pointer im Flash, nicht aber die Daten, 
oder umgekehrt.

Weitere Infos dazu findest Du in der Dokumentation zur Avr_LibC, im 
speziellen im Kapitel "9.10 Frequently Asked Questions"

Falls Dein "array" wirklich ein Pointer auf die gewünschte 
Progmem-Adresse ist, dann müsste es folgendermassen funktionieren:
1
uint8_t b = pgm_read_byte_far(FAR(array)+X);

So kannst Du sogar die Limitierung umgehen, dass mit dem Array-Index 
maximal 32kByte Adressierbar

Viel Erfolg
Ich habe das gleiche Problem wie Tobi. Ich glaube aber meinen String 
richtig im Flash liegen zu haben:
1
const char spi_start[] PROGMEM = "Aktiviere SPI...";
2
//...
3
void funktion(PGM_P s)
4
{
5
//...
6
  c= pgm_read_byte_far(FAR(s));
7
}

Hat da jemand schon eine Lösung?
Hy!
Versuche gerade einen Bootloader mit dem Usb Treiber von Objective 
Development auf einem Mega128 zum laufen zu kriegen.
Ich bekomme mit dem Makro leider auch Probleme, erhalte 3 malig die 
Fehlermeldung:
1
undefined reference to `r30'

Aufgebaut hab ich das so:
1
#define GET_FAR_ADDRESS(var)                          \
2
({                                                    \
3
    uint_farptr_t tmp;                                \
4
                                                      \
5
    __asm__ __volatile__(                             \
6
                                                      \
7
            "ldi    %A0, lo8(%1)"           "\n\t"    \
8
            "ldi    %B0, hi8(%1)"           "\n\t"    \
9
            "ldi    %C0, hh8(%1)"           "\n\t"    \
10
            "clr    %D0"                    "\n\t"    \
11
        :                                             \
12
            "=d" (tmp)                                \
13
        :                                             \
14
            "p"  (&(var))                             \
15
    );                                                \
16
    tmp;                                              \
17
}) 
18
#define PRG_RDB(addr)   pgm_read_byte_far( GET_FAR_ADDRESS(addr) )

Der Aufruf sieht dann folgendermaßen aus:
1
uchar *r = usbMsgPtr;
2
uchar c = PRG_RDB(r);

Wie kommt der Compiler da auf die Idee r30 anzumeckern?

Viele Grüsse
Hmm leider fehlt mir das nötige wissen über inline Assembler um da 
richtig durchzublicken.

Ich hab jetzt eine andere Lösung gefunden die für mich funktioniert:
1
#define PRG_RDB(addr)   pgm_read_byte_far(0x10000UL + (uint32_t) (uint16_t) addr)

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 21 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.