TWI/I2C am ATtiny417

[attiny417 (Gast)]

Hallo,

Ich benötige einen I2C Slave auf einem ATTiny417. Leider ändert sich 
beim einem Broadcast auf alle Adressen vom Master im SSTATUS Register 
nichts. Ich mutmaße das ich den Clock, Port oder CPUINT nicht richtig 
configuriert habe.

Was muss ich bei den Ports einstellen? Genügt das MUX oder muss ich den 
Port auf Output oder input stellen?

PORTMUX_CTRLB = 0x10;  // twi0

Bei den Clocks habe ich nichts geändert. CLK_MAIN dürfte auf 20MHz 
stehen. Der Prescaler steht auf 6. Aber wo kann ich den Takt für CLK_PER 
und CLK_CPU einstellen/aktivieren? Oder hängen beide direkt am Prescaler 
und haben nun 3,33Mhz?


Initialisierung:

      TWI0_SADDR = 0x11 << 1;
      //TWI0_SADDRMASK = 0xff;
      TWI0_SCTRLA = 0x71;
      TWI0_SCTRLB = 0x7;

Abfrage um festzustellen ob Adresse stimmt

      if(twi_test_ap() | twi_test_apif()) {
        REG->wr_bits.led_vtr1 = ~REG->wr_bits.led_vtr1;
        twi_ackact();
        state = (TWI0_SDATA & 1) ? DATX : DATX;
      }

[Rubimix (Gast)]

Hmm,
wenn Du über

TWI0_SCTRLA = 0x71;

die Interrupts aktivierst, brauchst Du eine ISR. Hast Du die?
Ansonsten noch: das obere Nibble (die 7) geht nach DB auch mit 
mindestens einem Bit ins Nirvana.

[attiny417 (Gast)]

>>Hmm,
>>wenn Du über
>>
>>TWI0_SCTRLA = 0x71;
>>
>>die Interrupts aktivierst, brauchst Du eine ISR. Hast Du die?
>>Ansonsten noch: das obere Nibble (die 7) geht nach DB auch mit
>>mindestens einem Bit ins Nirvana.

Nein habe ich nicht. Ich habe auch nicht vor den Slave über interrupt 
service routinen zu Behandeln. Ich wollte nur das "SSTATUS" Register 
abfragen. Damit ich aber die Bits im SSTATUS Register nutzen kann muss 
ich doch diese im TWI0_SCTRLA aktivieren oder?

[Rubimix (Gast)]

Nee, Die Statusflags werden auch ohne die aktivierten Interrupts 
gesetzt.
Andersrum: Wenn Du ohne ISR die Interrupts aktivierst, geht der 
Controller in der Regel auf die Klappe (führt bei jedem Interrupt zum 
Reset, weil in der Interrupttabelle kein gültiger Vektor steht).
Fazit: Setz das SCTRLA mal auf 0x01 (Nur das Enable-Bit)

[Veit D. (devil-elec)]

Hallo,

ob dein ATtiny417 mit 3,33MHz taktet hängt davon ab ob du das richtig 
eingestellt hast.
Interner MainClock sollte 20MHz sein und dein Prescaler 6 ist schon 
korrekt.
Der MainClock kann aber auch per Fuse zwischen 16 und 20MHz umgestellt 
werden.
Musste nachschauen um sicherzugehen oder am ClockOut nachmessen.
Siehe Manual ab 10.3.3.
Hast du auch den Registerzugriffsschutz beachtet vor deren 
Registeränderung für die Takteinstellungen?
CPU_CCP = CCP_IOREG_gc;
Siehe Manual 10.3.5.

Wegen Interrupt. Wenn man keine zugehörige ISR hat, darf man auch nicht 
die Interrupts aktivieren. Sonst springt das Programm an der Stelle ins 
Nirwarna. Die Statusbits kann man auch so abfragen und muss dann ggf. 
händisch löschen.

[attiny417 (Gast)]

Bei den Interrupts/Status Register sei dazu gesagt. Ich möchte keine 
Interrupt Routinen verwenden (ich möchte nur TWI0.SSTATUS auswerten), 
daher frage ich mich ob ich wirklich CPUINT configurieren muss?

[attiny417 (Gast)]

