Portierung des Atmel AES Bootloaders auf avrgcc

Anhang vergessen ...

So, ich habe nun ganz einfach den (kostenlosen) IAR Kickstart Compiler 
runtergeladen und damit kompiliert. Läuft zuverlässig.

Vielleicht habe ich ja irgendwann nochmal Zeit das Ganze avrgcc 
kompatibel zu machen.

Hallo zusammen ...

... ich sitze im Moment auch an einer Portierung des AES Bootloaders auf 
den atmega1281.
Ich benutze ebenfalls die Kickstart Version des IAR Compilers.
Aber so wie es aussieht, wird die folgende Abfrage im Modul loader.c 
[void loader(void)] mit dem else Zweig verlassen, weil die Abprüfung der 
Signatur nicht funktioniert:

----------------------------------------
// Check that the signature is correct
if ((*p++ == (unsigned char)(SIGNATURE >> 24)) &&
(*p++ == (unsigned char)(SIGNATURE >> 16)) &&
(*p++ == (unsigned char)(SIGNATURE >>  8)) &&
(*p++ == (unsigned char)(SIGNATURE >>  0))){

...
}
else
busReplyByte(ERROR_CRC);
----------------------------------------

und deshhalb wird auch das FLASH des Controllers nicht beschrieben.
Auf dem atmega128 funktioniert alles einwandfrei.
Habt Ihr da vielleicht noch eine Idee??

Falls Ihr ein schon funktionierendes Projekt auf Basis des IAR Compiler 
für den 1280/1281/2560/2561 habt, wäre ich über input sehr sehr 
dankbar!!!

Danke und Grüße
Michael

>----------------------------------------
>// Check that the signature is correct
>if ((*p++ == (unsigned char)(SIGNATURE >> 24)) &&
>(*p++ == (unsigned char)(SIGNATURE >> 16)) &&
>(*p++ == (unsigned char)(SIGNATURE >>  8)) &&
>(*p++ == (unsigned char)(SIGNATURE >>  0))){

Es könnte sein das der Compiler da etwas draus macht
was man nicht erwartet ! Das *p++ ist hier gefährlich.

----------------------------------------
// Check that the signature is correct
if ((p[0] == (unsigned char)(SIGNATURE >> 24)) &&
(p[1] == (unsigned char)(SIGNATURE >> 16)) &&
(p[2] == (unsigned char)(SIGNATURE >>  8)) &&
(p[3] == (unsigned char)(SIGNATURE >>  0))){

oder so ähnlich ist auf jeden Fall besser.

... ich bin es noch einmal

... erst einmal vielen Dank für Deine Tips Michael.

Das mit dem Löschen des "__MEMSPM__" Eintrags wusste ich bereits.
Aber das was mir wirklich weitergoholfen hat war die Beschreibung:
----------------------------------------
BOOT_SIZE=1000
FLASH_SIZE=40000
IVT_SIZE=CC
RAM_SIZE=2000
RAM_BASE=200
APP_SRAM_USAGE=41F
----------------------------------------
.
Für den atmega1281 musste ich das ganze aber noch verändern:
----------------------------------------
BOOT_SIZE=1000
FLASH_SIZE=20000
IVT_SIZE=CC
RAM_SIZE=2000
RAM_BASE=200
APP_SRAM_USAGE=41E
----------------------------------------

... un damit hat es dann funktioniert.

Vielen Dank für Eure Mühe.
Viele Grüße
Michael

Hallo zusammen,

ich bin es noch einmal ... die Protierung des Bootloaders ist so weit
fertig (Zielplattformem ATmega644 und ATmega1281).
Allerdings habe ich ein Problem wenn das verschlüsselte File
in den Controller gespielt wurde und danach der "Watchdog" zuschlägt.

Das passier ja in loader.c in der Fuktion "__C_task void loader(void)"
an der Stelle:

// Reset (let the Watchdog Timer expire)!
case TYPE_RESET:
busReplyByte(ERROR_OK);
for(;;);

Vorher wurde in der gleichen Funktion mit:

// Start the Watchdog Timer (sorry for the mess, there is a bug in
// iom8.h (in version 2.28A) which is informed to IAR).
#if 1
  #if !defined(WDTCR) && !defined(__IOM8_H)
    #define WDTCR WDTCSR
  #endif

  #if defined(__IOM16_H) || defined(__IOM32_H)
    WDTCR = (7 << WDP0) | (1 << WDE);
  #else
    #if defined(__IOM8_H)
      WDTCR = (1 << WDTOE) | (1 << WDE);
    #else
      WDTCR = (1 << WDCE) | (1 << WDE);
    #endif
    WDTCR = (7 << WDP0) | (1 << WDE);
  #endif
#endif

der Watchdog initialisier. Er schlägt auch an der oben genannten
stelle zu, allerdings wird hier, entgegen meinen Erwartungen,
der Bootloader nicht erneut ausgeführt, sonder der Controller
landet irgendwo im "Nirwana".

Ich habe mir jetzt mit folgendem Konstrukt geholfen:

// Reset (let the Watchdog Timer expire)!
  case TYPE_RESET:
    busReplyByte(ERROR_OK);
    wdt_off();
    __asm("JMP 0xF000");

... allerdings habe ich mal gehört, dass so ein "JMP" nicht im 
Bootloader
ausgeführt werden sollte.

Dieses konstrukt funktioniert im Übrigen nur auf dem 644. Hier landet
er über den JMP laut disassembler im ?RESET des Bootloaders und
überprüft anschließend den Speicher mit der Checksumme und startet dann
bei positivem Ergebnis die eigentliche Firmware.

Bei dem 1281 hingegen lautet der JMP befehl auf eine andere
Speicherstelle, die der disassembler ebenfalls mit dem ?RESET label
auflößt. Anschließend wird auch hier der Speicher überprüft.
Allerdings kommt hierbei immer eine negative Auswertung der Checksumme
heraus. Mache ich das Device anschließend Stromlos und starte es erneut,
erfolgt eine Checksummenberechnung mit positiver Bewertung, so dass
anschließend die Firmware gestartet wird.

Hat evtl. jemand eine Idee, warum durch den Watchdog der Bootloader
nicht erneut ausgeführt wird? Das wäre mit Sicherheit die eleganteste
Methode!!!

... Viele Dank für Eure Mühe ... bis denn
Michael

Hallo ...

... ich bin es noch einmal.

Es hat sich erledigt. Ein Kollege hat mich darauf gebracht!!!

Und zwar muss unmittelbar nach dem Systemstart (in den ersten Zeilen der 
"main") das MCUSR register zu Null gesetzt, und anschließend der 
WATCHDOG ausgeschaltet werden.

Das ist aber nur bei den neueren µC von Atmel notwendig.

Zu finden im Datenblatt des ATmega1281 auf S.65 in einer Fußnote:
Note: 1. The example code assumes that the part specific header file is 
included.
2. If the Watchdog is accidentally enabled, for example by a runaway 
pointer or brown-out condition,
the device will be reset and the Watchdog Timer will stay enabled. If 
the code is not set up
to handle the Watchdog, this might lead to an eternal loop of time-out 
resets. To avoid this situation,
the application software should always clear the Watchdog System Reset 
Flag
(WDRF) and the WDE control bit in the initialisation routine, even if 
the Watchdog is not in use.

... da gehört doch wohl so ein wichtiger Kommentar nicht hin, oder?!?

Sei es drum ... danke Trotzdem für Eure Hilfe
Michael