MICROCONTROLEURS 12F683 Interruption ne fonctionne pas

[invitead6c50a3]

Bonne nouvelle!
La routine d'interruption si elle est mal déclarée sera traitée comme une simple fonction et le compilateur ne la traitera pas comme il se doit, d'où le message de warning que tu avais.
Je pense que c'était le problème.
INTCON qui contient les activations d'interruptions doit être déclaré après avoir configurer le timer0.
-----

[invitead6c50a3]

Maintenant il te reste à ajouter ton BP et configurer le registre qui va gérer l'interruption.
C'est un bon exercice

[invite4d59cbf7]

J'ai rajouté une interruption extèrieure ( doublant la fréquence) en mettant un contact + sur GP2.
ça ne marche pas la fréquence est constamment doublée sans que j'appuie.

Le programme, en gras ce que j'ai rajouté:

#include <xc.h>
#include <pic12f683.h>

// Définition de la fréquence du crystal pour la fonction __delay_ms(x);
#define _XTAL_FREQ 31000
#define led GP4
// CONFIG
#pragma config FOSC = INTOSCIO // Oscillator Selection bits (INTOSCIO oscillator: I/O function on RA4/OSC2/CLKOUT pin, I/O function on RA5/OSC1/CLKIN)
#pragma config WDTE = OFF // Watchdog Timer Enable bit (WDT disabled)
#pragma config PWRTE = OFF // Power-up Timer Enable bit (PWRT disabled)
#pragma config MCLRE = OFF // MCLR Pin Function Select bit (MCLR pin function is digital input, MCLR internally tied to VDD)
#pragma config CP = OFF // Code Protection bit (Program memory code protection is disabled)
#pragma config CPD = OFF // Data Code Protection bit (Data memory code protection is disabled)
#pragma config BOREN = OFF // Brown Out Detect (BOR disabled)
#pragma config IESO = OFF // Internal External Switchover bit (Internal External Switchover mode is disabled)
#pragma config FCMEN = OFF // Fail-Safe Clock Monitor Enabled bit (Fail-Safe Clock Monitor is disabled)



void __interrupt () my_isr_routine (void)
{
if ((INTCONbits.T0IF == 1) && (INTCONbits.T0IE == 1))
{
TMR0 = 16;
led =~ led;
INTCONbits.T0IF = 0;
}
if ((INTCONbits.INTF == 1) && (INTCONbits.INTE ==1))
{
OPTION_REGbits.PS2 = 0;
OPTION_REGbits.PS1 = 1;
OPTION_REGbits.PS0 = 0;

INTCONbits.INTF = 0;
}
}


void main() {
ANSEL = 0b00000000;

TRISIO = 0b00000100;

OSCCON = 0b00000111;

// timer

OPTION_REGbits.T0CS = 0;
OPTION_REGbits.T0SE = 0;
OPTION_REGbits.PSA = 0;
OPTION_REGbits.PS2 = 1;
OPTION_REGbits.PS1 = 0;
OPTION_REGbits.PS0 = 0;

OPTION_REGbits.nGPPU = 1;
OPTION_REGbits.INTEDG = 1;

TMR0 = 16; // le timer0 doit contenir ce préchargement pour avoir 1s

//interruptions

INTCON = 0b11110000; // ici on active les interruptions

IOC = 0b00000100;


while (1)
{
}
}

[invite4d59cbf7]

soit la broche est au + constamment, soir le flag INTF ne se remet pas à 0 ??

[invite4d59cbf7]

pourtant j'ai désactivé les pull_up et même mis GP2 = 0 dans le main

[invite4d59cbf7]

je vais repasser en revue tous les registres

[invitead6c50a3]

Il faut que tu mettes une pull-down sur GP2, par exemple une 10K reliée au GND.
Ton BP sera relié entre GP2 et Vcc.
As-tu fait ça?

[invitead6c50a3]

Corrige ce registre:

