Problem mit pgm_read_byte_far

Gast #237045
Lesenswert?

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 #237046
Lesenswert?

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 #237047
Lesenswert?

"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 #237050
Lesenswert?

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 #237051
Lesenswert?

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 #787625
Lesenswert?

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 #790087
Lesenswert?

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
#850290
Lesenswert?

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?
#876930
Lesenswert?

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

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

oder

Mit Google-Account einloggen

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren