I2C Kommunikation Fehlerhaft

[Robert A. (Gast)]

Hallo,

An einem ATMega168 betreibe ich einen I2C BUS mit 100kHz. Als Lib 
verwende ich den von Peter F.
Als I2C Slave ist ein MCP23008 im Einsatz.
Für Testzwecke habe ich folgenden Code

  i2c_init();
  
  i2c_start(TASTER_I2C_DEVICE+I2C_WRITE);
  i2c_write(MCP23008_IODIR);
  i2c_write(0b11011111);
  i2c_stop();
  
      
  i2c_start(TASTER_I2C_DEVICE+I2C_WRITE);
  i2c_write(MCP23008_GPIO);
  i2c_write(0b11011111);
  i2c_stop();
  
  
      
  
  i2c_start(TASTER_I2C_DEVICE+I2C_READ);
  i2c_write(MCP23008_OLAT);
  uint8_t regvalue = i2c_readAck();
  i2c_stop();

Problem:
Ich bekomme 255 als Ergebnis geliefert.
Warum bekomme ich bei Index 20 (Log Auszug im Anhang) einen NACK und 
keinen ACK?

Grüße, Robert A.

[Karl H. (kbuchegg)]

Die Sequenz muss lauten

  start( Adresse + Write );
  write( Registernummer von der zu lesen ist )
  restart( Adresse + Read );
  read
  stop

Um das Register auszuwählen, von dem du lesen willst, musst du zuerst 
mittels write die Adresse übermitteln und dann mit einem Restart auf 
Lesen umschalten.


http://ww1.microchip.com/downloads/en/DeviceDoc/21919e.pdf
(Seite 6 und folgende)

[Robert A. (Gast)]

  i2c_start(TASTER_I2C_DEVICE+I2C_WRITE);
  i2c_write(MCP23008_OLAT);
  i2c_start(TASTER_I2C_DEVICE+I2C_READ);
  uint8_t regvalue = i2c_readAck();
  i2c_stop();

Ok, alles klar, funktioniert nun wunderbar.
Danke!

[Robert A. (Gast)]

Ich habe noch was:

    while(1)
    {        
    _delay_ms(100);
    lcd_command(LCD_CLEAR);
    uint8_t regvalue = mcp23008_reg_read(TASTER_I2C_DEVICE,MCP23008_GPIO);
    i2c_stop();
    show_value(regvalue);
      
    
    }

Das ist wohl die bescheurteste Zustandsabfrage der IO Pins am MCP23008 
wenn diese als Augang konsifguriert worden sind?!

Geht es auch eleganter?

[Karl H. (kbuchegg)]

Robert A. schrieb:

> Das ist wohl die bescheurteste Zustandsabfrage der IO Pins am MCP23008
> wenn diese als Augang konsifguriert worden sind?!
>
> Geht es auch eleganter?

Wenn sie als Ausgang konfiguriert sind, dann haben sie ja wohl die 
Werte, die du als letztes an die Ausgänge geschrieben hast (ausser du 
hast ein elektrisches Problem). Merk dir, was du zuletzt ausgegeben hast 
und du brauchst den MCP nicht befragen.

[Karl H. (kbuchegg)]

Karl Heinz schrieb:
> Robert A. schrieb:
>
>> Das ist wohl die bescheurteste Zustandsabfrage der IO Pins am MCP23008
>> wenn diese als Augang konsifguriert worden sind?!
>>
>> Geht es auch eleganter?
>
> Wenn sie als Ausgang konfiguriert sind, dann haben sie ja wohl die
> Werte, die du als letztes an die Ausgänge geschrieben hast (ausser du
> hast ein elektrisches Problem). Merk dir, was du zuletzt ausgegeben hast
> und du brauchst den MCP nicht befragen.


Falls du allerdings 'Eingang' anstatt 'Ausgang' meintest.

Was spricht dagegen, das ganze mit ein bischen Makromagie verschwinden 
zu lassen? Zumal:

    uint8_t regvalue = mcp23008_reg_read(TASTER_I2C_DEVICE,MCP23008_GPIO);
    i2c_stop();
    show_value(regvalue);

was macht der i2c_stop da?
Warum macht den der reg_read nicht automatisch?

Persönlich halte ich nicht viel davon, im Funktionsnamen die 
Bauteilbezeichnung einzubringen. Denn das hilft mir als aussenstehender 
nicht wirklich weiter. Was mir aber weiter hilft, das ist wenn ich zb 
weiss, dass in diesem Baustein zb die Konfigurationsschalter sitzen. 
Warum dann nicht

#define KONFIG_KEYS   mcp23008_reg_read(TASTER_I2C_DEVICE,MCP23008_GPIO)

....


  while( 1 ) {
    _delay_ms(100);
    lcd_command(LCD_CLEAR);

    show_value( KONFIG_KEYS );
  }

über den LCD_CLEAR kann man geteilter Meinung sein, aber abgesehen davon 
ist das so unelegant dann auch wieder nicht.

Oder man definiert sich eine Funktion, die die Aufrufargumente vom MCP 
ergänzt

uint8_t ConfigKeys()
{
  return mcp23008_reg_read(TASTER_I2C_DEVICE, MCP23008_GPIO);
}

....

  while( 1 ) {
    _delay_ms(100);
    lcd_command(LCD_CLEAR);

    show_value( ConfigKeys() );
  }

Eleganz liegt im Auge des Betrachters. In der Programmierung besteht 
Eleganz des öfteren darin, den Code so gut lesbar wie möglich zu machen. 
Erreicht wird das dadurch, dass man Details an anderen Stellen 
'versteckt'. Mich interessiert nicht, wie die Config Keys abgefragt 
werden, welches Device dazu zuständig ist. Ich ruf einfach die Funktion 
auf und die Funktion weiss über die Details Bescheid.
Damit ist

    show_value( ConfigKeys() );

die 'eleganteste' Methode, um die Werte von den Dip-Switches auszulesen 
und anzuzeigen. Wenn mir die Darstellung nicht gefällt, dann ist 
'show_value()' dafür zuständig; wenn die Dip-Switches 'umziehen' dann 
ist ConfigKeys() dafür zuständig. Saubere Arbeitstrennung und bei 
Veränderungen leicht anpassbar.