PROGMEM Problem

[Nino K. (lnino)]

Hi an alle.

Ich habe eine Funktion mit welcher man einen Text am LCD scrollend 
ausgeben kann. Da ich aber nicht immer den Code eingeben möchte, möchte 
ich eine eigene lib daraus machen. Entweder kommen beim compelieren 
immer neue Fehlermeldungen oder am Display erscheinen nur schwarze 
Striche.

Hier der Code der Funktion innerhalb eines normalen C Projekts. So 
funktioniert er ohne Probleme:

#define F_CPU 14745600

#include <stdio.h>

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

#include "../libnerdkits/delay.h"
#include "../libnerdkits/lcd.h"
  
  // prototypes
  void scroll_line_two(char);

int main() {

  lcd_init();
  lcd_home();
  
  scroll_line_two("TEST");

while(1) {   

} 
  
  return 0;
}

void scroll_line_two(char a)
{
  int8_t scrollPos=0;   

  uint8_t i;   

  const char *linetwo = PSTR(a);

  uint8_t linetwolen = strlen_P(linetwo);
  
    while(1) {   
 
    lcd_line_two();     

      for(i=0; i<20; i++) {     

      lcd_write_data(pgm_read_byte(linetwo + ((scrollPos+i)%linetwolen)));   
      }     

    scrollPos = (scrollPos + 1) % linetwolen;     
  
    delay_ms(200); 
    }
}

Wenn ich nun versuche die Funktion in eine eigene Lib zu geben um sie 
dann mittels include einzubinden, geht der Code plötzlich nicht mehr. 
Habe mich dann im Netz etwas umgesehen und dann herausgefunden, dass ich 
char* statt char schreiben muss, usw. Aber die richtige Kombination 
scheine ich nicht gefunden zu haben.

Hier der Code in der lcd4x20ext.c und lcd4x20ext.h

lcd4x20ext.c

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

#include "lcd.h"
#include "delay.h"

void scroll_line_two(char* a)
{
  int8_t scrollPos=0;   

  uint8_t i;   
 
  const char *linetwo = PSTR(a);

  uint8_t linetwolen = strlen_P(linetwo);
  
    while(1) {     
    lcd_line_two();     

      for(i=0; i<20; i++) {        
      lcd_write_data(pgm_read_byte(linetwo + ((scrollPos+i)%linetwolen)));   
      }     
 
    scrollPos = (scrollPos + 1) % linetwolen;     
  
    delay_ms(200); 
    }
}

lcd4x20ext.h

#ifndef __LCD4x20EXT_H
#define __LCD4x20EXT_H

void scroll_line_two(char* a);

#include <stdio.h>
int lcd_putchar(char x, FILE *stream);

#endif

Und hier noch das C Project in welchem ich versuche die lcd4x20ext.h zu 
includen und darauf zuzugreifen.

lcd4x20ext.c

#define F_CPU 14745600

#include <stdio.h>

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

#include "../libnerdkits/delay.h"
#include "../libnerdkits/lcd.h"
#include "../libnerdkits/lcd4x20ext.h"
  
int main() {

  // fire up the LCD
  lcd_init();
  lcd_home();
  
  scroll_line_two("TEST TEST TEST TEST");

// Main loop. 
while(1) {   

} 
  
  return 0;
}

Kann mir jemand sagen, was ich da noch falsch mache?

Vielen Dank für die Hilfe.

[Karl H. (kbuchegg)]

El Nino schrieb:

> Hier der Code der Funktion innerhalb eines normalen C Projekts. So
> funktioniert er ohne Probleme:

Glaub ich nicht.
Der ist voller Fehler


> Habe mich dann im Netz etwas umgesehen und dann herausgefunden, dass ich
> char* statt char schreiben muss,

Du solltest dich weniger im Netz umsehen, sondern besser ein C-Buch, 
nämlich dein ganz persönliches C-Buch zu Rate ziehen


>   const char *linetwo = PSTR(a);
>

Du kannst hier nicht PSTR benutzen.
PSTR ist ein Makro, welches einen konstanten Text während des 
Compilierens ins Flash verschiebt und einen Pointer darauf liefert. Du 
hast hier keinen konstanten Text! Die Verschiebung, und damit die 
Anwendung des PSTR muss bereits der Aufrufer der Funktion machen. 
Derjenige, der die Stringkonstante hat, muss sie mittels PSTR ins Flash 
verlagern.

D.h. der Aufrufer der Funktion gibt dir bereits den korrekten Pointer 
ins Flash:

void scroll_line_two(char* linetwo)
{
....


> int main() {
>
>   // fire up the LCD
>   lcd_init();
>   lcd_home();
>
>   scroll_line_two("TEST TEST TEST TEST");

und hier dann

   scroll_line_two( PSTR("TEST TEST TEST TEST") );

[Nino K. (lnino)]

Hi Karl Heinz,

danke für deine Rückmeldung.

Ich hoffe ich habe dich richtig verstanden, dass der Code dann 
folgendermaßen aussehen soll:

lcd4x20ext.c

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

#include "lcd.h"
#include "delay.h"

void scroll_line_two(char* linetwo)
{
  int8_t scrollPos=0;   

  uint8_t i;   
 
  //const char *linetwo = PSTR(a);

  uint8_t linetwolen = strlen_P(linetwo);
  
    while(1) {     
    lcd_line_two();     

      for(i=0; i<20; i++) {        
      lcd_write_data(pgm_read_byte(linetwo + ((scrollPos+i)%linetwolen)));   
      }     
 
    scrollPos = (scrollPos + 1) % linetwolen;     
  
    delay_ms(200); 
    }
}

lcd_test.c

#define F_CPU 14745600

#include <stdio.h>

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

#include "../libnerdkits/delay.h"
#include "../libnerdkits/lcd.h"
#include "../libnerdkits/lcd4x20ext.h"
  
int main() {

  // fire up the LCD
  lcd_init();
  lcd_home();
  
  scroll_line_two( PSTR("TEST TEST TEST TEST") );

// Main loop. 
while(1) {   

} 
  
  return 0;
}

Ist das so korrekt oder stehe ich noch auf der Leitung?

Danke für deine Hilfe.

[Nino K. (lnino)]

Habe nun versucht den Code dementsprechend umzuschreiben.

Wenn ich aber versuche mit meinem makefile das umzuwandeln, dann erhalte 
ich folgenden Fehler:

ed/scrolling_text_embedded.c:24: undefined reference to `scroll_line_two'
make: *** [scrolling_text_embedded.hex] Error 1

In Zeile 24 steht folgendes:

scroll_line_two( PSTR("TEST TEST TEST TEST TEST") );

EDIT: Habe nun den Fehler gefunden. Habe in meinem Makefile nicht das 
korrekte .o file angegeben.