AVR I2C Anfängerprobleme

[Peter (Gast)]

Hallo,
ich versuche von Arduino wegzukommen und von Grund aus die Programme 
selber zu programmieren. Jetzt möchte ich gerne zwischen zwei ATmega via 
I2C kommunizieren. Auf dem Slave läuft dazu noch das Arduino Programm 
welches via I2C empfangene Daten auf dem Serialport ausgibt. Auf dem 
Master habe ich das Programm laufen welches ich aus dem Datenblatt vom 
Atmega erlesen habe. Nur leider gibt das Programm sofort beim ersten 
Statuscheck eine Fehlerroutine aus (blinkende LED). könntet ihr bitte 
das Programm prüfen und mir sagen wo der Fehler liegt? Danke!

#define START 0x08
#define Slave 1
#define MT_SLA_ACK 0x18
#define MT_DATA_ACK 0x28
#define F_CPU 16000000

#include <avr/io.h>
#include <util/delay.h>

uint8_t Zahl = 0;

void Error(void)
{
  while (1)
  {
PORTB &= ~(1<<PB5);
_delay_ms(100);
PORTB |= (1<<PB5);
_delay_ms(100);  
  }
}

void TWIStart(void)
{
  TWCR = (1<<TWINT) | (1<<TWSTA) | (1<<TWEN);
  while (!(TWCR & (1<<TWINT)));
}

void TWIStop(void)
{
  TWCR = (1<<TWINT) | (1<<TWEN) | (1<<TWSTO);
}

int main(void)
{
  DDRB |= (1<<DDB5);
  PORTB &= ~(1<<PB5);
  _delay_ms(1000);
  
    while(1)
  {
    PORTB |= (1<<PB5);
    TWIStart();
    if ((TWSR & 0xF8) != START) Error();
    TWDR = (Slave<<1);
    TWCR = (1<<TWINT) | (1<<TWEN);
    while (!(TWCR & (1<<TWINT)));
    if ((TWSR & 0xF8) != MT_SLA_ACK) Error();
    TWDR = Zahl;
    TWCR = (1<<TWINT) | (1<<TWEN);
    while (!(TWCR & (1<<TWINT)));
    if ((TWSR & 0xF8) != MT_DATA_ACK) Error();
    TWIStop();
        Zahl++;
    _delay_ms(1000);
    PORTB &= ~(1<<PB5);
    _delay_ms(1000);
    }
}

[Hubert G. (hubertg)]

Bitrate etwas zu hoch?
PullUp vorhanden?

[Falk B. (falk)]

@ Peter (Gast)

>ich versuche von Arduino wegzukommen und von Grund aus die Programme
>selber zu programmieren.

Warum? Ist es dir zu langweilig? Willst du das Rad immer wieder neu 
erfinden?

[Peter (Gast)]

Hallo,

Pullups sind vorhanden! Habe keinen Vorteiler bestimmt. Vielleicht liegt 
da der Fehler? Takt ist 16 MHz. Nur da der Error schon beim ersten 
Prüfen passiert müsste ja das schon Schuld daran sein, dass nicht einmal 
ein Startbefehl funktioniert?

Warum weg von Arduino? Arduino ist perfekt zum schnellen probieren aber 
leider gibt es in den fertigen Codes viele Schleifen die einen 
Ausbremsen. Vielleicht nicht gerade beim I2C aber z.b. beim Ansteuern 
eines Displays habe ich bemerkt, dass ich mit meinem eigenen Code viel 
schneller Darstellungen habe als über die vorgegeben Codes. Darum möchte 
ich gerne meinen Code selber schreiben und wissen wenn etwas nicht geht 
warum und genau so wenn etwas geht warum!

[Frickelfritze (Gast)]

Erfinde nicht neu sondern verwende eine existierene Sammlung
von Funktionen.

http://homepage.hispeed.ch/peterfleury/avr-software.html

[Peter (Gast)]

Danke für die Sammlung. Werde ich morgen studieren! Aber trotzdem muss 
doch in meinem Versuch ein Fehler sein. Ich habe es Schritt für Schritt 
aus dem Datenblatt übernommen. Dann muss es doch auch gehen. Ich möchte 
es gerne nach dem Datenblatt machen, da es so viele MCU's gibt für die 
es leider keiner Vorgefertigten Codes gibt und die man nur an Hand vom 
Datenblatt programmieren kann. Daher jetzt mit einer MCU bei der es 
viele Lösungen gibt, da man sich das besser kontrollieren kann.

[Doppelkeks (Gast)]

Hi,

aus dem TWI tutorial:

"Zuerst muss vom Master ein START-Signal auf den Bus gegeben werden. 
Dies erfolgt durch Schreiben eines speziellen Wertes in TWCR. Nach 
Abschluss der Operation wird TWINT gesetzt. Die Logik wartet nun so 
lange, bis durch den Anwender das Flag wieder gelöscht wurde (durch 
Schreiben einer '1'). Unmittelbar nach dem Löschen von TWINT beginnt die 
Logik damit das START-Signal zu senden. "