>>Hast du auch den Registerzugriffsschutz beachtet vor deren
>>Registeränderung für die Takteinstellungen?
>>CPU_CCP = CCP_IOREG_gc;
Ich habe diese Einstellung gelassen. Ich habe am Clock nichts geändert. 
Das Fusebit OSCCFG.FREQSEL steht auf 20MHz (sagt MC Studio beim 
auslesen).

Bezüglich des SSTATUS Registers habt ihr recht :D
Ich habe die Interrupt sachen rausgenommen.

meine Initialisierung:

//   PB0    I2C_CLK
//  PB1    I2C_SDA
// PB{0,1} ist aktuell als Input definiert

  PORTMUX_CTRLB = 0x10;  // twi0
  TWI0_SADDR = 0x10 | 0x01;
  TWI0_SCTRLB = 0x7;
  TWI0_SCTRLA = TWI_ENABLE_bm;

    case IDLE:      
      if(twi_test_ap() | twi_test_apif()) {
        REG->wr_bits.led_vtr1 = ~REG->wr_bits.led_vtr1; // DEBUG
        twi_ackact();
        state = (TWI0_SDATA & 1) ? DARX : DATX;
      }
    break;

.
.
.

/* Address or Stop */
ui8 twi_test_ap() {
  return (TWI0_SSTATUS & TWI_AP_bm) ? 1:0;
}

/* Address or Stop Interrupt Flag */
ui8 twi_test_apif() {
  return (TWI0_SSTATUS & TWI_APIF_bm) ? 1:0;
}

[Rubimix (Gast)]

Veit D. schrieb:
> Sonst springt das Programm an der Stelle ins
> Nirwarna.

Ich hab's gerade noch mal im Listing nachgeschaut.

Ohne die Definition eines ISR(BADISR_vect){}...
00000054 <__bad_interrupt>:
54:  d5 cf         rjmp  .-86       ; 0x0 <__vectors>

...rennt der Controller wohldefiniert auf den Resetvektor 0.


Mit ISR(BADISR_vect){}...
00000054 <__bad_interrupt>:
54:  00 c1         rjmp  .+512      ; 0x256 <__vector_default>
... rennt er ebenso wohldefiniert auf die ISR(BADISR_vect)).

Nirvana war gestern ;)

[attiny417 (Gast)]

>> Nirvana war gestern ;)
Zu einem Reset ist es bei mir tatsächlich auch nicht gekommen :D

Mein SSTATUS Register steht immer auf 0x00. Es ändert sich leider nie 
etwas. Mir kommt so so vor als ob der Clock fehlt oder ob es nicht 
aktiviert ist oder ob die I2C Signale nicht an der I2C Logik ankommen.
Beim SAM Controller musste ich explizit CLocks aktivieren und die 
Logiken einzeln zuschalten. Bin mir jetzt wirklich nicht sicher ob das 
beim Tiny417 schon auch so ist.

[Veit D. (devil-elec)]

Hallo,

wenn vermutlich nichts taktet, dann musste das prüfen. Programmiere eine 
blinkende LED und du weißt es und kannst deren Takt messen ob alles 
passt. Wenn du das ohne hartes delay machst, kannste die 
"Arbeitsanzeige" auch weiterverwenden wenn das Programm wächst.

Anonsten solltest du das komplette Programm zeigen, möglichts auf das 
Wesentliche reduziert was den Fehler noch zeigt. Kannst du für die 
Registerkonfig statt Hex auch Bitnamen bzw. Bitnummern verwenden? Das 
wäre leichter lesbar. Veroderte Hexwerte habe ich noch nicht gesehen.

[m.n. (Gast)]

Keine Ahnung, was Du da für Zahlen wohin schreibst. Ein funktionierendes 
Beispiel für einen ATtiny416 sieht so aus:

#define MY_ADDRESS    0x20
#define BIT(x)    (1 << x)
#define SCL_PIN    2   // PA2
#define SDA_PIN    1   // PA1

uint8_t tw_status, tw_cmd = 0;

void init_iic()
{
  PORTA_DIRCLR = (BIT(SCL_PIN) | BIT(SDA_PIN));   // auf Eingang
  PORTA_INTFLAGS = (BIT(SCL_PIN) | BIT(SDA_PIN)); // löschen
  PORTA_PIN1CTRL = PORT_PULLUPEN_bm;              // mit pullup
  PORTA_PIN2CTRL = PORT_PULLUPEN_bm;              // mit pullup

  PORTMUX.CTRLB |= PORTMUX_TWI0_bm;                // PA1+PA2 als alternative 
  TWI0_SCTRLA = TWI_ENABLE_bm;
  TWI0_SADDR  = MY_ADDRESS;
}