ANSEL = 0b00000100; // déclare GP2 en entrée analogique

[invitead6c50a3]

Tu peux aussi utiliser le pull-up interne et mettre ton BP au GND.
Pour cela il faut activer le pull-up:

OPTION_REGbits.nGPPU = 0; // GPIO Pull-up Enable bit
OPTION_REGbits.INTEDG = 0; // Détection sur front descendant
WPU=0b00000100; // pull-up sur GP2 activé

[invite4d59cbf7]

Je fais ça et te tiens au courant, merci, bonne soirée.

[invite4d59cbf7]

Sans rien changer en rallumant le montage:

Au départ clignotement 1Hz - OK - j'appuie sur BP clignotement 0.5Hz - OK- et si je le relache le clignotement ne reviens pas à 1Hz, il continue à 0.5Hz.
Si j'éteins et je rallume, idem.
On dirait que le Flag ne s'efface pas malgré l'instruction.


Ensuite j'ai mis GP2 en analogique, je l'ai relié au GND par une 12K:

La led clignote à 1Hz , le BP n'a aucun effet

Je vais essayer avec le pull-up

[invite4d59cbf7]

Je confirme en remettant GP2 en numérique, le BP fonctionne mais l'interruption continue si on le relache

[invite4d59cbf7]

en retirant la 12K, le montage passe en 0.5Hz, GP2 doit donc bien être polarisé en négatif.
J'essaye avec le pull_up

[invite4d59cbf7]

avec le pull-up, 1Hz et PB n'a aucun effet

[invite4d59cbf7]

le OPTION_REGbits.INTEDG : front montant ou descendant fait bien son effet ( effet si on appuie ou relache le BP)

[invite4d59cbf7]

OPTION_REGbits.nGPPU = soit 1 soit 0
OPTION_REGbits.INTEDG = idem
n'ont aucun effet

[invite4d59cbf7]

en résumé, avec ce programme , l'interruption PB fonctionne mais ne s'arrête pas ( flag reste à 1 ou se remet à 1 automatiquement ?). Et le système se réinitialise à 1Hz en éteignant et rallumant le montage:

#include <xc.h>
#include <pic12f683.h>

// Définition de la fréquence du crystal pour la fonction __delay_ms(x);
#define _XTAL_FREQ 31000
#define led GP4
// CONFIG
#pragma config FOSC = INTOSCIO // Oscillator Selection bits (INTOSCIO oscillator: I/O function on RA4/OSC2/CLKOUT pin, I/O function on RA5/OSC1/CLKIN)
#pragma config WDTE = OFF // Watchdog Timer Enable bit (WDT disabled)
#pragma config PWRTE = OFF // Power-up Timer Enable bit (PWRT disabled)
#pragma config MCLRE = OFF // MCLR Pin Function Select bit (MCLR pin function is digital input, MCLR internally tied to VDD)
#pragma config CP = OFF // Code Protection bit (Program memory code protection is disabled)
#pragma config CPD = OFF // Data Code Protection bit (Data memory code protection is disabled)
#pragma config BOREN = OFF // Brown Out Detect (BOR disabled)
#pragma config IESO = OFF // Internal External Switchover bit (Internal External Switchover mode is disabled)
#pragma config FCMEN = OFF // Fail-Safe Clock Monitor Enabled bit (Fail-Safe Clock Monitor is disabled)



void __interrupt () my_isr_routine (void)
{
if ((INTCONbits.T0IF == 1) && (INTCONbits.T0IE == 1))
{
TMR0 = 16;
led =~ led;
INTCONbits.T0IF = 0;
}
if ((INTCONbits.INTF == 1) && (INTCONbits.INTE ==1))
{
OPTION_REGbits.PS2 = 0;
OPTION_REGbits.PS1 = 1;
OPTION_REGbits.PS0 = 0;

INTCONbits.INTF = 0;
}
}


