Das verstehe ich nicht.

[Chricky (Gast)]

Hallo zusammen,

ich habe folgenden Code auf einem ATMega162 laufen:

#define F_CPU 8000000UL

#include <uart.c>
#include <util\delay.h>
#include <string.h>

const char GrussStr1[] PROGMEM     = "Hallo...\r";
const char GrussStr2[] PROGMEM     = "Guten Tag...\r";
 
const char* const Gruesse[] PROGMEM = 
{
   GrussStr1,
   GrussStr2
};

void SendString_P(int UART, const PGM_P aString)
{
   char Data[128]     = "\0";

   strcpy_P(Data,aString);
   uart0_puts(Data);
}

int GetGruss(void)
{
   int Firma    = 1;

//   for (Firma=1; Firma<=2; Firma++)
   {
      SendString_P(0,Gruesse[Firma-1]);

      _delay_ms(2000);
   }
   return 0;
}

int main(void)
{
   int GrussNr;

   uart0_init(UART_BAUD_SELECT(19200,F_CPU));
   sei();

   while (1) 
   {
      GrussNr=GetGruss();
   }
   return 1;
}

Der µC sendet - wie erwartet - "Hallo..." an die Com.
Ändere ich in GetGruss int Firma    = 1; auf int Firma    = 2; sendet 
der µC - wie erwartet - "Guten Tag..." an die Com.
Nehme ich nun die for-schleife wieder rein erhalte ich nur noch 
ASCII(12)F.

Was läuft hier falsch?

(Die UART ist von Peter Fleury)

Gruß
Chricky

[Karl H. (kbuchegg)]

Da das Array Grüesse selbst auch im FLash liegt, kannst du hier nicht 
einfach

     SendString_P(0,Gruesse[Firma-1]);

über eine Indexoperation zugreifen.

Du musst dir den bewussten Pointer zum Text selbst auch wieder mit einem 
pgm_read_word holen

      const PGM_P text = pgm_read_word( &Gruesse[Firma-1] );
      SendString_P( 0, text );

> Ändere ich in GetGruss int Firma    = 1; auf
> int Firma    = 2; sendet der µC - wie erwartet - "Guten Tag..."
> an die Com.

Es ist zwar auch in dem Fall falsch, allerdings rettet dich hier der 
Optimizer, der den ganzen Zugriff durch das Ergebnis ersetzt (das er ja 
kennt, weil es konstant ist). D.h. der Zugriff ins Gruesse Array, 
welcher eigentlich ins Flash gehen müsste (es aber nicht tut), findet 
überhaupt nicht statt. Und deswegen funkioniert das (und ist ein gutes 
Beispiel für: nur weil es funktionioert, muss es noch lange nicht 
richtig sein)

[Chricky (Gast)]

Als Workaround funktioniert Folgendes:

int GetGruss(void)
{
   int Firma    = 1;

   for (Firma=1; Firma<=2; Firma++)
   {
      if (Firma==1) SendString_P(0,Gruesse[0]);
    if (Firma==2) SendString_P(0,Gruesse[1]);

      _delay_ms(2000);
   }
   return 0;
}

Wunder...

[Karl H. (kbuchegg)]

Chricky schrieb:
> Als Workaround funktioniert Folgendes:

Wie schon gesagt:
'Funktioniert' bedeutet nicht, dass es richtig ist
Auch das ist immer noch falsch. Dein Programm funktioniert zufällig, 
weil dir der Optimizer unter die Arme gegriffen hat.

so wäre es richtig

      const PGM_P text = pgm_read_word( &Gruesse[Firma-1] );
      SendString_P( 0, text );

oder in einem Rutsch ohne Zwischenvariable

      SendString_P( 0, pgm_read_word( &Gruesse[Firma-1] ) );

[Chricky (Gast)]

Hallo Karl Heinz Buchegger,

ja das macht Sinn! Danke. Manchmal sieht man eben den Wald nicht mehr...


Gruß
Chricky

[Chricky (Gast)]

Das hat sich jetzt 2mal überschnitten...

Danke.

Gruß
Christian

[Karl H. (kbuchegg)]

Allerdings würde ich anstelle deiner SendString Funktion die originale 
bevorzugen. In der Fleury Lib ist nämlich schon eine Funktion drinnen, 
die einen String aus dem Flash ausgeben kann

      uart0_puts_p( 0, pgm_read_word( &Gruesse[Firma-1] ) );

und die hat nicht die Beschränkung auf 128 Zeichen, so wie deine 
SendString_P

[Chricky (Gast)]

Hallo Karl Heinz Buchegger,

ja, die habe ich gesehen. Ich muß allerdings noch Steuer- und 
Prüfzeichen anhängen. Ich hatte die Funktionen nur aufs Nötigste 
zusammengestrichen...

Gruß
Christian

[Karl H. (kbuchegg)]

Chricky schrieb:
> Hallo Karl Heinz Buchegger,
>
> ja, die habe ich gesehen. Ich muß allerdings noch Steuer- und
> Prüfzeichen anhängen. Ich hatte die Funktionen nur aufs Nötigste
> zusammengestrichen...

Dann sieh dir an, wie Peter den String aus dem Flash ausgibt. Ohne 
Array.
Das kannst du auch.