[attiny417 (Gast)]

Hallo,

Ich habe den MAIN_CLK auf PB5 gelegt und er lag in etwa bei 3,3 MHz

Hier nun das gesamte Program. Es macht nichts weiteres als zu blinken 
und es sollte bei der erkannten slave adresse LED2 einschalten. Dies 
passiert leider nie. Generell bleibt TWI0_SSTATUS immer 0x00.

#include <avr/io.h>

#define MY_ADDRESS    0x20
#define BIT(x)    (1 << x)
//#define LED1     6  // PA6
//#define LED2     7  // PA7
#define SCL_PIN    0   // PB0
#define SDA_PIN    1   // PB1

#define LED1      0x40
#define LED2      0x80


uint8_t tw_status, tw_cmd = 0;

void init_iic()
{
  PORTB_DIRCLR = (BIT(SCL_PIN) | BIT(SDA_PIN));   // auf Eingang
  PORTB_INTFLAGS = (BIT(SCL_PIN) | BIT(SDA_PIN)); // löschen
  PORTB_PIN1CTRL = PORT_PULLUPEN_bm;              // mit pullup
  PORTB_PIN2CTRL = PORT_PULLUPEN_bm;              // mit pullup
  PORTMUX.CTRLB |= PORTMUX_TWI0_bm;                
  TWI0_SCTRLA = TWI_ENABLE_bm;
  TWI0_SADDR  = MY_ADDRESS;
}

void update_iic() {
  if((TWI0_SSTATUS & TWI_APIF_bm) || (TWI0_SSTATUS & TWI_AP_bm))  {
    PORTA.OUTTGL = LED2;
  }
}

int main(void)
{
  init_iic();
  
  //PORTA_DIRSET = (BIT(LED1)|BIT(LED2));
  PORTA_DIRSET = LED1|LED2;
  
    while (1) 
    {
    
    update_iic();
    
    {
      // debug
      static uint32_t cnt = 0;
      if(cnt>25000) {
        cnt = 0;
        PORTA.OUTTGL = LED1;
      }
      cnt++;
    }
    }
}

[m.n. (Gast)]

Wenn Du PB0 und PB1 verwenden willst, darfst Du den Multiplexer 
bezüglich TWI nicht aktivieren.

[attiny417 (Gast)]

>>Wenn Du PB0 und PB1 verwenden willst, darfst Du den Multiplexer
>>bezüglich TWI nicht aktivieren.
Danke sehr. Jetzt lleuchtet die LED und der bus ist gestört weil mein 
slave nicht korrekt anwortet :D

Aber wieso darf ich PB0/1 nicht zu TWI0 multiplexen?

[attiny417 (Gast)]

Ich verstehe nicht was dieses Flag bewirkt.

Was ist denn mit Alternative gemeint? offensichtlich wird hier nicht 
PB{0,1} auf TWI0 gemuxt.

[Veit D. (devil-elec)]

Hallo,

default für I2C ist Pin PB0 und PB1.
Mit Portmux liegt I2C auf PA0 und PA1.
Das heißt wenn du im PORTMUX.CTRLB das "TWI0" Bit setzt, dann ist I2C 
auf PA0, PA1 aktiv.
Was du aber nicht machen darfst, wenn du auf Port.A deine Leds 
betreibst.
Also Portmux für I2C nicht einschalten!
Das geht auch aus der Tabelle 5.1 hervor.

[attiny417 (Gast)]

Vielen Dank für die Erklärung. Ja zu Fußnote 3 ist der Hinweis mit 
PORTMUX. Das habe ich komplett falsch verstanden. Ich dachte 
defaultmäßig sind die Pins erstmal an den PORT{A,B,C} gemuxt und wenn 
ich was anderes haben möchte nutze ich PORTMUX. Ich glaube diese Fehler 
hätte ich nie gefunden :D

[Veit D. (devil-elec)]

Hallo,

kleine Minikorrektur. Mit Portmux liegt I2C natürlich auf PA1/PA2.  ;-)
Haste bestimmt selbst schon bemerkt. Weitermachen ...