void main() {
ANSEL = 0b00000000;

TRISIO = 0b00000100;

OSCCON = 0b00000111;

// timer

OPTION_REGbits.T0CS = 0;
OPTION_REGbits.T0SE = 0;
OPTION_REGbits.PSA = 0;
OPTION_REGbits.PS2 = 1;
OPTION_REGbits.PS1 = 0;
OPTION_REGbits.PS0 = 0;



TMR0 = 16; // le timer0 doit contenir ce préchargement pour avoir 1s

//interruptions

INTCON = 0b11110000; // ici on active les interruptions

IOC = 0b00000100;

GP2 = 0;

while (1)
{
}
}

[RISC]

Salut,

#include <xc.h>
#include <htc.h>
#include <pic12f683.h>
#include <pic.h>

Toutes ces lignes sont redondantes...Il suffit de mettre celle là :

#include <xc.h>

Je pense que ton problème vient peut*-être de cette ligne :

#define _XTAL_FREQ 31000

Cette ligne dit au compilateur..la fréquence du PIC est 31000 Hz...
Le PB c'est que si cette déclaration est incorrecte, le calcul des délais sera complètement faux...
Tu as entré la valeur de l'oscillateur basse vitesse (31000)...mais j'ai l'impression que tes bits de configuration sélectionne l'oscillateur haute vitesse ...qui est à 4MHz....
Tu devrais essayer de changer cette ligne avec :

#define _XTAL_FREQ 4000000

a+

[invite4d59cbf7]

Salut,
merci, je vais essayer
Bonne nuit

[invitead6c50a3]

Au départ clignotement 1Hz - OK - j'appuie sur BP clignotement 0.5Hz - OK- et si je le relache le clignotement ne reviens pas à 1Hz, il continue à 0.5Hz.
Si j'éteins et je rallume, idem.
On dirait que le Flag ne s'efface pas malgré l'instruction.

Attention, ce n'est pas comme ça qu'on programme.
Essayer des choses sans modifier les registres qui vont avec, on y sera encore à Noël.
Il faut que tu lises la doc du µC et que tu fasses les choses de manière structurée.
Par exemple comment veux-tu que la tempo revienne à l'initial si tu ne modifie pas les conditions du timer?
Le registre ANSEL gère les I/O, si tu déclares GP2 en numérique ce n'est pas correct.
Donc pose toi et écris un flowchart qui décomposera les actions à mener, tu verras que les choses vont t'apparaître plus simplement.

Si ton timer0 réalise bien la seconde de tempo c'est que ta base de temps est bien reconnue.
En même temps tu n'as pas choisi le meilleur des compilateurs....
Bon courage pour la suite.

[invite4d59cbf7]

Bonjour,

J'ai vérifié, les configurations sont cohérentes. Même pb si je fais tourner à 4Mhz

bonne journée

[invite4d59cbf7]

OK, bonne journée

[invitead6c50a3]

Sans rien changer en rallumant le montage:

Au départ clignotement 1Hz - OK - j'appuie sur BP clignotement 0.5Hz - OK- et si je le relache le clignotement ne reviens pas à 1Hz, il continue à 0.5Hz.
Si j'éteins et je rallume, idem.
On dirait que le Flag ne s'efface pas malgré l'instruction.


Ensuite j'ai mis GP2 en analogique, je l'ai relié au GND par une 12K:

La led clignote à 1Hz , le BP n'a aucun effet

Je vais essayer avec le pull-up

Fais au moins cette modification:

void __interrupt () my_isr_routine (void)
{
if ((INTCONbits.T0IF == 1) && (INTCONbits.T0IE == 1))
{
TMR0 = 16;
OPTION_REGbits.PS2 = 1;
OPTION_REGbits.PS1 = 0;
OPTION_REGbits.PS0 = 0;
led =~ led;
INTCONbits.T0IF = 0;
}
if ((INTCONbits.INTF == 1) && (INTCONbits.INTE ==1))
{
OPTION_REGbits.PS2 = 0;
OPTION_REGbits.PS1 = 1;
OPTION_REGbits.PS0 = 0;

INTCONbits.INTF = 0;
}
}

