MCP23017 IOCON-Register Problem

[Student T. (hberg)]

Hi,

ich versuche mich gerade an einem MCP23017 (i2c) und habe etwas 
Schwierigkeiten beim setzen des IOCON-Registers.

Folgendes funktioniert hervorragend:

// reset ic
PORTB |= (1 << PORTB1);
_delay_ms(100);
PORTB |= (1 << PORTB1);

// GPIOA as OUTPUT
i2c_start(MCP23017_ADDR + I2C_WRITE);
i2c_write(0x00);
i2c_write(0x00);
i2c_stop();
  
// GPIOB as OUTPUT
i2c_start(MCP23017_ADDR + I2C_WRITE);
i2c_write(0x01);
i2c_write(0x00);
i2c_stop();

while (1) {
  i2c_start(MCP23017_ADDR + I2C_WRITE);
  i2c_write(0x00);
  i2c_write(0xFF);
  i2c_write(0xFF);
  i2c_stop();

  _delay_ms(100);

  i2c_start(MCP23017_ADDR + I2C_WRITE);
  i2c_write(0x00);
  i2c_write(0x00);
  i2c_write(0x00);
  i2c_stop();

  _delay_ms(100);
}

Um sicherzugehen, dass nach jedem Flash keine Werte in dem IC überleben, 
resette ich ihn vorher. Danach setze ich x00 == GPIOA und x01 == GPIOB 
auf OUTPUT.

http://ww1.microchip.com/downloads/en/devicedoc/21952b.pdf

Ich möchte nun zusätzlich zum oben gezeigten Beispiel das IOCON Register 
verändern. Ersteinmal möchte ich es auf 0x00 setzen, damit es identisch 
ist zum Defaultwert.

Im Datenbatenblatt ist das Register standardmäßig gesetzt auf IOCON = 
0x00. Dadurch ist IOCON.BANK = 0x00 und das wiederum bedeutet laut 
Datenblatt:
"0 = The registers are in the same bank (addresses are sequential)"

Durch IOCON.BANK kann man die Adressen der Register anders anordnen. 
IOCON.BANK = 1 bedeutet, die Adressen liegen in zwei Blöcken. IOCON.BANK 
= 0 bedeutet, die Adressen liegen nacheinander (0x00, 0x01 = GPIOA, 
GPIOB, ..). Laut dem Datenblatt sollte das IOCON Register auf 0x0A oder 
0x0B liegen, da IOCON.BANK = 0. In der Überschrift dagegen steht:
"IOCON – I/O EXPANDER CONFIGURATION REGISTER (ADDR 0x05)"

Welche ist nun die richtige Adresse? Meiner Meinung nach 0x0A und 0x0B, 
da eben IOCON.BANK = 0 durch IOCON = 0x00. Im Anhang der Abschnitt. Aber 
warum steht dann in der Überschrift 0x05, was auf IOCON.BANK = 1 
hindeuten würde? Das verstehe mal einer!

Um nun testweise das IOCON-Register mit 0x00 zu setzen (was es 
eigentlich lt. Datenblatt schon haben sollte), schreibe ich:

i2c_init();

i2c_start(MCP23017_ADDR + I2C_WRITE);
i2c_write(0x05); // oder auch 0x0A, geht beides nicht
i2c_write(0x00);

// ...

Danach ist nur noch Port A (0x00) auf OUTPUT gesetzt, Port B bleibt auf 
INPUT. Auch das schreiben auf Port B klappt nicht mehr.

Da ich aber den gleichen Wert für das Register setze, welcher im 
Register bereits gesetzt ist, sollte sich doch eigentlich nichts ändern? 
Ich überschreibe 0x00 mit 0x00. Warum verhält er sich da nun anders, wie 
davor?

Für jeder Hilfe bin ich dankbar.

Gruß,
Kevin

[Student T. (hberg)]

Fehler gefunden.. ich habe i2c_stop() nach dem i2c_write() beim 
IOCON-Register vergessen...

Falls jemand ähnliche Probleme hat, mit dem hier sollte es auf Anhieb 
klappen:

#define F_CPU 16000000
 
#include <avr/io.h>
#include <util/delay.h>
#include "i2cmaster.h"
 
#define MCP23x17_ADDR 0x40
 
// control registers
// ONLY VALID WHEN ICON.BANK = 0
#define MCP23x17_IODIRA   0x00
#define MCP23x17_IODIRB   0x01
#define MCP23x17_IOPOLA   0x02
#define MCP23x17_IOPOLB   0x03
#define MCP23x17_GPINTENA 0x04
#define MCP23x17_GPINTENB 0x05
#define MCP23x17_DEFVALA  0x06
#define MCP23x17_DEFVALB  0x07
#define MCP23x17_INTCONA  0x08
#define MCP23x17_INTCONB  0x09
#define MCP23x17_IOCON    0x0A
//#define MCP23x17_IOCON    0x0B
#define MCP23x17_GPPUA    0x0C
#define MCP23x17_GPPUB    0x0D
#define MCP23x17_INTFA    0x0E
#define MCP23x17_INTFB    0x0F
#define MCP23x17_INTCAPA  0x10
#define MCP23x17_INTCAPB  0x11
#define MCP23x17_GPIOA    0x12
#define MCP23x17_GPIOB    0x13
#define MCP23x17_OPLATA   0x14
#define MCP23x17_OPLATB   0x15
 
// IOCON register bit positions
#define MCP23x17_IOCON_BANK   7
#define MCP23x17_IOCON_MIRROR 6
#define MCP23x17_IOCON_SEQOP  5
#define MCP23x17_IOCON_DISSLW 4
#define MCP23x17_IOCON_HAEN   3
#define MCP23x17_IOCON_ODR    2
#define MCP23x17_IOCON_INTPOL 1
 
int main(void)
{
 
  // init i2c
  i2c_init();
 
  // IOCON
  // configuration register
  i2c_start(MCP23x17_ADDR + I2C_WRITE);
  i2c_write(MCP23x17_IOCON);
  i2c_write(0x00
    | (0 << MCP23x17_IOCON_BANK)
    | (0 << MCP23x17_IOCON_MIRROR)
    | (0 << MCP23x17_IOCON_SEQOP)
    | (0 << MCP23x17_IOCON_DISSLW)
    | (1 << MCP23x17_IOCON_HAEN)
    | (0 << MCP23x17_IOCON_ODR)
    | (0 << MCP23x17_IOCON_INTPOL)
  );
  i2c_stop();
 
  // IODIRA
  // set PORTA to OUTPUT
  i2c_start(MCP23x17_ADDR);
  i2c_write(MCP23x17_IODIRA);
  i2c_write(0x00);
  i2c_stop();
 
  // IODIRB
  // set PORTB to OUTPUT
  i2c_start(MCP23x17_ADDR);
  i2c_write(MCP23x17_IODIRB);
  i2c_write(0x00);
  i2c_stop();
 
  while (1) {
 
    // set GPIOA (first write) and GPIOB (second write)
    // to HIGH
    i2c_start(MCP23x17_ADDR);
    i2c_write(MCP23x17_GPIOA);
    i2c_write(0xFF);
    i2c_write(0xFF);
    i2c_stop();
 
    _delay_ms(100);
 
    // set GPIOA (first write) and GPIOB (second write)
    // to HIGH
    i2c_start(MCP23x17_ADDR);
    i2c_write(MCP23x17_GPIOA);
    i2c_write(0x00);
    i2c_write(0x00);
    i2c_stop();
 
    _delay_ms(100);
  }
}