STM32 I2C bleibt hängen

[Tr1bl3 (Gast)]

Hi Leute,
ich weiß es hat bereits ein paar Beiträge zu ähnlichen Problemen 
gegeben, jedoch wurden diese nicht gelöst oder die Lösung wurde nicht 
gepostet.

Ich möchte via I2C meinen Beschleunigungsensor BMA020 auslesen. 
(STM32F100-Discovery)

Ich habe auch externe Pullups an die beiden Leitungen geschalten.

Hierzu der Code:

#define I2C1_SLAVE_ADDRESS7  0x38    // I2C Adresse Sensor
#define I2C1_Own_ADDRESS7  0x00    // eigene I2C Adresse 
#define ClockSpeed    200000     // 200 KHz  

// Private variables 
I2C_InitTypeDef I2C_InitStructure;
GPIO_InitTypeDef GPIO_InitStructure;

void I2C_Initialisieren(void);
void I2C_Write(uint8_t Register_Adress, uint8_t Data);
uint8_t I2C_Read(uint8_t Register_Adress);
void I2C_Demo(void);

int main(void)       
{
   RCC_APB2PeriphClockCmd(RCC_APB2Periph_AFIO | RCC_APB2Periph_GPIOB, ENABLE);   
    /* I2C1 Periph clock enable */   
    RCC_APB1PeriphClockCmd(RCC_APB1Periph_I2C1, ENABLE); 
  RCC_AHBPeriphClockCmd(RCC_AHBPeriph_FSMC, DISABLE);
  
  I2C_Initialisieren();
  I2C_Write(0x48, 0x33); //Testfunktion
  while(1){  } 
}

void I2C_Initialisieren(void)
{
 
  //Pins konfigurieren
  GPIO_InitStructure.GPIO_Pin =  (GPIO_Pin_6) | (GPIO_Pin_7);     //SCL (PB6),SDA (PB7): Alternate Function, Open Drain
  GPIO_InitStructure.GPIO_Speed = GPIO_Speed_50MHz;
  GPIO_InitStructure.GPIO_Mode = GPIO_Mode_AF_OD; // Open Drain, I2C bus pulled high externally  
  GPIO_Init(GPIOB, &GPIO_InitStructure);
                
  I2C_Cmd(I2C1, ENABLE); 
  
  I2C_DeInit(I2C1);            
  //Register       I2C ->Seite 539
  I2C_InitStructure.I2C_ClockSpeed=ClockSpeed;
  I2C_InitStructure.I2C_Mode=I2C_Mode_I2C;
  I2C_InitStructure.I2C_DutyCycle=I2C_DutyCycle_2;
  I2C_InitStructure.I2C_OwnAddress1=I2C1_Own_ADDRESS7;
  I2C_InitStructure.I2C_Ack=I2C_Ack_Enable;
  I2C_InitStructure.I2C_AcknowledgedAddress=I2C_AcknowledgedAddress_7bit; 
  I2C_Init(I2C1, &I2C_InitStructure);
     
    
}

void I2C_Write(uint8_t Register_Adress, uint8_t Data)
{ 
  while(I2C_GetFlagStatus(I2C1, I2C_FLAG_BUSY));
  
  /* Send I2C1 START condition */
  I2C_GenerateSTART(I2C1, ENABLE);
  
  /* Test on I2C1 EV5 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_MODE_SELECT));
    
  /* Send slave Address for write */
  I2C_Send7bitAddress(I2C1, I2C1_SLAVE_ADDRESS7<<1, I2C_Direction_Transmitter);
  
  /* Test on I2C1 EV6 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_TRANSMITTER_MODE_SELECTED)); //BLEIBT HIER HÄNGEN!!!
  
  /* Send I2C1 slave register address */
  I2C_SendData(I2C1, Register_Adress);
    
  /* Test on I2C1 EV8_2 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_BYTE_TRANSMITTED));
  
  /* Send I2C1 slave Register data */
  I2C_SendData(I2C1, Data);
    
  /* Test on I2C1 EV8_8 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_BYTE_TRANSMITTED));
  
  /* Send I2C1 STOP Condition */
  I2C_GenerateSTOP(I2C1, ENABLE);  
}