Remets en situation tes registres dans la version qui fonctionnait à 1Hz avec la bonne déclaration à 31000Hz.

[invitead6c50a3]

Ca donnera sans doute un fonctionnement bizarre, vu que pour faire les choses proprement il faudrait que tu ajoutes une variable dans la routine d'interruption.

Fonctionnellement ça devrait donner ça:

1/ on charge le timer0 avec la valeur d'offset 16 et la config des bits <PS2:PS0> pour avoir 1s.
2/ lors de l'appui sur BP tu entres dans la routine d'interruption et tu actives une variable, par exemple t_change = ~t_change;
Il faut déclarer cette variable et l'initialiser à 0;

Dans le while(1) tu écris une condition qui ressemble à ça:

if(t_change == 1) 
  {
  OPTION_REGbits.PS2 = 0;
  OPTION_REGbits.PS1 = 1;
  OPTION_REGbits.PS0 = 0;
  }
else
  {
  TMR0 = 16;
  OPTION_REGbits.PS2 = 1;
  OPTION_REGbits.PS1 = 0;
  OPTION_REGbits.PS0 = 0;
  }

Ainsi ton timer0 cadencera à la bonne valeur à chaque appui sur ton BP.

[invitead6c50a3]

Ce qui donnerait à tester ceci:

unsigned char t_change = 0;

void __interrupt () my_isr_routine (void)
{
if ((INTCONbits.T0IF == 1) && (INTCONbits.T0IE == 1))
  {
TMR0 = 16;
led =~ led;
INTCONbits.T0IF = 0;
  }
if ((INTCONbits.INTF == 1) && (INTCONbits.INTE ==1))
 {
t_change = ~t_change:
INTCONbits.INTF = 0;
 }
}

......

while(1)
{  
  if(t_change == 1) 
  {
  OPTION_REGbits.PS2 = 0;
  OPTION_REGbits.PS1 = 1;
  OPTION_REGbits.PS0 = 0;
  }
else
  {
  TMR0 = 16;
  OPTION_REGbits.PS2 = 1;
  OPTION_REGbits.PS1 = 0;
  OPTION_REGbits.PS0 = 0;
  }
}

[RISC]

Salut Exotique,

Je pense que tu peux simplifier ton écriture en utilisant les "champs de bits" :

  OPTION_REGbits.PS2 = 0;
  OPTION_REGbits.PS1 = 1;
  OPTION_REGbits.PS0 = 0;

ou :

  OPTION_REGbits.PS = 0b010;

a+

[invitead6c50a3]

On peut alléger l'écriture par:

unsigned char t_change = 0;

void __interrupt () my_isr_routine (void)
{
if ((INTCONbits.T0IF) && (INTCONbits.T0IE))
  {
TMR0 = 16;
led =~ led;
INTCONbits.T0IF = 0;
  }
if ((INTCONbits.INTF) && (INTCONbits.INTE))
 {
t_change = ~t_change:
INTCONbits.INTF = 0;
 }
}

......

while(1)
{  
  if(t_change) 
  {
  OPTION_REGbits.PS2 = 0;
  OPTION_REGbits.PS1 = 1;
  OPTION_REGbits.PS0 = 0;
  }
else
  {
  TMR0 = 16;
  OPTION_REGbits.PS2 = 1;
  OPTION_REGbits.PS1 = 0;
  OPTION_REGbits.PS0 = 0;
  }
}

[invite4d59cbf7]

et encore merci.

[invite4d59cbf7]

J'avais essayé d'inverser le prescaler dans l'interruption mais ça ne fonctionnait pas, je n'avais pas pensé à une variable.

J'ai appliqué ta modif, ça ne fonctionne pas, la led reste allumée et aucun effet du BP

[invite4d59cbf7]

je vai essyer une variable à 1 bit ( _Bool)