Problem mit pgm_read_word_far

[Der T. (Gast)]

Hi,

ich versteh's nicht.. ;-(

Ich habe im Programmspeicher folgende Daten:

const unsigned int daten[] PROGMEM = { 0xf0aa,  0x0f55, .... }

Wenn ich nun mit

val = pgm_read_word_far(FAR(daten) + offset);

darauf zugreifen will, bekomme ich nur die einzelnen Bytes (!!), auf die 
'offset' zeigt. D.h. der Reihe nach: 0x00aa, 0x00f0, 0x0055, 0x000f, 
ect.

Das pgm_read_word_far kann ich aktuell gegen ein pgm_read_byte_far 
tauschen, ohne dass es irgendwelche Auswirkungen hat. Keine 
Compiler-Warnung (GCC) ect.. ;-)

Das FAR() habe ich hier aus dem Forum und ist wie folgt definiert:

#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" \
  : "=d" (tmp)                  \
  : "p"  (&(var)));             \
tmp;                                  \
})

Wo liegt der Hund begraben..? :-)
Für Tipps wäre ich sehr dankbar!

[Davis (Gast)]

Wie ist "val" definiert/deklariert?

[Der T. (Gast)]

Sorry, natürlich unsigned int.. ;-)
(Das hat aber nichts mit den Ergebnis zu tun, das mir sonst "nur" die 
oberen 8Bit je Wort verloren gehen würden.)

[Der T. (Gast)]

Keiner eine Idee, woran das liegt? :-/

(Ich lese nun als Workaround 2x 8Bit und baue mir daraus mein Integer.)

[Stefan E. (sternst)]

Zeig mal allen relevanten Code, nicht nur Einzeiler. Dazu gehört z.B. 
auch der Code, mit dessen Hilfe du überhaupt feststellst, was gelesen 
wurde.

Davon abgesehen: Dass 'offset' tatsächlich nur ein Byte-Offset darstellt 
und nicht etwa ein Index für das Array, ist hoffentlich klar, oder?

[Der T. (Gast)]

Stefan Ernst schrieb:
> Zeig mal allen relevanten Code, nicht nur Einzeiler. Dazu gehört z.B.
> auch der Code, mit dessen Hilfe du überhaupt feststellst, was gelesen
> wurde.

Ich habe zu Eingrenzung ein extra Projekt erstellt und gebe ein Wort via 
16 LEDs (PORTA und PORTB) aus. Ansonsten gibt es wirklich nur die paar 
Zeilen..

> Davon abgesehen: Dass 'offset' tatsächlich nur ein Byte-Offset darstellt
> und nicht etwa ein Index für das Array, ist hoffentlich klar, oder?

Das ist wohl mein erstes Problem. Ich dachte, dass dies ein Index 
ist..?!?

Jedoch unabhängig davon: Sollte die Funktion nicht trotzdem (wenn auch 
verschoben) echte 16Bit-Werte liefern? z.B. bei offset=1 => 0x55f0

Ich verstehe jedoch nicht, warum mir pgm_read_word_far immer die oberen 
8Bit auf 0 setzt..?!? :-/

[Stefan E. (sternst)]

Der Techniker schrieb:
>> Davon abgesehen: Dass 'offset' tatsächlich nur ein Byte-Offset darstellt
>> und nicht etwa ein Index für das Array, ist hoffentlich klar, oder?
>
> Das ist wohl mein erstes Problem. Ich dachte, dass dies ein Index
> ist..?!?

Nein, denn "FAR(daten) + offset" ist keine Pointer-Arithmetik, sondern 
einfach nur eine Addition zweier Integer.

Der Techniker schrieb:
> Jedoch unabhängig davon: Sollte die Funktion nicht trotzdem (wenn auch
> verschoben) echte 16Bit-Werte liefern? z.B. bei offset=1 => 0x55f0

Ja.

Der Techniker schrieb:
> Ich verstehe jedoch nicht, warum mir pgm_read_word_far immer die oberen
> 8Bit auf 0 setzt..?!? :-/

Tut es mit ziemlicher Sicherheit nicht. Es ist wahrscheinlicher, dass 
irgendwo anders ein Fehler ist, der dich das nur glauben lässt. Aber 
schon klar, im Rest des Codes kann natürlich unter keinen Umständen 
irgendein Fehler sein, richtig?

Der Techniker schrieb:
> Ich habe zu Eingrenzung ein extra Projekt erstellt und gebe ein Wort via
> 16 LEDs (PORTA und PORTB) aus. Ansonsten gibt es wirklich nur die paar
> Zeilen..

Umso unverständlicher, warum man dir den Code aus der Nase pulen muss, 
und du ihn nicht einfach komplett postest.

[Der T. (Gast)]

Stefan Ernst schrieb:
> Umso unverständlicher, warum man dir den Code aus der Nase pulen muss,
> und du ihn nicht einfach komplett postest.

Frisch aus der Nase:

main.c

#include <avr/io.h>
#include <avr/pgmspace.h>

#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" \
  : "=d" (tmp)                  \
  : "p"  (&(var)));             \
tmp;                                  \
})

const unsigned int daten[] PROGMEM = { 0xf0aa,  0x0f55 };

int main (void)
{
 unsigned int var = 0;
 unsigned char offset = 0;

 DDRA = 0xff;
 DDRB = 0xff;
 
 val = pgm_read_word_far(FAR(daten) + offset);
 
 PORTA = (val >>8);
 PORTB = val;
 
 while(1);
}

Zur Fehlereingrenzung sieht das "Programm" genau so aus.
Nicht mehr und nicht weniger..

[Stefan E. (sternst)]

Welcher Controller?

Und bitte mal zusätzlich das lss-File posten.