Problème de programmation PIC via mikroC

Version imprimable

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

Quand je suis bloqué je fais ça ; on met ton programme à poil, c'est à dire qu'on va retirer tout ce qui ne sert pas à comprendre "où est le problème" et ajouter les choses une part une, mais vraiment une par une sinon tu ne vas jamais trouver ce qui ne va pas.

Pas de fonction, pas de gestion d'interrupteur et on allume qu'une seule LED avec un masque et rien d'autre. Une programme minimal. Si le bit vaut 1 alors

Ci dessous, à compiler par tes soins, un programme qui regarde qu'elle est la valeur du bit 0 de donne. Si le bit n°0 vaut 1 alors RB1 clignote sinon il ne clignote pas. La LED devrait clignoter car donnee vaut 0b00000001

Est ce que ce truc minimaliste fonctionne ?
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 #include <stdio.h> void main() { unsigned char donnee = 0b00000001; 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) { if(donnee & 0x01 == 1) { RB1_bit = 1 ; delay_ms(250); RB1_bit = 0 ; } else { RB1_bit = 0 ; } } }
ps : tiens c'est bizarre delay_ms a un 'd' minuscule tandis que dans le code qui ne marche pas c'est un 'D' majuscule. Ça aussi, élimine le doute, si ça clignote tu essayes delay avec et sans majuscule histoire qu'on ne se pose plus la question plus tard. Soit c'est pareil, soit un des deux ne marche pas.

22/04/2020, 21h57
sparow1

1 pièce(s) jointe(s)

Pour commencer, ça compile.
Mais quand je test sur Proteus : au début dans la simulation la LED clignote 1 seul fois. Et puis la LED est constamment allumée, que j'appuie ou non sur le bouton.
J'ai testé avec et sans la majuscule le delay ça ne change rien.
Pièce jointe 559236

Citation:

Envoyé par Vincent PETIT

MAIS TU POUVAIS PAS VENIR AVAAANNNTTTT :ptdr: :D

Salut,

J'essaie, enfin, quand j'ai un moment de détente. Mais du temps de libre est un luxe que je n'ai malheureusement pas, contrairement à vous, les mecs. :p

J'allais terminer sur une salutation, mais tu sais quoi,

