Auslesen einer AtMega Flash-Page im Bootloader

Florian V. OP #3653362
Lesenswert?
• ▲
▼ 
Hallo liebe Leute,

ich schreibe zur Zeit einen Bootloader, mit welchem es möglich ist per 
RS485 den Flash eines AVR-AtMega 88 page-weise zu beschreiben.

Um die geschriebene Page zu verifizieren möchte ich diese nach dem 
Schreiben auslesen und mit meinem Daten-Puffer vergleichen, welcher als 
Pointer vorliegt. Dies soll eventuelle Schreibfehler erkennen.

1
uint8_t *written_data = message;  // To avoid undeclaration

2
uint8_t *flash_data = message +3;

3


4
program_page(write_page, flash_data);

5


6
for(uint8_t word = 0; word < (SPM_PAGESIZE/2); word++){

7
  written_data[0+word*2] = pgm_read_word(write_page + (word*2));

8
}

9


10
if(written_data == flash_data){

11
  response[0] = CORRECT_SIZE;

12
}

13
else response[0] = FAIL_SIZE;


Ich habe nun schon mehrere Stunden damit verbracht den Code zum laufen 
zu bekommen, leider erfolglos.
Über eine übergeordnete Ausgabestruktur wird mir immer "FAIL_SIZE" 
zurückgegeben.

Ich würde mich freuen, wenn sich jemand den Code anschauen und meine 
Fehler finden könnte.

Liebe Grüße
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden
Daniel A. #3653466
Lesenswert?
• ▲
▼ 
Florian V. schrieb:
>
> uint8_t *written_data = message;  // To avoid undeclaration

Der kommentar ist hier etwas verwirrend, was ist damit gemeint?

> for(uint8_t word = 0; word < (SPM_PAGESIZE/2); word++){
>   written_data[0+word*2] = pgm_read_word(write_page + (word*2));

uint16_t hat in uint8_t nicht platz. Jedes zweite element von 
written_dat zu überschreiben macht keinen sinn.

Ist write_page vom type uint8_t* oder unsigned char*?
War hier soetwas gemeint? ((uint16_t*)written_data)[word] = 
pgm_read_word(write_page + (word*2));

> if(written_data == flash_data){
Du vergleichst nur Adressen, was zur überprüfung vollkomes sinlos ist.
Ausserdem wird der Pointer written_data nie verändert, der vergleich 
muss false ergeben.

Mach den vergleich in der  Schleife. z. B. so:

1
uint8_t word;

2
for(word = 0; word < (SPM_PAGESIZE/2); word++)

3
  if((uint16_t*)flash_data)[word] != pgm_read_word(write_page + word*2)) 

4
    break;

5


6
if(word == (SPM_PAGESIZE/2))

7
  response[0] = CORRECT_SIZE;

8
else

9
  response[0] = FAIL_SIZE;
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Florian V. OP #3654307
Lesenswert?
• ▲
▼ 
Daniel A. schrieb:
> Mach den vergleich in der  Schleife. z. B. so:
>
1
> uint8_t word;

2
> for(word = 0; word < (SPM_PAGESIZE/2); word++)

3
>   if((uint16_t*)flash_data)[word] != pgm_read_word(write_page + word*2))

4
>     break;

5
> 

6
> if(word == (SPM_PAGESIZE/2))

7
>   response[0] = CORRECT_SIZE;

8
> else

9
>   response[0] = FAIL_SIZE;

10
> 

11
>


Vielen Dank für die Idee.
Es hat nach ein paar Anpassungen wunderbar funktioniert.

Da mein flash_data ein uint8_t* Pointer ist, habe ich pgm_read_word auf 
uint8_t gecastet und nur um "word" erhöht. Werde es in Byte umbenennen.

1
if(flash_data[word] != (uint8_t)pgm_read_word(write_page + word))


Vielen Dank noch einmal für eure Hilfe.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Thread beobachten

Antwort schreiben

Hinweis: der ursprüngliche Beitrag ist mehr als 6 Monate alt. Bitte hier nur auf die ursprüngliche Frage antworten, für neue Fragen einen neuen Beitrag erstellen.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Dein Name und deine E-Mail-Adresse

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.