MCP23S17 lesen geht nicht (AVR SPI)

[abermel (Gast)]

moin,

bei meinem Aufbau eines Atmega 328 mit dem MCP23S17 schreibe ich auf 
GPIOA (7-Segment-Display) und lese auf GPIOB (Taster ziehen auf GND). 
Das Schreiben funktioniert ohne Probleme. Lesen geht gar nicht.

#define SPI_PORT PORTB
#define SPI_DDR  DDRB
#define SPI_CS   PB1

#define SPI_SLAVE_ID    0x40
#define SPI_SLAVE_ADDR  0x00      // A2=0,A1=0,A0=0
#define SPI_SLAVE_WRITE 0x00
#define SPI_SLAVE_READ  0x01

#define IODIRA 0x00
#define IODIRB 0x01
#define IOCONA 0x0A
#define IOCONB 0x0B
#define GPPUA  0x0C
#define GPPUB  0x0D
#define GPIOA  0x12
#define GPIOB  0x13

void init_SPI() {
  SPI_DDR = (1 << PB3) | (1 << PB5) | (1 << SPI_CS);
  SPI_PORT |= (1 << SPI_CS);

  SPCR = (1 << SPE) | (1 << MSTR);
  SPSR |= (1 << SPI2X);

  SPI_Write(IOCONA, 0x28); // I/O Control Register: BANK=0, SEQOP=1, HAEN=1 (Enable Addressing)
  SPI_Write(IODIRA, 0x00);   // GPIOA As Output

  SPI_Write(IODIRB, 0xFF);   // GPIOB As Input
  SPI_Write(GPPUB, 0xFF);    // Enable Pull-up Resistor on GPIOB
  SPI_Write(GPIOA, 0x00);    // Reset Output on GPIOA
}

void awaitTransmission() {
  while (!(SPSR & (1 << SPIF))) ;
}

void SPI_Write(unsigned char addr, unsigned char data) {
  SPI_PORT &= ~(1 << SPI_CS);
  SPDR = SPI_SLAVE_ID | ((SPI_SLAVE_ADDR << 1) & 0x0E) | SPI_SLAVE_WRITE;
  awaitTransmission();

  SPDR = addr;
  awaitTransmission();

  SPDR = data;
  awaitTransmission();

  SPI_PORT |= (1 << SPI_CS);
}

unsigned char SPI_Read(unsigned char addr) {
  SPI_PORT &= ~(1 << SPI_CS);
  SPDR = SPI_SLAVE_ID | ((SPI_SLAVE_ADDR << 1) & 0x0E) | SPI_SLAVE_READ;
  awaitTransmission();

  SPDR = addr;
  awaitTransmission();

  SPDR = 0x00; // dummy data
  awaitTransmission();

  SPI_PORT |= (1 << SPI_CS);

  return (SPDR);
}

Hat jemand eine Idee, wo der Fehler liegen könnte?

[Stefan F. (Gast)]

abermel schrieb:
> Lesen geht gar nicht.
> Hat jemand eine Idee, wo der Fehler liegen könnte?

Mit so wenig Informationen wohl kaum. Außer:

Der Fehler ist, dass du die Funktion SPI_Read() nirgends aufrufst.

Was genau bedeutet "geht gar nicht"? Untersuche die Signale mit einem 
Logic Analyzer oder Oszilloskop.

[Patrick L. (Firma: S-C-I DATA GbR) (pali64)]

abermel schrieb:
> (Taster ziehen auf GND).
> Das Schreiben funktioniert ohne Probleme. Lesen geht gar nicht.

Sind PullUp's gesetzt (Entweder Intern oder Extern)?

Und das ist ein Trifftiges Argument:
Stefan ⛄ F. schrieb:
> Der Fehler ist, dass du die Funktion SPI_Read() nirgends aufrufst.

;-)

[abermel (Gast)]

sorry:


1. dafür, falls ich irgendjemandes Zeit verschwendet habe. Der Fehler 
war eine Lötbrücke. Das MISO wurde nicht übertragen. Alles funktioniert 
jetzt.

2. für die unzureichende Problembeschreibung. SPI_Read wird schon 
aufgerufen, hatte den nur weggelassen, weil ich das nicht relevant fand.
Ich benutze die internen Pullups. Dachte das ginge aus dem Code hervor. 
Weil ich da aber auch schon dran gedacht hatte, hab ich der Schaltung 
sicherheitshalber trotzdem welche verpasst.

[Patrick L. (Firma: S-C-I DATA GbR) (pali64)]

abermel schrieb:
> dafür, falls ich irgendjemandes Zeit verschwendet habe. Der Fehler
> war eine Lötbrücke. Das MISO wurde nicht übertragen. Alles funktioniert
> jetzt.
Danke für die Mitteilung.
Fehler machen kann jeder :-)
Aber wenn man dazu steht, fängt nicht unnötiges Rätselraten an :-D

Schönes W.E. und viel erfolg beim Tüfteln.....

[Stefan F. (Gast)]

abermel schrieb:
> hatte den nur weggelassen, weil ich das nicht relevant fand.

Wie du siehst war der Fehler nicht in den Teilen, die du gezeigt hast. 
So ist es fat jedes mal. Denn das was der fragende für relevant hält, 
hat er sich bereits genauer angeschaut. Nicht wahr?

[abermel (Gast)]

dass einem auch 5 Minuten nachdem man sich endlich Hilfe sucht die 
Antwort selbst kommt, ist etwas blöd. Dennoch danke für eure Hilfe!

[Stefan F. (Gast)]

Ist gut jetzt, du brauchst nicht auf Knien zu rutschen.

[abermel (Gast)]

Stefan ⛄ F. schrieb:
> Ist gut jetzt, du brauchst nicht auf Knien zu rutschen.

so, jetzt fühl ich mich schlecht :D

[Patrick L. (Firma: S-C-I DATA GbR) (pali64)]

abermel schrieb:
> so, jetzt fühl ich mich schlecht :D

Autsch