( !!! Risque de style télégraphique. Rédiger sur téléphone en émulation desktop n'est pas évident :\ ).

Bon tu sais quoi, j'ai regarder le début de la discussion, la fin, je visualise un peu près ce que sparow1 veut faire: boucle, tableau, RBx_bit et tout...
Et grâce a toi il est au fait du décalage de bit et de l’algèbre de Boole. Très bon point.

Je vais essayer d'aider un peu:

Objectif: reproduire RBx_bit=y au niveau des PORTs, permettre l'usage boucle/tableau.

Piste de réflexion: Encoder une représentation PORT_X/bit_x RXx_bit utilisable par 2 fonctions-outils de modèles set_bit, clear_bit transposées aux leds et tout en restant indépendant du sens de montage de ces dernières <=> abstraction (désolée, j'abrège.)

Encodage et définition de LEDs
Type famille: entier
Déclaration/définition: aux choix uchar, enum, #define

exemple
Code:

1 2 3 #define LED0 0xA0 #define LED1 0xB1 /*...*/
où A0 dans 0xA0 représente le bit O du PORT A, et le A0 dans RA0_bit

Implémentation (tentative d') des 2 fonctions-outils :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 void set_led(unsigned char led_id) { if ((led_id&0xa0)==0xa0) PORTA|=(1<<(led_id&0x0f)); if ((led_id&0xb0)==0xb0) PORTB|=(1<<(led_id&0x0f)); } void clear_led(unsigned char led_id) { if ((led_id&0xa0)==0xa0) PORTA&=~(1<<(led_id&0x0f)); if ((led_id&0xb0)==0xb0) PORTB&=~(1<<(led_id&0x0f)); }
où un set_led(0xPn) serait équivalent à RPn_bit=1,
et clear_led(0xPn) serait équivalent à RPn_bit=0

Usage possible, manipulation de LEDs
Code:

1 2 3 set_led(LED1) delais_ms(250) clear_led(LED1)
J'espère que ça aidera.
Bye.

ps: il faudra renommer la fonction LED1() en autre chose.

1 pièce(s) jointe(s)

Je comprends à peu près ce que tu veux faire, j'ai essayé de mettre ça dans mon code.
Mais ça reste un peu brouillon, j'ai réussi à enlever les erreurs, mais ça ne donne pas le résultat attendu.

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
#include <stdio.h>
 
#define LED4 0xA0
#define LED5 0xB1
#define LED6 0xB2
#define LED7 0xB3
#define LED8 0xB4
#define LED9 0xB5
#define LED10 0xB6
#define LED11 0xB7
 
void set_led(unsigned char led_id) {
 if ((led_id&0xa0)==0xa0)
  PORTA|=(1<<(led_id&0x0f));
 if ((led_id&0xb0)==0xb0)
  PORTB|=(1<<(led_id&0x0f));
}
 
void clear_led(unsigned char led_id) {
 if ((led_id&0xa0)==0xa0)
  PORTA&=~(1<<(led_id&0x0f));
 if ((led_id&0xb0)==0xb0)
  PORTB&=~(1<<(led_id&0x0f));
}
 
void LEDcontrol(unsigned char d){ // Début du Sous-programme LED1, on allume et éteind les leds de D4 à D11 l'une après l'autre
 unsigned int ma_variable_16bits = 0;
 ma_variable_16bits = d;
 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
 
void main() {
 unsigned char donnee = 0b00000001;
 
 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)
 {
  if(donnee & 0x01 == 1) {
   set_led(0xA0);
   delay_ms(250);
   clear_led(0xA0);
  }
 }
}

Dès que je démarre la simulation, la LED (D4) s'allume une fois très rapidement, puis reste constamment allumé que j'appuie ou non sur le bouton.
Pièce jointe 559402

Essaye avec ceci, sans gestion de bouton :
Code:

1 2 3 4 5 6 while(1){ set_led(LED4); delay_ms(1000); clear_led(LED4); delay_ms(1000); }

23/04/2020, 13h53
sparow1

La LED 4 clignote bien, selon le délai imposé
23/04/2020, 21h05
kaitlyn

Ah mais il ne fallait pas attendre sur moi. Tu avais tout ce qu'il faut pour faire ta boucle, ce que tu souhaitais simplifier avec un tableau. Tu te fais une nouvelle fonction que tu appelles depuis ton while. Tu y mets un tableau que tu remplis avec des LEDX: unsigned char leds[]={LED4, LED5, ....}; que tu parcours avec ta boucle for, puis set_led(leds[i]), etc.

Fais d'abord sans bouton, et après avec tout en expliquant le comportement que tu en attendais.

Je bloque en fait..

je dois faire une fonction juste pour le tableau ?

Code:

1
2
3
4
 
void tabLed(unsigned char led_id){
 unsigned char leds[]={LED4, LED5, LED6, LED7, LED8, LED9, LED10, LED11};
}

J'ai des erreurs dans le while, à cause du type de tabLed, je sais pourquoi mais je n'arrive plus à le corriger. Ça fait un petit moment que j'ai fait du C, pour tout vous dire

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
#include <stdio.h>
 
#define LED4 0xA0
#define LED5 0xB1
#define LED6 0xB2
#define LED7 0xB3
#define LED8 0xB4
#define LED9 0xB5
#define LED10 0xB6
#define LED11 0xB7
 
void set_led(unsigned char led_id) {
 if ((led_id&0xa0)==0xa0)
  PORTA|=(1<<(led_id&0x0f));
 if ((led_id&0xb0)==0xb0)
  PORTB|=(1<<(led_id&0x0f));
}
 
void clear_led(unsigned char led_id) {
 if ((led_id&0xa0)==0xa0)
  PORTA&=~(1<<(led_id&0x0f));
 if ((led_id&0xb0)==0xb0)
  PORTB&=~(1<<(led_id&0x0f));
}
void tabLed(unsigned char led_id){
 unsigned char leds[]={LED4, LED5, LED6, LED7, LED8, LED9, LED10, LED11};
}
/*
void LEDcontrol(unsigned char d){ // Début du Sous-programme LED1, on allume et éteind les leds de D4 à D11 l'une après l'autre
 unsigned int ma_variable_16bits = 0;
 ma_variable_16bits = d;
 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
*/
void main() {
 unsigned char donnee = 0b00000001;
 
 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){
  set_led(tabLed);
  delay_ms(250);
  clear_led(tabLed);
  delay_ms(250);
 }
}

Ce n'est pas grave.

Ta fonction disons play_mode1() :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
void play_mode1(void)
{
    const unsigned char leds[]= {LED4, LED5, ........ };
    const unsigned char leds_size= sizeof(leds)/sizeof(leds[0]);
    unsigned char i;
 
    for (i= 0; i < leds_size; i++) {
        set_led(leds[i]);
        delay_ms(250);
        clear_led(leds[i]);
    }
}

Ton while :

Code:

1
2
3
while (1) {
    play_mode1();
}

Nickel ça fonctionne, les LED s'allument de la 4 à la 11 à l'infini
Donc maintenant il faut gérer le bouton

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
 
#include <stdio.h>
 
#define LED4 0xA0
#define LED5 0xB1
#define LED6 0xB2
#define LED7 0xB3
#define LED8 0xB4
#define LED9 0xB5
#define LED10 0xB6
#define LED11 0xB7
 
void set_led(unsigned char led_id) {
 if ((led_id&0xa0)==0xa0)
  PORTA|=(1<<(led_id&0x0f));
 if ((led_id&0xb0)==0xb0)
  PORTB|=(1<<(led_id&0x0f));
}
 
void clear_led(unsigned char led_id) {
 if ((led_id&0xa0)==0xa0)
  PORTA&=~(1<<(led_id&0x0f));
 if ((led_id&0xb0)==0xb0)
  PORTB&=~(1<<(led_id&0x0f));
}
 
void play_mode1(void)
{
    unsigned char leds[]={LED4, LED5, LED6, LED7, LED8, LED9, LED10, LED11};
    const unsigned char leds_size=sizeof(leds)/sizeof(leds[0]);
    unsigned char i;
 
    for (i= 0; i < leds_size; i++) {
        set_led(leds[i]);
        delay_ms(250);
        clear_led(leds[i]);
    }
}
 
void main() {
 unsigned char donnee = 0b00000001;
 
 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){
  play_mode1();
 }
}

24/04/2020, 09h26
Vincent PETIT

Juste une petite remarque, en #64 les LED D5 à D11 sont câblé à l'envers. Elles ne peuvent pas s'allumer, peut être que l'outil Proteus en fait abstraction pendant la simulation ?
24/04/2020, 10h14
sparow1

Schéma imposé, c'est une contrainte en plus...
24/04/2020, 12h07
Vincent PETIT

Oui ça d'accord mais un schéma faux imposé c'est sacrément embêtant car c'est sur que ça va pas marcher. Si les LED sont à l'envers, aucune chance qu'elles s'allument même avec un code qui marche.

Dans les autres schéma que tu as posté, comme en début de discussion, les LED étaient dans le bon sens.

LED à l'envers dans ton message ici https://www.developpez.net/forums/d2.../#post11497483

LED à l'endroit dans celui là https://www.developpez.net/forums/d2.../#post11491049

Si j'étais toi je vérifierai ce point pour éviter de tourner en rond, à moins que le simulateur s'en fou ? Mais ça serait très étonnant. Ce que je veux dire c'est que tu peux te battre longtemps avec le code si jamais c'est le schéma qui ne va pas.
24/04/2020, 12h21
sparow1

1 pièce(s) jointe(s)

Et bien, c'est les mêmes.
C'est juste dans la simulation avec Proteus, lorsqu'elle s'allume il la "retourne".
Je ne sais pas pourquoi, mais c'est comme ça apparemment

Pièce jointe 559764
24/04/2020, 13h43
Vincent PETIT

Citation:

Envoyé par sparow1

C'est jusque dans la simulation avec Proteus, lorsqu'elle s'allume il la "retourne".

C'est parce qu'il te montre qu'elles sont à l'envers et c'est très certainement pour ça qu'elles s'allument en jaune.
D1, D2 et D3, elles, sont à l'endroit.

Visiblement, Proteus semble en tenir compte mais ne bloque pas le fonctionnement, donc ça va.
24/04/2020, 17h18
sparow1

1 pièce(s) jointe(s)

Effectivement... Maintenant c'est bon j'ai tout revérifié, c'est ok. Bien vu :)

Mais avec ce code les led D1, D2 et D3 s'allument à certains moment.
Pièce jointe 559862

Donc maintenant il me faut gérer le bouton
24/04/2020, 17h39
Vincent PETIT

1 pièce(s) jointe(s)

Euh non, seule D6 est à l'endroit.

Citation:

Envoyé par sparow1

Mais avec ce code les led D1, D2 et D3 s'allument à certains moment.

Attention car il se peut que le simulateur soit perturbé par les mauvais câblages. Fait même très attention car peut être que tu n'arrives pas à mettre au point le logiciel à cause de ça.

Voici ton image modifiée (j'ai fait de copier/coller) pour te montrer le bon sens des LED, que j'ai entouré avec des ronds violets

Pièce jointe 559868
25/04/2020, 00h02
sparow1

3 pièce(s) jointe(s)

Je les ai retourné juste avant je ne comprends plus rien

Le schéma que j'ai depuis le début :
Pièce jointe 559963

Le schéma que j'ai depuis votre intervention en #75 :
Pièce jointe 559955
Pièce jointe 559959
25/04/2020, 08h58
kaitlyn

Bonjour,

Citation:

Envoyé par sparow1

Donc maintenant il faut gérer le bouton

Citation:

Envoyé par sparow1

Donc maintenant il me faut gérer le bouton

Tu fais bien de nuancer, mais ce n'est pas en te répétant que cela se fera tout seul.

Petit rappel :

Citation:

Envoyé par kaitlyn

Fais d'abord sans bouton, et après avec tout en expliquant le comportement que tu en attendais.

Et donc on attend sur toi.

Citation:

Envoyé par Vincent PETIT

C'est parce qu'il te montre qu'elles sont à l'envers et c'est très certainement pour ça qu'elles s'allument en jaune.

Oui! Pour symboliser un courant inverse. Ça explique le prix, 6000 euros :mrgreen:
25/04/2020, 18h36
sparow1

La répétition n'était pas volontaire, je tourne juste en rond sur ce projet.. Je n'y arrive pas et j'ai l'impression de ne pas avancer

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