__DATE__ in Bootloader

[Peter O. (Gast)]

Hallo Forum,

ich möchte gerne die GCC Defines für das Kompilierdatum auch im 
Bootloader benutzen, bin mir aber nicht sicher, ob ich es korrekt 
umgesetzt habe.

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

//gcc builtin defines:
static const char PROGMEM CompileDateP[12] = __DATE__;
static const char PROGMEM CompileTimeP[9]  = __TIME__;
static const char PROGMEM CompilerVerP[6]  = __VERSION__;

//In der Main vor der Endlosschleife einmalig:
unint8_t str[31];
unint8_t i;

//Read compile date and time
  str[ 0]='R';
  str[ 1]='C';
  str[ 2]='D';
  str[ 3]='T';
  str[ 4]=' ';
  for(i=0;i<=11;i++) {
    str[5+i] = pgm_read_byte_far(&CompileDateP[i]);
    //str[5+i] = '1';
  }
  str[16]=' ';
  for(i=0;i<=8;i++) {
    str[17+i] = pgm_read_byte_far(&CompileTimeP[i]);
    //str[17+i] = '2';
  }
  str[25]=' ';
  for(i=0;i<=5;i++) {
    str[26+i] = pgm_read_byte_far(&CompilerVerP[i]);
    //str[26+i] = '3';
  }

  ua_sendbuf((unint8_t*)str,31);

Prinzipiell erhalte ich die Ausgage, die ich erwarte.

Beim pgm_read_byte_far() bekomme ich allerding noch jeweils eine 
Warnung:

warning: cast from pointer to integer of different size

Wie bekomme ich die weg?
Und meine eigentliche Frage ist, ob ich das so machen kann, oder ob ich 
da evtl. Probleme mit meinem eigentlichen Programm bekommen kann?
Liegt z.B. CompileDateP wirklich im Flash des Bootloaderbereichs?

[Peter O. (Gast)]

Noch als Zusatzinfo:
Ich nutze einen AT90CAN128 mit den Fuses:
LFUSE = 0x5f
HFUSE = 0xD8
EFUSE = 0xFB

[(prx) A. K. (prx)]

Nur so nebenbei: Man kann sich das Leben auch mit dem Gesamtstring

   ... [] = "RCDT " __DATE__ " " __TIME__ " " __VERSION__;

erleichtern. Auch wenns nicht gleich ins Auge sticht, ist das doch nur 
EIN String.

[bal (Gast)]

PROGMEM ist eine obsolete Krücke und noch dazu fehleranfällig.

[Fläsch Rom (Gast)]

bal schrieb:
> PROGMEM ist eine obsolete Krücke und noch dazu fehleranfällig.

Und was soll man stattdessen verwenden?

[Peter O. (Gast)]

A. K. schrieb:
> Nur so nebenbei: Man kann sich das Leben auch mit dem Gesamtstring   ...
> [] = "RCDT " _DATE_ " " _TIME_ " " _VERSION_;
> erleichtern. Auch wenns nicht gleich ins Auge sticht, ist das doch nur
> EIN String.

Stimmt, das reicht hier im Prinzip völlig aus.
Danke für den Tipp.

[bal (Gast)]

Fläsch Rom schrieb:
> bal schrieb:
>> PROGMEM ist eine obsolete Krücke und noch dazu fehleranfällig.
>
> Und was soll man stattdessen verwenden?

__flash, und dem Compiler die Drecksarbeit machen lassen.
Der Code wird auch hübscher.