AVR, EEProm, I2C, Fehler in twitest?

OP #1808808 
Ich vermute, daß es etliche AVR-Anwender gibt, die ihren Microcontroller 
durch ein I2C-EEProm ergänzen.
Nun liegt dankenswerterweise jeder WinAVR-Version ein Beispiel für die 
Ansteuerung eines EEProms mittels I2C (twi) bei (z.B. 
...\WinAVR-20100110\doc\avr-libc\examples\twitest). Obwohl ich diese 
c-Implementierung (modifiziert) seit längerem benutze, ist mir erst 
jetzt ein Fehler aufgefallen.
Werden mehr als ein Byte drei Byte vor einer Page-Grenze geschrieben, so 
wird ein Byte zuviel ins EEProm geschrieben.
Der Fehler liegt m.E. in einer falschen Vergleichsabfrage in der 
Funktion int ee24xx_write_page(uint16_t eeaddr, int len, uint8_t *buf).
Falsch:
...
  if (eeaddr + len < (eeaddr | (PAGE_SIZE - 1)))
    endaddr = eeaddr + len;
...
Richtig:
...
  if (eeaddr + len <= (eeaddr | (PAGE_SIZE - 1)))
                   ===
    endaddr = eeaddr + len;
...

Kann jemand aus dem Forum meine Feststellung verifizieren? Das 
twitest-Beispiel wird schon seit vielen Jahren so publiziert. Vielleicht 
ist schon anderen dieses Verhalten aufgefallen - oder habe ich mich 
einfach geirrt?

Harald Peters
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Langsamer Modus Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Melden Danke 
Der Code ist ja ziemlich unübersichtlich.
Ich würde modular programmieren:
1
static void si2c_start( void );

2
static void si2c_stop( void );

3
static uint8_t si2c_r( uint8_t nack );

4
static uint8_t si2c_w( uint8_t val );


Und dann die Schreibfunktion für nen 24C512:
1
uint8_t eeprom_wr( uint16_t eeaddr, uint8_t *sram, uint16_t len )

2
{

3
  do{

4
    if( !set_addr( eeaddr ))

5
      return 0;                                         // 0 = failed

6
    do

7
      si2c_w( *sram++ );

8
    while( --len && (++eeaddr & (PAGE_SIZE-1)));        // end or next page

9
    si2c_stop();

10
  }while( len );                                        // end

11
  return 1;                                             // 1 = success

12
}


Dann ist es auch wurscht, ob man das I2C in HW oder SW macht.


Peter
Antwort Zitieren Bearbeiten Löschen Melden Danke
OP 
Ich bin wie Peter der Meinung, daß man dieses Teil tatsächlich noch 
einmal sauber programmieren sollte. Mit den vielen GoTos ist das 
Programm doch recht unübersichtlich. Aber wie dem auch sei, es sollte 
wenigstens korrekt sein.
Harald
Antwort Zitieren Bearbeiten Löschen Melden Danke
Thread beobachten

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 16 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

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.