PROGMEN Array in Schleife verarbeiten

Auf Werte im Flash kannst du nicht direkt zugreifen. Du musst sie mit 
den dafür vorgesehenen Funktionen aus dem Flash lesen. Das die eine 
Variante auch mit direktem Zugriff zu funktionieren scheint, liegt 
daran, dass der Optimierer hier gleich die Zugriffe durch die 
entsprechenden Werte ersetzt.

Die Vergleichsbedingung in der for Schleife ist falsch, Du willst wohl:

1

uint8_t sub=0;

2

3

//Schleife, die nicht funktioniert:

4

for(uint8_t j=0; j<=1; j++){

5

sub= &MENUSTR[j] + sub);

6

}

http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Programmspeicher_.28Flash.29

Wenn es ein AVR + GCC ist, was mir meine Glaskugel sagt... *Zaunpfahl 
wink*

Und nachdem ich nochmal drauf geschaut habe, wage ich auch zu 
bezweifeln, dass die funktionierende Variante tatsächlich so mit den '&' 
darin funktioniert.

Alex schrieb:

> War es so gemeint wegen der automatischen Optimierung?

Was meinst du mit "automatischen Optimierung"?

Bei der Schleifen-Variante führt kein Weg an den Flash-Funktionen 
vorbei.

Alex schrieb:
> Ob das die gewünschten Funktionen sind, mit denen man die Werte korrekt
> aus dem Flash liest?

Ich sehe in dem Code gar keine Funktionen.

Sieh dir den Link an, den docean gepostet hat.

Alex schrieb:
> pgm_read_byte klappt allerdings nicht

Dann hast du es falsch benutzt.
Wie sieht dein Code dazu aus?

Und ich verstehe nicht, was du jetzt die ganze Zeit mit dem "Optimieren" 
meinst. Der Optimierer hat nur unbeabsichtigterweise dafür gesorgt, dass 
ein Teil deines Codes zufällig funktioniert hat, obwohl er eigentlich 
fehlerhaft war.

Ist richtig. Was genau klappt damit nicht?

Alex schrieb:

> Hatte es vorher ja auch mit pgm_read_byte versucht aber dann einfach mal
> weg gelassen. Und es ging.

Zeig mal dein ganzes Programm mit dem du testest.

Sorry, aber das funktioniert so nie und nimmer. Nicht nur, dass der 
richtige Flashzugriff fehlt, du addierst auch Pointer auf. Wenn du ein 
"ok" bekommst, ist das reiner Zufall.

Tja.
Und aus der Ausgabe 'OK' bzw. 'Nicht OK' kann man so wunderbare 
Rückschlüsse darauf ziehen, was denn das Problem ist. :-)

Warum lässt du dir denn nicht einfach mal sub ausgeben!
Aus dem Zahlenwert sieht man doch viel mehr als aus einer OK/Nicht OK 
Meldung.


for(uint8_t j=0; j<=1; j++){
    sub= &MENUSTR[j] + sub;
    }

Das ist Unsinn!
Du MUSST über pgm_read_byte gehen.
Alles andere kann ev. das Richtige ergeben (weil ja immer an jeder 
Speicherstelle irgendetwas drinnen steht und jedes Teil eine Adresse 
hat), aber das ist dann Zufall!

Und achte ein bischen auf die äußere Form deines Programmes. 
Interessanterweise sind es nämlich meistens die 
Hässlichen-Kraut-und-Rüben Programme, die solch seltsame sporadische 
Fehler aufweisen.

1

#include <stdio.h>

2

#include <string.h>

3

4

// AVR include files

5

#include <avr/io.h>

6

#include <avr/interrupt.h>

7

#include <avr/pgmspace.h>

8

9

//P.Fleury

10

#include "lcd.h"

11

12

uint8_t sub=0;

13

14

const uint8_t MENUSTR[] PROGMEM ={

15

  3, 

16

  7, 

17

  3, 

18

  0, 

19

};

20

21

void lcd_puti( int number )

22

{

23

  char buffer[10];

24

25

  itoa( number, buffer, 10 );

26

  lcd_puts( buffer );

27

}

28

29

int  main()

30

{  

31

  DDRG &=(1<<PG2);

32

  DDRA &=(1<<PA4) & (1<<PA5) & (1<<PA6) & (1<<PA7);

33

34

  //Werden noch nicht benötigt

35

  PORTG |= (1<<PG2);  // Taster Pullups

36

  PORTA |= (1<<PA4) | (1<<PA5) | (1<<PA6) | (1<<PA7); // Taster Pullups

37

38

  //   LCD Initialisierung

39

  lcd_init (LCD_DISP_ON);

40

  lcd_clrscr();

41

42

  sub = 0;

43

  for( uint8_t j = 0; j <= 1; j++ ) {

44

    sub = sub + pgm_read_byte( &MENUSTR[j] );

45

  }

46

47

  lcd_gotoxy( 1, 1 );

48

  lcd_puti( sub );

49

50

  if( sub == 10 ) {

51

    lcd_puts("ok");

52

  }

53

  else {

54

    lcd_puts("nicht ok");

55

  }

56

57

  for (;;)

58

  {

59

  }

60

}