Problème de programmation PIC via mikroC

Version imprimable

Voir 40 message(s) de cette discussion en une page

20/04/2020, 09h28
Vincent PETIT

2 essais :

:arrow: Dé-commente juste cette ligne unsigned char donnee = 0xAA; (le reste laisse en commentaire) je veux voir si il y a encore

16 315 Invalid expression Chenillard3.c
16 424 '}' expected ';' found Chenillard3.c

:arrow: Si ces erreurs apparaissent, je vais te demander d'effacer et de ré-écrire unsigned char donnee = 0xAA; (j'imagine que tu as fait un copier collé et je voudrai juste être sur qu'un caractère blanc ne se soit pas inséré sans qu'on le voit)
20/04/2020, 11h15
sparow1

2 pièce(s) jointe(s)

Pièce jointe 558225

Pièce jointe 558228

Toujours pas de changement

Ok, le dernier test est intéressant car il met en évidence le problème. unsigned char donnee = 0xAA; est considéré comme une expression "invalide" alors que dans n'importe quel compilateur c'est tout à fait valide, si tu essais dans CodeBloks tu verras qu'il n'y a pas de problème. Le compilateur te répond qu'il attendais une accolade } à la plus d'un point virgule ; (c'est presque incroyable :mrgreen:)

Bon, il va falloir essayer de voir si le compilateur n'est pas entrain de faire un caca nerveux pare que unsigned char donnee = 0xAA; n'est pas déclaré à un endroit qu'il n'aime pas.

Essai1 (j'ai viré le prototype et placé la fonction LED1 en haut. J'ai viré aussi #include <stdio.h> je ne suis pas sur que tu en ais besoin sur MikroC, je te laisse voir si il faut le remettre ou pas.

Code:

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
//#include <stdio.h>
 
void LED1(unsigned char d, unsigned int nbr_fois)
{ 
 
    unsigned int ma_variable_16bits = 0;
 
    ma_variable_16bits = d;
 
    for(int j=0;j<nbr_fois;j++){
        for(int i=0;i<8;i++){
			if(ma_variable_16bits & 0x0100 == 1){
				RB1_bit = 1 ; // le bit n°8 est à 1, j'allume la LED
			}
			else{
				RB1_bit = 0 ; // le bit n°8 est à 0, j'éteins la LED
			}
			if(ma_variable_16bits & 0x0200 == 1){
				RB2_bit = 1 ; // le bit n°9 est à 1, j'allume la LED
			}
			else{
				RB2_bit = 0 ; // le bit n°9 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x0400 == 1){
				RB3_bit = 1 ; // le bit n°10 est à 1, j'allume la LED
			}
			else{
				RB3_bit = 0 ; // le bit n°10 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x0800 == 1){
				RB4_bit = 1 ; // le bit n°11 est à 1, j'allume la LED
			}
			else{
				RB4_bit = 0 ; // le bit n°11 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x1000 == 1){
				RB5_bit = 1 ; // le bit n°12 est à 1, j'allume la LED
			}
			else{
				RB5_bit = 0 ; // le bit n°12 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x2000 == 1){
				RB6_bit = 1 ; // le bit n°13 est à 1, j'allume la LED
			}
			else{
				RB6_bit = 0 ; // le bit n°13 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x4000 == 1){
				RB7_bit = 1 ; // le bit n°14 est à 1, j'allume la LED
			}
			else{
				RB7_bit = 0 ; // le bit n°14 est à 0, j'éteints la LED
			}
 
            ma_variable_16bits = ma_variable_16bits << 1;
        }
        ma_variable_16bits = ma_variable_16bits | d; // petite modif je fais un OU au lieu d'un +
    }
 
} // Fin du sous-programme LED1 */
 
void main() 
{
 unsigned char donnee = 0b10101010;
 
 TRISA = 0b00000010; //port A en sortie sauf RA1
 PORTA = 0b11001000 ; //mettre les sorties à zéro sauf RA6, RA7,RA3
 TRISB = 0b00000001; //port B en sortie sauf RB_0
 PORTB = 0 ; //mettre les sorties à zéro
 pcon.OSCF = 1; //configure le bit 3 du registre pcon pour 4 Mhz
 CMCON = 0b00000111; //désactiver les comparateurs sur RA0
 
 
 
 
while(1)  //boucle infinie
 {
  if (RB0_bit==0) { //si RB0 = 0 exécuter la ligne suivante
       LED1(donnee, 10) ; //Sous-programme LED1, on charge la donnée et on la fait tourner 10 fois.
 } //fin du SI
}
}

Essai2, j'ai mis les variables en globale pour voir si ce n'est pas un soucis de portée (mais j'en doute)

Code:

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
//#include <stdio.h>
 
unsigned int ma_variable_16bits; //variable globale
int i; //variable globale
int j; //variable globale
unsigned char donnee;//variable globale
 
void LED1(unsigned char d, unsigned int nbr_fois)
{ 
 
    ma_variable_16bits = d;
 
    for(j=0;j<nbr_fois;j++){
        for(i=0;i<8;i++){
			if(ma_variable_16bits & 0x0100 == 1){
				RB1_bit = 1 ; // le bit n°8 est à 1, j'allume la LED
			}
			else{
				RB1_bit = 0 ; // le bit n°8 est à 0, j'éteins la LED
			}
			if(ma_variable_16bits & 0x0200 == 1){
				RB2_bit = 1 ; // le bit n°9 est à 1, j'allume la LED
			}
			else{
				RB2_bit = 0 ; // le bit n°9 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x0400 == 1){
				RB3_bit = 1 ; // le bit n°10 est à 1, j'allume la LED
			}
			else{
				RB3_bit = 0 ; // le bit n°10 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x0800 == 1){
				RB4_bit = 1 ; // le bit n°11 est à 1, j'allume la LED
			}
			else{
				RB4_bit = 0 ; // le bit n°11 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x1000 == 1){
				RB5_bit = 1 ; // le bit n°12 est à 1, j'allume la LED
			}
			else{
				RB5_bit = 0 ; // le bit n°12 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x2000 == 1){
				RB6_bit = 1 ; // le bit n°13 est à 1, j'allume la LED
			}
			else{
				RB6_bit = 0 ; // le bit n°13 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x4000 == 1){
				RB7_bit = 1 ; // le bit n°14 est à 1, j'allume la LED
			}
			else{
				RB7_bit = 0 ; // le bit n°14 est à 0, j'éteints la LED
			}
 
            ma_variable_16bits = ma_variable_16bits << 1;
        }
        ma_variable_16bits = ma_variable_16bits | d; // petite modif je fais un OU au lieu d'un +
    }
 
} // Fin du sous-programme LED1 */
 
void main() 
{
 donnee = 0b10101010;
 
 TRISA = 0b00000010; //port A en sortie sauf RA1
 PORTA = 0b11001000 ; //mettre les sorties à zéro sauf RA6, RA7,RA3
 TRISB = 0b00000001; //port B en sortie sauf RB_0
 PORTB = 0 ; //mettre les sorties à zéro
 pcon.OSCF = 1; //configure le bit 3 du registre pcon pour 4 Mhz
 CMCON = 0b00000111; //désactiver les comparateurs sur RA0
 
 
 
 
while(1)  //boucle infinie
 {
  if (RB0_bit==0) { //si RB0 = 0 exécuter la ligne suivante
       LED1(donnee, 10) ; //Sous-programme LED1, on charge la donnée et on la fait tourner 10 fois.
 } //fin du SI
}
}

20/04/2020, 13h32
sparow1

2 pièce(s) jointe(s)

Clairement mikorC, c'est horrible il trouve des erreurs là où il n'y en a pas..

Donc j'ai testé pour le 1er, avec ou sans le #include<stdio.h>, ça ne change rien :

Pièce jointe 558284

Le 2ème à l'air de fonctionner nickel :D

Pièce jointe 558285

Rapport complet :
0 1139 Available RAM: 208 [bytes], Available ROM: 2048 [bytes]
0 122 Compilation Started Chenillard3.c
85 123 Compiled Successfully Chenillard3.c
0 127 All files Compiled in 78 ms
0 1144 Used RAM (bytes): 12 (6%) Free RAM (bytes): 196 (94%) Used RAM (bytes): 12 (6%) Free RAM (bytes): 196 (94%)
0 1144 Used ROM (program words): 173 (8%) Free ROM (program words): 1875 (92%) Used ROM (program words): 173 (8%) Free ROM (program words): 1875 (92%)
0 125 Project Linked Successfully Chenillard3.mcppi
0 128 Linked in 47 ms
0 129 Project 'Chenillard3.mcppi' completed: 141 ms
0 103 Finished successfully: 20 avr. 2020, 13:30:26 Chenillard3.mcppi
20/04/2020, 13h56
Vincent PETIT
Citation:

Envoyé par sparow1

Le 2ème à l'air de fonctionner nickel :D

Tant mieux mais...

Citation:

Envoyé par sparow1

Clairement mikorC, c'est horrible

En effet !
Ce n'est absolument pas normal car la portée des variables ne pose pas de soucis ici. Le fait d'avoir tout mis en global est une limitation très étrange.

Puisque c'est du global, peut être que mettre volatile devant chaque déclaration de variable globale serait un petit plus - en tout cas c'est vitale si tu utilises des interruptions.
Code:

1 2 3 4 5 6 volatile unsigned int ma_variable_16bits; //variable globale volatile int i; //variable globale volatile int j; //variable globale volatile unsigned char donnee;//variable globale ... ...
Tu as de quoi essayer avec Proteus ?

2 pièce(s) jointe(s)

J'ai rajouté volatile au variable

Code:

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
 
//#include <stdio.h>
 
volatile unsigned int ma_variable_16bits; //variable globale
volatile int i; //variable globale
volatile int j; //variable globale
volatile unsigned char donnee;//variable globale
 
void LED1(volatile unsigned char d, volatile unsigned int nbr_fois)
{
 
    ma_variable_16bits = d;
 
    for(j=0;j<nbr_fois;j++){
        for(i=0;i<8;i++){
			if(ma_variable_16bits & 0x0100 == 1){
				RB1_bit = 1 ; // le bit n°8 est à 1, j'allume la LED
			}
			else{
				RB1_bit = 0 ; // le bit n°8 est à 0, j'éteins la LED
			}
			if(ma_variable_16bits & 0x0200 == 1){
				RB2_bit = 1 ; // le bit n°9 est à 1, j'allume la LED
			}
			else{
				RB2_bit = 0 ; // le bit n°9 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x0400 == 1){
				RB3_bit = 1 ; // le bit n°10 est à 1, j'allume la LED
			}
			else{
				RB3_bit = 0 ; // le bit n°10 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x0800 == 1){
				RB4_bit = 1 ; // le bit n°11 est à 1, j'allume la LED
			}
			else{
				RB4_bit = 0 ; // le bit n°11 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x1000 == 1){
				RB5_bit = 1 ; // le bit n°12 est à 1, j'allume la LED
			}
			else{
				RB5_bit = 0 ; // le bit n°12 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x2000 == 1){
				RB6_bit = 1 ; // le bit n°13 est à 1, j'allume la LED
			}
			else{
				RB6_bit = 0 ; // le bit n°13 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x4000 == 1){
				RB7_bit = 1 ; // le bit n°14 est à 1, j'allume la LED
			}
			else{
				RB7_bit = 0 ; // le bit n°14 est à 0, j'éteints la LED
			}
 
            ma_variable_16bits = ma_variable_16bits << 1;
        }
        ma_variable_16bits = ma_variable_16bits | d; // petite modif je fais un OU au lieu d'un +
    }
 
} // Fin du sous-programme LED1 */
 
void main()
{
 donnee = 0b10101010;
 
 TRISA = 0b00000010; //port A en sortie sauf RA1
 PORTA = 0b11001000 ; //mettre les sorties à zéro sauf RA6, RA7,RA3
 TRISB = 0b00000001; //port B en sortie sauf RB_0
 PORTB = 0 ; //mettre les sorties à zéro
 pcon.OSCF = 1; //configure le bit 3 du registre pcon pour 4 Mhz
 CMCON = 0b00000111; //désactiver les comparateurs sur RA0
 
 
 
 
while(1)  //boucle infinie
 {
  if (RB0_bit==0) { //si RB0 = 0 exécuter la ligne suivante
       LED1(donnee, 10) ; //Sous-programme LED1, on charge la donnée et on la fait tourner 10 fois.
 } //fin du SI
}
}

La simulation bouton False
Pièce jointe 558320

La simulation bouton True
Pièce jointe 558324

Ah oui ! Maintenant qu'on a tout mis en global, les arguments dans la fonction LED ne servent plus à rien.

Tu peux essayer de compiler ça et si ça compile, d'essayer ? /!\ RAJOUTE LES TEMPORISATIONS SINON CA VA ALLER TROP VITE POUR VOIR QUELQUE CHOSE

Code:

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
volatile unsigned int ma_variable_16bits; //variable globale
volatile int i; //variable globale
volatile int j; //variable globale
volatile unsigned char donnee;//variable globale
volatile unsigned int nbr_fois;
 
void LED1()
{
 
    ma_variable_16bits = donnee;
 
    for(j=0;j<nbr_fois;j++){
        for(i=0;i<8;i++){
			if(ma_variable_16bits & 0x0100 == 1){
				RB1_bit = 1 ; // le bit n°8 est à 1, j'allume la LED
			}
			else{
				RB1_bit = 0 ; // le bit n°8 est à 0, j'éteins la LED
			}
			if(ma_variable_16bits & 0x0200 == 1){
				RB2_bit = 1 ; // le bit n°9 est à 1, j'allume la LED
			}
			else{
				RB2_bit = 0 ; // le bit n°9 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x0400 == 1){
				RB3_bit = 1 ; // le bit n°10 est à 1, j'allume la LED
			}
			else{
				RB3_bit = 0 ; // le bit n°10 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x0800 == 1){
				RB4_bit = 1 ; // le bit n°11 est à 1, j'allume la LED
			}
			else{
				RB4_bit = 0 ; // le bit n°11 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x1000 == 1){
				RB5_bit = 1 ; // le bit n°12 est à 1, j'allume la LED
			}
			else{
				RB5_bit = 0 ; // le bit n°12 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x2000 == 1){
				RB6_bit = 1 ; // le bit n°13 est à 1, j'allume la LED
			}
			else{
				RB6_bit = 0 ; // le bit n°13 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x4000 == 1){
				RB7_bit = 1 ; // le bit n°14 est à 1, j'allume la LED
			}
			else{
				RB7_bit = 0 ; // le bit n°14 est à 0, j'éteints la LED
			}
 
            ma_variable_16bits = ma_variable_16bits << 1;
        }
        ma_variable_16bits = ma_variable_16bits | donnee; // petite modif je fais un OU au lieu d'un +
    }
 
} // Fin du sous-programme LED1 */
 
void main()
{
 donnee = 0b10101010;
 nbr_fois = 10;
 
 TRISA = 0b00000010; //port A en sortie sauf RA1
 PORTA = 0b11001000 ; //mettre les sorties à zéro sauf RA6, RA7,RA3
 TRISB = 0b00000001; //port B en sortie sauf RB_0
 PORTB = 0 ; //mettre les sorties à zéro
 pcon.OSCF = 1; //configure le bit 3 du registre pcon pour 4 Mhz
 CMCON = 0b00000111; //désactiver les comparateurs sur RA0
 
 
 
 
while(1)  //boucle infinie
 {
  if (RB0_bit==0) { //si RB0 = 0 exécuter la ligne suivante
       LED1() ; //Sous-programme LED1, on charge la donnée et on la fait tourner 10 fois.
 } //fin du SI
}
}

20/04/2020, 14h28
sparow1

ça compile et ça fonctionne, les leds s'allument toute une fois dans l'ordre.

Où est-ce que je dois mettre les delay ?

Il faudrait que ça se répète à l'infinie et si possible une boucle sans répétition (où on ne mets pas plusieurs fois la même instruction, simplifier la boucle)
20/04/2020, 15h04
Vincent PETIT

Le delay (je ne connais pas l'instruction dans mikroC) se place juste après ma_variable_16bits = ma_variable_16bits << 1;
Un seul delay suffit.

Citation:

les leds s'allument toute une fois dans l'ordre

Peut être que c'est à cause de la tempo qui n'est pas là, ça doit aller super vite.

Dit moi si ça fonctionne avant de voir la boucle infinie. Tu devrais voir la donnée 10101010 avancée jusqu'à ce que ça clignote à tour de rôle (en fait le clignotement est une impression, c'est la donnée qui continue d'avancer mais comme elle vaut 10101010...)
20/04/2020, 16h23
sparow1

Je ne vois aucun changement peut importe le délai, ça avance de led en led une seule fois puis c'est tout. Je dois rappuyer sur le bouton.
20/04/2020, 16h55
Vincent PETIT
C'est quoi encore ce délire :koi:
On a deux boucles for imbriquées et ça ne marche pas ?
Code:

1 2 3 4 5 ... for(j=0;j<nbr_fois;j++){ for(i=0;i<8;i++){ ...
Tu as une version limitée de mikroC ? Là je commence a être à court d'idée parce que sur un PC ça marche sans problème. Tu te rappelles le programme PC en #24 ? On a fait dérouler la première boucle 3 fois avec la variable j et dedans il y a i 8 fois et il n'y a pas eu de problème.

Citation:

00000000 10101010 --------> j = 0, i = 0
00000001 01010100 ----> i = 1
00000010 10101000 ----> i = 2
00000101 01010000 ----> i = 3
00001010 10100000 ----> i = 4
00010101 01000000 ----> i = 5
00101010 10000000 ----> i = 6
01010101 00000000 ----> i = 7
10101010 10101010 --------> j = 1, i = 0
01010101 01010100 ----> i = 1
10101010 10101000 ----> i = 2
01010101 01010000 ----> i = 3
10101010 10100000 ----> i = 4
01010101 01000000 ----> i = 5
10101010 10000000 ----> i = 6
01010101 00000000 ----> i = 7
10101010 10101010 --------> j = 2, i = 0
01010101 01010100 ----> i = 1
10101010 10101000 ----> i = 2
01010101 01010000 ----> i = 3
10101010 10100000 ----> i = 4
01010101 01000000 ----> i = 5
10101010 10000000 ----> i = 6
01010101 00000000 ----> i = 7

Normalement dans Proteus tu peux faire dérouler le programme en mode pas à pas et mettre des points d'arrêts. Avec ça tu peux voir la valeur des variables en direct et il faudrait voir ce qui se passe avec la première boucle. Notamment avec les valeurs dans j et nbr_fois
20/04/2020, 17h29
sparow1

Je ne comprends pas non plus, je suis dépité, le principe des 2 boucles est simple mais pourtant il n'y a qu'un passage qui est effectué. Une boucle infinie ne fonctionnerait pas non plus ?

Oui oui les tests sur codeblocks étaient nickel, mais mikroC fait des siennes.

J'ai testé plusieurs manières on dirait que le bouton ne donne l'impulsion que pour un unique passage.

Je viens de lire quelques limitations de ton compilateur et notamment l'impossibilité de déclarer une variable autre part qu'en début de fonction (pas possible de déclarer un int dans une boucle for par exemple).

Tu peux essayer de compiler ceci ?

Code:

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
void LED1(unsigned char d, unsigned int nbr_fois)
{ 
    unsigned int ma_variable_16bits;
    unsigned int i;
    unsigned int j;
 
    ma_variable_16bits = d;
 
    for(j = 0; j < nbr_fois; j++)
    {
        for(i = 0; i < 8; i++)
        {
			if(ma_variable_16bits & 0x0100 == 1)
				RB1_bit = 1 ; // le bit n°8 est à 1, j'allume la LED
			else
				RB1_bit = 0 ; // le bit n°8 est à 0, j'éteins la LED
			if(ma_variable_16bits & 0x0200 == 1)
				RB2_bit = 1 ; // le bit n°9 est à 1, j'allume la LED
			else
				RB2_bit = 0 ; // le bit n°9 est à 0, j'éteints la LED
			if(ma_variable_16bits & 0x0400 == 1)
				RB3_bit = 1 ; // le bit n°10 est à 1, j'allume la LED
			else
				RB3_bit = 0 ; // le bit n°10 est à 0, j'éteints la LED
			if(ma_variable_16bits & 0x0800 == 1)
				RB4_bit = 1 ; // le bit n°11 est à 1, j'allume la LED
			else
				RB4_bit = 0 ; // le bit n°11 est à 0, j'éteints la LED
			if(ma_variable_16bits & 0x1000 == 1)
				RB5_bit = 1 ; // le bit n°12 est à 1, j'allume la LED
			else
				RB5_bit = 0 ; // le bit n°12 est à 0, j'éteints la LED
			if(ma_variable_16bits & 0x2000 == 1)
				RB6_bit = 1 ; // le bit n°13 est à 1, j'allume la LED
			else
				RB6_bit = 0 ; // le bit n°13 est à 0, j'éteints la LED
			if(ma_variable_16bits & 0x4000 == 1)
				RB7_bit = 1 ; // le bit n°14 est à 1, j'allume la LED
			else
				RB7_bit = 0 ; // le bit n°14 est à 0, j'éteints la LED
 
 
            Delay_ms(500);
            ma_variable_16bits = ma_variable_16bits << 1;
        }
 
        ma_variable_16bits = ma_variable_16bits | d; // petite modif je fais un OU au lieu d'un +
    }
} 
 
 
void main() 
{
	unsigned char donnee = 0b10101010;
 
 	TRISA = 0b00000010; //port A en sortie sauf RA1
 	PORTA = 0b11001000 ; //mettre les sorties à zéro sauf RA6, RA7,RA3
 	TRISB = 0b00000001; //port B en sortie sauf RB_0
 	PORTB = 0 ; //mettre les sorties à zéro
 	pcon.OSCF = 1; //configure le bit 3 du registre pcon pour 4 Mhz
 	CMCON = 0b00000111; //désactiver les comparateurs sur RA0
 
	while(1)  //boucle infinie
 	{
  		if (RB0_bit==0) { 
       		LED1(donnee, 10) ; 
 		}
	}
}

Citation:

Essai2, j'ai mis les variables en globale pour voir si ce n'est pas un soucis de portée (mais j'en doute)

Code:

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
//#include <stdio.h>
 
unsigned int ma_variable_16bits; //variable globale
int i; //variable globale
int j; //variable globale
unsigned char donnee;//variable globale
 
void LED1(unsigned char d, unsigned int nbr_fois)
{ 
 
    ma_variable_16bits = d;
 
    for(j=0;j<nbr_fois;j++){
        for(i=0;i<8;i++){
			if(ma_variable_16bits & 0x0100 == 1){
				RB1_bit = 1 ; // le bit n°8 est à 1, j'allume la LED
			}
			else{
				RB1_bit = 0 ; // le bit n°8 est à 0, j'éteins la LED
			}
			if(ma_variable_16bits & 0x0200 == 1){
				RB2_bit = 1 ; // le bit n°9 est à 1, j'allume la LED
			}
			else{
				RB2_bit = 0 ; // le bit n°9 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x0400 == 1){
				RB3_bit = 1 ; // le bit n°10 est à 1, j'allume la LED
			}
			else{
				RB3_bit = 0 ; // le bit n°10 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x0800 == 1){
				RB4_bit = 1 ; // le bit n°11 est à 1, j'allume la LED
			}
			else{
				RB4_bit = 0 ; // le bit n°11 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x1000 == 1){
				RB5_bit = 1 ; // le bit n°12 est à 1, j'allume la LED
			}
			else{
				RB5_bit = 0 ; // le bit n°12 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x2000 == 1){
				RB6_bit = 1 ; // le bit n°13 est à 1, j'allume la LED
			}
			else{
				RB6_bit = 0 ; // le bit n°13 est à 0, j'éteints la LED
			}
			if(ma_variable_16bits & 0x4000 == 1){
				RB7_bit = 1 ; // le bit n°14 est à 1, j'allume la LED
			}
			else{
				RB7_bit = 0 ; // le bit n°14 est à 0, j'éteints la LED
			}
 
            ma_variable_16bits = ma_variable_16bits << 1;
        }
        ma_variable_16bits = ma_variable_16bits | d; // petite modif je fais un OU au lieu d'un +
    }
 
} // Fin du sous-programme LED1 */
 
void main() 
{
 donnee = 0b10101010;
 
 TRISA = 0b00000010; //port A en sortie sauf RA1
 PORTA = 0b11001000 ; //mettre les sorties à zéro sauf RA6, RA7,RA3
 TRISB = 0b00000001; //port B en sortie sauf RB_0
 PORTB = 0 ; //mettre les sorties à zéro
 pcon.OSCF = 1; //configure le bit 3 du registre pcon pour 4 Mhz
 CMCON = 0b00000111; //désactiver les comparateurs sur RA0
 
 
 
 
while(1)  //boucle infinie
 {
  if (RB0_bit==0) { //si RB0 = 0 exécuter la ligne suivante
       LED1(donnee, 10) ; //Sous-programme LED1, on charge la donnée et on la fait tourner 10 fois.
 } //fin du SI
}
}

Je viens de me rendre compte que je testais mon ancien fichier sur proteus depuis cette étape ci... ça aurait pu être une bonne nouvelle en testant ce qu'on a fait depuis ici, que ça fonctionne.
Mais malheureusement il n'allume aucune LED.
Le seul point positif c'est que tout compile à partir d'ici

21/04/2020, 11h55
kaitlyn

Salut ou resalut,

Mais oh que c'est dommage quand je vois tout ce temps que vous avez perdu à cause de ce mikroc. Bien que ses messages d'erreurs ne voulaient rien dire, les lignes qu'ils pointaient sont typique d'un compilateur qui ne comprend que le C89/C90 (avec ici des extensions non standards comme // et 0b). Et donc le fond du problème est que C89/90 ne permet pas de mélanger les déclarations avec les instructions, d'où la nécessité de placer les variables en début de fonction, avant même leurs utilisations.

En ce qui concerne les extensions de ce genre de compilateur, il faut éviter de les utiliser. Ça apprend de mauvaises pratiques, ça ancre de toutes aussi mauvaises habitudes et c'est aussi une façon non avouée de rendre l'utilisateur un peu dépendant au produit. Pour aider, il y a GCC avec ses options -std=c90 et -pedantic. Et comme bonne alternative à 0bxxxx, il y a l'octal (un chiffre représentant un paquet de 3 bits).

Bon courage.
21/04/2020, 12h06
sparow1

Citation:

Mais oh que c'est dommage quand je vois tout ce temps que vous avez perdu à cause de ce mikroc. Bien que ses messages d'erreurs ne voulaient rien dire, les lignes qu'ils pointaient sont typique d'un compilateur qui ne comprend que le C89/C90 (avec ici des extensions non standards comme // et 0b). Et donc le fond du problème est que C89/90 ne permet pas de mélanger les déclarations avec les instructions, d'où la nécessité de placer les variables en début de fonction, avant même leurs utilisations.

Il fallait le savoir, merci :)
Pas très pratique ce mikroC..

Citation:

En ce qui concerne les extensions de ce genre de compilateur, il faut éviter de les utiliser. Ça apprend de mauvaises pratiques, ça ancre de toutes aussi mauvaises habitudes et c'est aussi une façon non avouée de rendre l'utilisateur un peu dépendant au produit. Pour aider, il y a GCC avec ses options -std=c90 et -pedantic. Et comme bonne alternative à 0bxxxx, il y a l'octal (un chiffre représentant un paquet de 3 bits).

Je ne comprends pas bien pourquoi ces précisions, qu'attendez vous que je fasse ? Je suis bloqué, je n'arrive à allumer mes LED avec ce code
21/04/2020, 12h14
Vincent PETIT

Citation:

Envoyé par kaitlyn

Mais oh que c'est dommage quand je vois tout ce temps que vous avez perdu à cause de ce mikroc.

MAIS TU POUVAIS PAS VENIR AVAAANNNTTTT :ptdr: :D

Merci car franchement j'étais entrain d'halluciner lorsque j'ai vu que ça compilait avec toutes les variables en globales (ce qui était une très mauvaise idée) et pourtant des microcontrôleurs j'en ai programmé et beaucoup !
21/04/2020, 14h32
sparow1

Des idées ? J'ai compris ce qu'on a vu ensemble, mais je suis toujours perdu pour la suite
22/04/2020, 13h52
Vincent PETIT

Là je suis sec car je n'ai pas mikroC.

J'utilise Proteus mais je n'ai que la version "démo" et je ne crois pas pouvoir charger un programme dans un Arduino ?

Sous Proteus je sais, car quand j'étais étudiant nous l'avions, que tu peux faire dérouler le programme en mode pas à pas et tu peux voir ce qui se passe en RAM (notamment dans les variables i, j, etc) et tu peux trouver pourquoi ça ne fonctionne pas. Ou du moins voir là où ça ne fait pas ce que c'est sensé faire.

D'ailleurs pourquoi Delay_ms(500); ne fonctionne pas ?????? Pourtant Delay_ms semble être une fonction native de mickroC :aie:

ps : ce soir je vais essayer de placer un PIC et de charger un programme dedans mais je ne garantie rien car j'ai la version d'évaluation de Proteus 8. Envoi moi en zip en pièce jointe, les fichiers qu'a généré MickroC (le .hex et tout le reste)

J'arrête pas de tourner en rond, j'ai regardé ce que j'ai fait au début et fait des recherches supplémentaires.
J'ai réussis à parcourir les led à l'infini, l'une après l'autre elle s'allume.

Ce que je cherche c'est également de simplifier ma boucle.

Code:

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
#include <stdio.h>
// Prototypes
void LED1(unsigned char d) ; // Sous-programme LED1
 
//*********************************************
 
void main() {
 unsigned char donnee = 0b10101010;
 TRISA = 0b00000010; //port A en sortie sauf RA1
 PORTA = 0b11001000 ; //mettre les sorties à zéro sauf RA6, RA7,RA3
 TRISB = 0b00000001; //port B en sortie sauf RB_0
 PORTB = 0 ; //mettre les sorties à zéro
 pcon.OSCF = 1; //configure le bit 3 du registre pcon pour 4 Mhz
 CMCON = 0b00000111; //désactiver les comparateurs sur RA0
 
 
 
 while(1)  //boucle infinie
 {
  if (RB0_bit==0){ //si RB0 = 0 exécuter la ligne suivante
   LED1(donnee) ; //Sous-programme LED1
 } //fin du SI
}
}
 
//*********************************************
 
//sous-programme
void LED1(unsigned char d){ // Début du Sous-programme LED1, on allume et éteind les leds de D4 à D11 l'une après l'autre
 for(;;){
  RA0_bit = 1 ;    //sortie RA0 à 1
  delay_ms(250);   //delai d'1/4 sec
  RA0_bit = 0 ;    //sortie RA0 à 0
  RB1_bit = 1 ;
  delay_ms(250);
  RB1_bit = 0 ;
  RB2_bit = 1 ;
  delay_ms(250);
  RB2_bit = 0 ;
  RB3_bit = 1 ;
  delay_ms(250);
  RB3_bit = 0 ;
  RB4_bit = 1 ;
  delay_ms(250);
  RB4_bit = 0 ;
  RB5_bit = 1 ;
  delay_ms(250);
  RB5_bit = 0 ;
  RB6_bit = 1 ;
  delay_ms(250);
  RB6_bit = 0 ;
  RB7_bit = 1 ;
  delay_ms(250);
  RB7_bit = 0 ;
 }
} // Fin du sous-programme LED1

Je n'arrive pas avec le code qu'on a effectué je suis convaincu qu'il ne manque qu'un petit quelque chose mais je ne sais pas quoi

Code:

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
void LED1(unsigned char d, unsigned int nbr_fois)
{
    unsigned int ma_variable_16bits;
    unsigned int i;
    unsigned int j;
 
    ma_variable_16bits = d;
 
    for(j = 0; j < nbr_fois; j++)
    {
        for(i = 0; i < 8; i++)
        {
			if(ma_variable_16bits & 0x0100 == 1)
				RB1_bit = 1 ; // le bit n°8 est à 1, j'allume la LED
			else
				RB1_bit = 0 ; // le bit n°8 est à 0, j'éteins la LED
			if(ma_variable_16bits & 0x0200 == 1)
				RB2_bit = 1 ; // le bit n°9 est à 1, j'allume la LED
			else
				RB2_bit = 0 ; // le bit n°9 est à 0, j'éteints la LED
			if(ma_variable_16bits & 0x0400 == 1)
				RB3_bit = 1 ; // le bit n°10 est à 1, j'allume la LED
			else
				RB3_bit = 0 ; // le bit n°10 est à 0, j'éteints la LED
			if(ma_variable_16bits & 0x0800 == 1)
				RB4_bit = 1 ; // le bit n°11 est à 1, j'allume la LED
			else
				RB4_bit = 0 ; // le bit n°11 est à 0, j'éteints la LED
			if(ma_variable_16bits & 0x1000 == 1)
				RB5_bit = 1 ; // le bit n°12 est à 1, j'allume la LED
			else
				RB5_bit = 0 ; // le bit n°12 est à 0, j'éteints la LED
			if(ma_variable_16bits & 0x2000 == 1)
				RB6_bit = 1 ; // le bit n°13 est à 1, j'allume la LED
			else
				RB6_bit = 0 ; // le bit n°13 est à 0, j'éteints la LED
			if(ma_variable_16bits & 0x4000 == 1)
				RB7_bit = 1 ; // le bit n°14 est à 1, j'allume la LED
			else
				RB7_bit = 0 ; // le bit n°14 est à 0, j'éteints la LED
 
 
            Delay_ms(500);
            ma_variable_16bits = ma_variable_16bits << 1;
        }
 
        ma_variable_16bits = ma_variable_16bits | d; // petite modif je fais un OU au lieu d'un +
    }
}
 
 
void main()
{
	unsigned char donnee = 0b10101010;
 
 	TRISA = 0b00000010; //port A en sortie sauf RA1
 	PORTA = 0b11001000 ; //mettre les sorties à zéro sauf RA6, RA7,RA3
 	TRISB = 0b00000001; //port B en sortie sauf RB_0
 	PORTB = 0 ; //mettre les sorties à zéro
 	pcon.OSCF = 1; //configure le bit 3 du registre pcon pour 4 Mhz
 	CMCON = 0b00000111; //désactiver les comparateurs sur RA0
 
	while(1)  //boucle infinie
 	{
  		if (RB0_bit==0) {
       		LED1(donnee, 10) ;
 		}
	}
}

Voir 40 message(s) de cette discussion en une page