boot_page_write funktioniert nicht

[Nu langt mir dat aber (Gast)]

Hallo Forum,

Hänge seit Tagen fest.

Ich versuche einen Bootloader zu schreiben aufgrund einiger Beispiele.

Die Daten-Blöcke werden korrekt übertragen, auch die Adressen, und 
jeweils vom Zielsystem korrekt bestätigt mit '\r'.

Der ganze Schreibvorgang läuft reibungslos durch, aber wenn ich den 
AVR-Flash hernach mit AVR-Studio und AVRISP auslese, stehen im ganzen 
Speicherbereich ab Adresse 0 bis zum Beginn des Bootloaders nur 'FF'en.

boot_page_write schreibt die Daten nicht in den Speicher.

Kennt jemand das Problem?

Ach ja, ich verwede die 2010er-WINAVR-Version (hab aber auch die 2006er 
getestet, gleiches Ergebnis), und einen ATMega168.

Hier mal meine Blockloader-Routine:

unsigned char BlockLoad(unsigned int size, ADDR_T *address)
{
    unsigned int data;
    ADDR_T tempaddress;

    (*address) <<= 1; // Convert address to bytes temporarily.
    tempaddress = (*address);  // Store address in page.

    do
  {
        data = recchar();
        data |= (recchar() << 8);
      
        boot_page_fill(*address,data);
        (*address)+=2; // Select next word in memory.
        size -= 2; // Reduce number of bytes to write by two.
  } 
  while(size); // Loop until all bytes written.

  boot_page_write(tempaddress);
  boot_spm_busy_wait();
  boot_rww_enable();

    (*address) >>= 1; // Convert address back to Flash words again.
    return '\r'; // Report programming OK
}

Danke für Eure Hilfe.

spätnächtliche Grüsse Dirk

[Daniel A. (daniel-a)]

Da die Funktion immer ein '\r' zeichen zurück gibt, sagt das nichts 
darüber aus, ob irgendetwas funktioniert hat. Vermutlich ist dass 
Problem eher in recchar.

Prüfe mal ob immer 0xff ankommt:

unsigned char BlockLoad(unsigned int size, ADDR_T *address)
{
unsigned int data,check=0xffff;
ADDR_T tempaddress;
(*address) <<= 1; // Convert address to bytes temporarily.
tempaddress = (*address); // Store address in page.
do
{
data = recchar();
data |= (recchar() << 8);
check &= data;
boot_page_fill(*address,data);
(*address)+=2; // Select next word in memory.
size -= 2; // Reduce number of bytes to write by two.
} while(size); // Loop until all bytes written.
boot_page_write(tempaddress);
boot_spm_busy_wait();
boot_rww_enable();
(*address) >>= 1; // Convert address back to Flash words again.
return check==0xffff?0:'\r';

Wenn der rückgabewert jetzt 0 ist, hat recchar immer 0xff zurückgegeben.

[Nu langt mir dat aber (Gast)]

Danke Daniel für die Antwort, hab das gecheckt, es liegt nicht am 
recchar.

Auch die Adress-Hi- bzw. Lo-Bytes habe ich per return gecheckt.

Ich kann auch direkt data einen Wert zuweisen oder den Adressen, es wird 
nichts geschrieben.

[Michael N. (betonmicha)]

Die Funktion boot_page_write(tempaddress); wird doch bestimmt einen 
Rückgabewert haben. Schreibt die Funktion überhaupt oder kommt ein Error 
zurück, den du ignorierst?

[Sepp (Gast)]

Löscht Du auch irgendwo die Page?

      boot_page_erase(Addresse);
      while(boot_rww_busy()) boot_rww_enable(); 

  
      for(fc8 = 0; fc8 < SPM_PAGESIZE; fc8 += 2) 
        {
        boot_page_fill(fc8+BlockStartAddr, *(uint *)(Data_0_PtrTX + fc8));
        }  
        
      // Write page.
      boot_page_write(Addresse);
      while(boot_rww_busy()) boot_rww_enable();

[Nu langt mir dat aber (Gast)]

Das mit dem Rückgabewert muss ich mir anschauen.

Zum Testen habe ich ganz minimalistisch direkt ins main geschrieben:

  unsigned int size=256;
  unsigned int adr=0;

  boot_page_erase(adr);
  boot_spm_busy();
  boot_rww_enable();

  do
  {

        boot_page_fill(adr,0xEEEE);

    adr+=2; // Select next word in memory.
        size -= 2; // Reduce number of bytes to write by two.

  }
  while(size); // Loop until all bytes written.

  boot_page_write(0);
  boot_spm_busy();
  boot_rww_enable();

  sendchar('Y');
  for(;;);

Das müsste doch in den ersten Block ab Adresse 0 EE's schreiben. Macht 
es aber leider nicht.

Das Y kommt an.