AVR: I2C Master Software Library für alle AVR

Ein paar Sachen, die mir beim schnellen Lesen aufgefallen sind:

vor dem Ausgeben eines Starts wird nicht geprüft, ob der Bus Idle ist

Wenn der Slave (oder auch die Kapazität des Busses) den Clock streched, 
wird die minimale High-Zeit des Busses verletzt

MfG Klaus

Christian J. schrieb:
> Viel Spass!

Seltsam die invertierte Namensgebung der Signale

Beispiel:

1

#define SOFT_I2C_SDA_LOW  SDADDR|=((1<<SDA))

Wer soll da durchblicken wenn man es schnell verstehen soll?

Arduinoquäler schrieb:
> Wer soll da durchblicken wenn man es schnell verstehen soll?

Glaube dass der Code auch nicht für Arduino Benutzer geschrieben wurde.
Für die ist das zu hoch....

Klaus schrieb:
> Ein paar Sachen, die mir beim schnellen Lesen aufgefallen sind:
>
> vor dem Ausgeben eines Starts wird nicht geprüft, ob der Bus Idle ist
>
> Wenn der Slave (oder auch die Kapazität des Busses) den Clock streched,
> wird die minimale High-Zeit des Busses verletzt
>
> MfG Klaus

Ändere es und stelle den Code neu ein. Fertig ist ja nie was. Nur 
geprüft sollte es sein auf dem Breadboard. Fehlerhafte Codes gibt es 
genug im Netz. Auch dieser war es weil der Autor einfach nicht alle 
Funktionen gestestet hat.

Arduinoquäler schrieb:
> Christian J. schrieb:
>> Viel Spass!
>
> Seltsam die invertierte Namensgebung der Signale
>
> Beispiel:
> #define SOFT_I2C_SDA_LOW  SDADDR|=((1<<SDA))
>
> Wer soll da durchblicken wenn man es schnell verstehen soll?
Dann musst du einen PIC* nehmen, da entspricht 0 einem Ausgang.

* Vielleicht gibt es auch andere µC wo das so ist, ich kenne nur den 
PIC.

PICianer schrieb:
> Dann musst du einen PIC* nehmen, da entspricht 0 einem Ausgang.

Hä?

Wenn ich beim PIC eine 1 schreibe bekomme ich am Ausgang eine 0?

Arduinoquäler schrieb:
> PICianer schrieb:
>> Dann musst du einen PIC* nehmen, da entspricht 0 einem Ausgang.
>
> Hä?
>
> Wenn ich beim PIC eine 1 schreibe bekomme ich am Ausgang eine 0?
Nein, 1 ist Input.

Boah, jetzt müllt das hier doch nicht so zu !!!!!! Ja, beim PIC ist 1 = 
INPUT, das steht hier aber nicht zur Debatte, weil es für AVR ist und 
easy angepasst werden kann, da die Defindes die Hardware kapseln.

Christian J. schrieb:
> Glaube dass der Code auch nicht für Arduino Benutzer geschrieben wurde.
> Für die ist das zu hoch....

Wenn du eine Funktion zur Ausgabe auf SPI schreibst, dann nennst du
die vermutlich  SPI_Read, oder SPI_Input ... nicht wahr?

Ich würde sie uint8_t spiTransfer(uint8_t data) nennen.

Das ist noch mehr Grütze drin ;-( Frage mich manchmal was die Leute 
machen, die ihren Kot ins Netz blasen ohne richtig zu testen.

void i2c_init()
{
  SDAPORT&=(1<<SDA);
  SCLPORT&=(1<<SCL);

  SOFT_I2C_SDA_HIGH;
  SOFT_I2C_SCL_HIGH;

}

muss

void i2c_init()
{
  SDAPORT&= ~(1<<SDA);
  SCLPORT&= ~(1<<SCL);

  SOFT_I2C_SDA_HIGH;
  SOFT_I2C_SCL_HIGH;

}

heissen, sonst sind die anderen Bits auch platt.

Das ist doch (bis auf die üblichen Flüchtigkeitsfehler) eine gute 
Grundlage. Es hat was man "wartbar" nennt.
Hat der Mitbastler aus Indien sich eigentlich irgendwie zur Bug-Freiheit 
seines Geschenks verpflichtet?

Konrad S. schrieb:
> Ich finde es bedenklich, dass du hier Code anbietest, der nicht von dir
> stammt und ein Copyright, offenbar von einer Firma, enthält. Damit
> bewegst du dich auf sehr dünnem Eis.

Liegt sicher kein Patent drauf, zumindest auf den Fehlern nicht :-) Ist 
mir auch wurscht, Indien ist weit weg.