TWI Anfängerprobleme

[Natureengeneer]

Vielen Dank das war die Lösung.

Im Studio6 gehts so: Reiter Project => Properties => Toolchain => AVR/GNU C Compiler => Directories => IncludePaths===>> ADD ITEM

[Natureengeneer]

Ich bin jetzt schon ein ganzes Stück weiter gekommen und versuche einen DS1621 zu lesen.

Sensordefines:

uint16_t ds16_temperatur;
#define DS1621_Write  0x90
#define DS1621_Read   0x91 

Einschaltfunct.:
void ds1621_einschalten(void)

{
    i2c_init();
    i2c_start(DS1621_Write);
    i2c_write(0xEE);
    i2c_stop();
    i2c_start(DS1621_Write); 
    i2c_write(0xAA);
    i2c_stop();
    _delay_ms(200); 
} 

Daraufhin=>>>

Sensorabfrage:

i2c_start(DS1621_Read);
ds16_temperatur = i2c_readAck(); // <== int16_t Rückgabewert
i2c_readNak();
i2c_stop(); // <== 

Jetzt noch ==>

Stringformatierung:

char DEES [10];
sprintf(DEES,"DSTEMP: %d",ds16_temperatur); 

Soweit so gut. Keine Fehler beim Kompilieren. Einziges Problem ist es kommt immer eine 0 an

[oberallgeier]

... DS1621 ... lesen ... Keine Fehler beim Kompilieren ... es kommt immer eine 0 an ...

Ist doch klar. Die Compiler bringen nach ein Uhr nachts immer wieder Schrott (woran das nur liegen kann???) - nur bei mir liefs gestern nach eins (nach fehlerbehafteten Abenden) sauber - ´n RC-5-Dekoder im Interrupt.

Spass beiseite. Dein guter Wille ist im Code ja schon zu sehen, aber Du musst ziemlich kräftiger wollen, wenn Du bei den Atmels bei EINEM i2c_read ZWEI Bytes (Zitat aus Deinem Code : // <== int16_t Rückgabewert) haben willst. Meine Atmels spendieren dabei immer nur ein Byte : ( .

Abhilfe: lies mal diese Schnippsel (klick) - beachte, dass dabei aus unerfindlichen Gründen (angeblich will ich, dass der Quellcode übersichtlicher ist) teilweise zwei oder drei Befehle in einer einzigen Zeile stehen. Es ist der alte Trick aus zwei Bytes ein Word zu löten. Und achte bitte auch auf die Typdefinitionen uint8_t und uint16_t.

Viel Erfolg.

[Natureengeneer]

OK, weiterprobiert...

unsigned char ret, TempH, TempL, data, test;

void ds1621_lesen(void)
{
    
        i2c_start(DS1621_Read);
        data = i2c_readAck();
        TempH = data;
        data = i2c_readNak();
        TempL = data;
        i2c_stop();
    
    
} 

so müsste es doch klappen???? wenn ich aber TempH und TempL am LCD ausgebe, habe ich für beide den Wert 0...???

[oberallgeier]

OK, weiterprobiert ... so müsste es doch klappen ... für beide den Wert 0 ...

Sozusagen experimentelle Softwareentwicklung . . .

Ich kenne Dein dsDingsda nicht, ich will auch nicht DEIN Datenblatt für Dich lesen. Seltsam kommt mir vor, dass Du ab Register Null des Teilchens Deine Daten suchst - stimmt das so? Ich Suche meine Daten in I²C-Satelliten meist an einem bestimmten Registerplatz.

// =================================================================================
...
// ----------------------------------------------------------------------------
//      Lesen von 6 Registern ab Register 0x02. X, Y, Z-Koordinate
...
  ret = i2c_start (I2C_BMA_W); 
  ret = i2c_write (0x02); 
        i2c_stop  (); 
                                //
   ret = i2c_start (I2C_BMA_R); 
   ret = i2c_read  (ACK); x = (int16_t)ret; 
   ret = i2c_read  (ACK); x = x | ((int16_t)ret << 8); x /= 64; 
...
// ----------------------------------------------------------------------------

Steht so auch im zitierten Code .

[Natureengeneer]

Jetzt hab ichs gerafft!!!

void ds1621_lesen(void)
{
    
        
        i2c_start(DS1621_Write);
        i2c_write(0xAA);
        i2c_stop();
        i2c_rep_start(DS1621_Read);
        TempH = i2c_readAck();
        TempL = i2c_readNak();
        i2c_stop();
        ret = (TempH * 1 + TempL /256); // <===Noch nicht gut!
    
    
} 

Works!

DANKESCHÖN.

[oberallgeier]

... ret = (TempH * 1 + TempL /256); // <===Noch nicht gut! ...

Ja, sieht so aus. Es gibt sicher zahllose Beispiele um zwei Bytes zu einem Word zusammenzulöten.
(M)Eines wäre:

// ============================================================================== =
// ============================================================================== =
 void I2CTST01 (void)          // I2C-Testroutine
...
  uint8_t btst1     =  0;       // Read-back-Byte vom Slave
  uint8_t btst2     =  0;
  uint8_t btst3     =  0;
...
    btst2  = i2c_read (ACK);             //
    btst3  = i2c_read (ACK);             // 
...
//#define uniq(LOW,HEIGHT) ((HEIGHT << 8)|LOW)   // ###>>> Macro 2x 8Bit --> 16Bit
  iword         = uniq(btst3, btst4);   // 2à 8Bit --> 16Bit
  itoa(iword, zahlwort, 10);            //
  uart_puts("\tWert für Word4high3low\t");      // Ausgabe
  uart_puts(zahlwort);                  //
...
// ===  Ende I2CTST01 (void)  ============================
// ============================================================================== =

Die Makrodefiniton muss halt an irgendeiner Stelle deponiert werden - das ist Deine Freiheit. Ohne Makro gehts natürlich auch.