Wenn ich das richtig verstehe musst du nach deinem TWI_Start erst das 
TWINT bit schreiben (zurücksetzen).

Denk auch dran die richtige Adresse vom Slave zu nutzen, sonst gibts 
kein ACK :)

[Peter (Gast)]

TWInt wird dich gesetzt und gewartet wird auch der Fehler kommt ja beim 
prüfen vom Status nach dem Start!

[Felix A. (madifaxle)]

Meine Initroutine sieht so aus:

// --- HW initialisieren ---
  // Festlegung der BAUD-Rate
  // SCL-Frequenz = Prozessortakt / (16 + 2 * TWBR * Prescaler)
  // Bei ~100kBaud kHz und Prozessortakt 3,6864 MHz: 
  // 16+2*TWBR*TWPS ~= 38 -> TWBR = 11 bei TWPS = 0 (CK/1)
  TWBR = bd;      // Baudrate
  TWSR = presc;   // Prescaler
  TWDR = 0xFF;    // default content, bus released
    // TWI initialisieren
  TWCR = (1<<TWEN)|(1<<TWIE)|(1<<TWINT)|(0<<TWEA)|(0<<TWSTA)|(0<<TWSTO)|(0<<TWWC);

Meine Funktion zum Starten sieht so aus (vorletzte Zeile):

void I2MGenerateStartCondition(uint8_t* daten, uint8_t anzahl)
{
  uint8_t i = 0;

    // Schnittstelle in Aktion
  i2cInUse = TRUE;

  for (i = 0; i < anzahl; i++)
  {
    i2Mtxbuffer[ i ] = daten[ i ];    // Daten in Puffer übertragen
  }

  txbufPtr = I2M_INITIAL_BUFPTR_STATE;// Beginn mit SLA an Stelle 2 (i2Mtxbuffer[1])
  rxbufPtr = 0;

    // Schnittstelle eine Startcondition generieren lassen
  TWCR = (1<<TWEN)|(1<<TWIE)|(1<<TWINT)|(1<<TWEA)|(1<<TWSTA)|(0<<TWSTO)|(0<<TWWC);
}

[Doppelkeks (Gast)]

TWCR = (1<<TWINT) | (1<<TWSTA) | (1<<TWEN);
  while (!(TWCR & (1<<TWINT)));

Während du in der while-loop wartest wird das TWINT gesetzt. Laut 
Tutorial wartet die Logik nun so lange bis das flag gelöscht wird und 
beginnt dann damit das Start-Signal zu senden. Versuche also einfach mal 
nach der while-Schleife das TWINT nochmal zu setzen um es damit zu 
löschen.

[Felix A. (madifaxle)]

Ich sehe gerade, dass du TWIE auf 0 setzt und deine I2C-Hardware damit 
deaktivierst bzw. den INT abschaltest.

Ah, du nutzt den Int ja nicht, vergiss das dann.

[Felix A. (madifaxle)]

Also, nach nochmaligem gründlicherem Lesen eine Frage:
was tust du mit TWBR und TWSR? Schreibst du hier die passenden Werte 
rein, damit die Hardware überhaupt einen Takt bekommt?

Wähle mal TWBR = 50 und stelle den Prescaler auf 16. Dann sollte 
zumindest schonmal die Startcondition kommen

[Dieter F. (Gast)]

Ohne mir Deinen Code angeschaut zu haben:

Bei I2C / SPI & Co. bin ich ohne Logic-Analysator bzw. Bus Pirate nicht 
glücklich geworden. Wenn Du nicht siehst was passiert fällt eine 
Fehlersuche wirklich schwer.

Es mag Experten geben, die das alles aus der hohlen Hand schütteln - ich 
gehöre nicht dazu, aber seit ich die Aktionen am Bus verfolgen kann 
finde ich meine Fehler relativ schnell.

[R. R. (elec-lisper)]

Tolle Kommentare @Rad neuerfinden. Wie sonst soll man ein tieferes 
Verständnis von Protokollen bekommen? Die TWI vom AVR zu nutzen ist ein 
super Anfang mehr zu verstehen. Der nächste Schritt ist dann I2C 
komplett in SW zu machen. Dann kann man später beruhigt auf fertige 
Bibliotheken zurückgreifen und weiss besser bescheid wo es bei Problemen 
haken könnte.

Alleine vom Quellcode lesen lernt man wenig. Aus eigenen Fehlern lernt 
man (bzw manche) viel besser.

Als nächstes kommen Ratschläge wie gleich alles sein zu lassen, weil 
alles doch schon erfunden und verbessert ist. Soll man sich ein Tablett 
kaufen. Damit kann man dann nen ganzen Bildschirm blinken lassen und 
Pay-to-Win Spiele und P**nos gucken.

[Peter (Gast)]

Hallo,
habe den Fehler gefunden. Ich habe den Takt nicht eingestellt und somit 
konnte es nicht funktionieren. Und siehe da auch mit dem Datenblatt kann 
man die Funktionen zum laufen bekommen nicht nur durch fertige Codes :)