Er bleibt immer bei dieser Schleife
while(!I2C_CheckEvent(I2C1, 
I2C_EVENT_MASTER_TRANSMITTER_MODE_SELECTED));
hängen und das AF (Acknowledge Failure) Bit ist gesetzt...
Ich komme einfach nach drauf woran das liegen kann...

Ich bitte um Hilfe....
mfg T.

[holger (Gast)]

>  I2C_Cmd(I2C1, ENABLE);
>
>  I2C_DeInit(I2C1);

Lustige Reihenfolge;)

[Tr1bl3 (Gast)]

Haha....ups das war noch ein altes Codefragment was hineingerutscht 
ist....ist im aktuellen Programm nicht drinnen (I2C_DeInit(I2C1))...aber 
danke für den Hinweis. Nein, das Problem besteht leider weiterhin

[holger (Gast)]

>ist im aktuellen Programm nicht drinnen

Toll du Volltrottel. Warum zeigst du das dann hier?
Und noch ein ein Tip: Schau mal in I2C_DeInit(I2C1);
rein was das macht. Dann verstehst du vieleicht auch das

   RCC_APB2PeriphClockCmd(RCC_APB2Periph_AFIO | RCC_APB2Periph_GPIOB, 
ENABLE);
    /* I2C1 Periph clock enable */
    RCC_APB1PeriphClockCmd(RCC_APB1Periph_I2C1, ENABLE);

in void I2C_Initialisieren(void) reingehört und nicht davor.

[V. F. (10euro)]

Hallo,

Schaut mal in das Datanblatt zu dem Register I2C_SR1 unter dem bit ADDR.

Dort steht: "This bit is cleared by software reading SR1 register 
followed reading SR2 [...] " -- RM0008 Seite 773 Rev 15

Die STM StdPeriph Library tut genau das in der Routine 'I2C_CheckEvent'. 
Daher wird möglicherweise der ADDR Bit in I2C_SR1 gelöscht bevor I2C_SR2 
den richtigen Wert annehmen konnte. 
I2C_EVENT_MASTER_TRANSMITTER_MODE_SELECTED wird daher nie mehr erreicht.

Die Lösung ist nicht die I2C_CheckEvent Routine zu benutzten! 
Stattdessen zuerst auf SR1 zu warten und dann auf SR2.

Der AF (Acknowledge Failure) Bit war bei mir auch gesetzt, aber mit der 
Methode wird der AF-Bit erst beim zweiten Aufruf gesetzt. Daher wird es 
wohl daran liegen, dass der Slave noch nicht bereit war und man schon 
wieder Daten sendet.

Insgesammt sollte das hier funktionieren:

do{
    //Clear AF bit
    I2C1->SR1 = ~I2C_SR1_AF;
        
    I2C_GenerateSTART(I2C1, ENABLE);
    while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_MODE_SELECT));
    I2C_Send7bitAddress(I2C1, SlaveAddress, I2C_Direction_Transmitter);

    while(!(I2C1->SR1 & I2C_SR1_TXE || I2C1->SR1 & I2C_SR1_AF)); //Warte auf einen Fehler oder das senden ist fertig. Es funktioniert auch ohne diese Zeile aber der AF bit könnte ja noch nicht gesetzt sein.

}while(I2C1->SR1 & I2C_SR1_AF); //Der Slave war wohl nicht bereit, also wiederhole...

//Achtung: hier wird mit '==' verglichen! Bitte an euren Setup anpassen!
while(!(I2C1->SR1 == (I2C_SR1_ADDR | I2C_SR1_TXE) ));
while(!(I2C1->SR2 == (I2C_SR2_MSL | I2C_SR2_BUSY | I2C_SR2_TRA) )); //hier wird auch ADDR gelöscht, aber das ist jetzt egal

TL;DR
alternativ das ADDR Bit ignorieren, da I2C_CheckEvent es löscht und 
darauf gewartet wird:

while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_TRANSMITTER_MODE_SELECTED & ~((uint32_t)I2C_SR1_ADDR) ));

Auch wenn der Thread alt ist, hoffentlich konnte ich damit Jemandem 
weiterhelfen :)