Discussion :

[Flastick]

Bonjour,

En programmant un microprocesseur Atmega644P avec AVR Studio 4, je rencontre un bug qui me donne du fil à retordre, voilà pourquoi je viens à vous.

Je démarre une conversation I2C (i2c_start) puis j'attends qu'il génère une interrupt pour dire qu'il a bien effectué le start. Cette interruption remet la variable I2C_Etat à 0 tout simplement, et pour le programme principal, j'ai

I2C_Etat = 10;
i2c_start();

while(I2C_Etat != 0); //tourne en boucle en attendant qu'il ait terminé

Mais voilà, le programme semble se perdre, il ne tourne même pas dans la boucle, ni dans la routine d'interruption ! Donc je ne comprends pas..

Voici le code complet :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
 
#include <stdlib.h>
#include <string.h>
#include <avr/io.h>
#include <avr/pgmspace.h>
#include <avr/interrupt.h>
#include <avr/eeprom.h>
#include <avr/boot.h>
#include <avr/wdt.h>
 
 
#define SCL_CLOCK  200000L
#define SYSCLK  20000000L
 
//Lampe rouge
#define ROT_ON   	PORTB &=~0x01
#define ROT_OFF    	PORTB |= 0x01
 
//Lampe verte
#define GRN_ON  	PORTB &=~0x02
#define GRN_OFF    	PORTB |= 0x02
 
unsigned char I2C_Etat;
 
 
 
void I2C_Init(void)
{
  TWSR = 0;
  TWBR = ((SYSCLK/SCL_CLOCK)-16)/2; 
 
	I2C_Etat = 0;
 
}
 
void i2c_start(void) 
 
{
    TWCR = 0xA5 ;//10100101
}
 
 
void i2c_stop(void)
 
{
    TWCR = 0x95; //10010101
}
 
 
ISR (TWI_vect)
{
	I2C_Etat = 0;//Remet à 0
	TWCR |= 0x80;
}
 
int main()
{
	// 1 = out, 0 = IN
	DDRC  = 0xC3;
	DDRB  = 0x1B;
	DDRD  = 0xFE;
 
    PORTC = 0xFF; // Pullup SDA
    PORTB = 0x01; // LED_Rot
    PORTD = 0x77; // LED
 
 
	//disable watchdog
	MCUSR &=~(1<<WDRF);
	WDTCSR = 0;
 
	I2C_Init();
	sei();
 
		ROT_OFF;//Eteint les lampes
		GRN_OFF;
		while (I2C_Etat != 0);
 
		I2C_Etat = 10;
		i2c_start();
		ROT_ON;//Allume la rouge
 
		while (I2C_Etat != 0);
 
		while(1)
		{
			GRN_ON;//Allume la verte
			//En fait, la verte ne s'allumera jamais
			//Pourquoi ??
		}
 
	return 0;
}

Donc la lampe rouge sera allumée et la verte éteinte, et si on met GRN_ON; à l'intérieur de la boucle, pareil.. Une idée ??

En vous remerciant beaucoup (très beaucoup!) pour votre attention,

Flavien G.

[orfix]

Il me semble que tu as inversé les _ON _OFF:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
//Lampe rouge
#define ROT_OFF   	PORTB &=~0x01
#define ROT_ON    	PORTB |= 0x01
 
//Lampe verte
#define GRN_OFF  	PORTB &=~0x02
#define GRN_ON    	PORTB |= 0x02

[Obsidian]

Il me semble que tu as inversé les _ON _OFF:
[code]//Lampe rouge

Pas nécessairement. En polarisation inverse, la LED est montée à l'envers. Mettre un bit de sortie à 1 bloque alors le passage du courant. C'est assez fréquent.

[diogene]

Je ne connais pas le compilateur utilisé, mais ne faudrait-il pas utiliser le qualificatif volatile sur I2C_Etat pour éviter des optimisations intempestives sur les boucles while (I2C_Etat != 0); ?

[Flastick]

ça marche !!!! C'était bien le mot-clef volatile qui manquait !

Merci infiniment !!!