[PIC] demande d'aide pour démarrer en c ou asm

demande d'aide pour démarrer en c ou asm [PIC] - Nano-ordinateur, microcontrôleurs, FPGA - Electronique, domotique, DIY

Marsh Posté le 22-08-2022 à 13:58:48    

Bonjour,
 
Je cherche de l'aide pour démarrer un projet en c ou assembleur, tous les tutos que je trouve sont pour des vieilles versions de mplab..
Pour un 16f88
 


---------------
Ce n'est pas parce que vous avez des certitudes que vous avez raison
Reply

Marsh Posté le 22-08-2022 à 13:58:48   

Reply

Marsh Posté le 22-08-2022 à 16:06:55    

C'est un exercice scolaire? :o
 
Tu as quel problème/... exactement?

Reply

Marsh Posté le 22-08-2022 à 23:54:48    

C'est en théorie tout simple :
Je souhaite faire avec le µC un compteur 3 digits avec boutons +/- et enregistrer 2 valeurs (High et Low) dans la mémoire non volatile, et rappeler ensuite ces 2 valeurs en fonction d'un bouton (Hi/Lo).
 
Mais je ne sais même pas comment démarrer correctement avec MPLABX, dès que j'essaye de compiler il y a toujours un problème avec des fichiers qu'on récupère un peu partout sur le net qui ne sont plus mis à jour pour d'anciennes versions de MPLab et j'avoue ça me décourage pas mal.

Message cité 1 fois

---------------
Ce n'est pas parce que vous avez des certitudes que vous avez raison
Reply

Marsh Posté le 23-08-2022 à 01:49:39    

Je ne connais pas MPLABX, mais je pense que ton problème est plutôt là:

iosys99 a écrit :

un problème avec des fichiers qu'on récupère un peu partout sur le net


On ne récupère pas des fichiers n'importe où mais - en supposant que tu parles des header pour ton PIC - directement chez le fabriquant du CI ou du compilateur/IDE/... Donc dans ton cas quelque part là bas je suppose: https://www.microchip.com/en-us/too [...] plab-x-ide
 
Et puis il faudrait être plus précis: Quel(s) fichier(s) récupéré(s) où exactement Quel problème exactement? ...

Reply

Marsh Posté le 06-11-2022 à 16:08:48    

J'y suis finalement arrivé.
J'ai commencé par installer une ancienne version de MPLABxIDE et suivi quelques tutoriels.
Et j'étais assez embeté avec un kit PICKIT v2 qui semble abandonné par tout le monde..
J'ai acheté un PICKIT v4 pour programmer en live mon montage, puis suivi quelques tutos plus récents.
J'ai finalement abouti à un programme complet qui fait presque ce que je veux mais j'en suis fier ^^
J'ai d'ailleurs découvert qu'il y a des fonctions intégrées dans le code qui évitent d'en coder et débuguer soi même par exemple pour stocker en RAM une variable on la déclare simplement, puis on l'utilise en faisant attention à la criticité de la rapidité du code (interruptions pendant un enregistrement en EEPROM), mais le compilateur gère ça tout seul apparemment :
 
// Variables declarations stored on EEPROM
unsigned char __eeprom iEEUnitsHi;
unsigned char __eeprom iEETensHi;
unsigned char __eeprom iEEHundredsHi;
unsigned char __eeprom iEEUnitsLo;
unsigned char __eeprom iEETensLo;
unsigned char __eeprom iEEHundredsLo;


Message édité par iosys99 le 06-11-2022 à 17:20:55

---------------
Ce n'est pas parce que vous avez des certitudes que vous avez raison
Reply

Marsh Posté le 06-11-2022 à 16:14:32    

Le schema :
 
https://img.super-h.fr/images/2022/11/06/c1b39aa3ad8ce29e40740e7a4edfca26.th.jpg
 
Le code:

Code :
  1. /* Main.c file generated by New Project wizard
  2. *
  3. * Created:   lun. oct. 17 2022
  4. * Processor: PIC16F882
  5. * Compiler:  MPLAB XC8
  6. */
  8. #include <xc.h>
  9. #include "Config16f882.h"
  11. #define _XTAL_FREQ 20000000
  12. #define PreLoad     65480
  14. #define SW1 PORTCbits.RC0 // Units up
  15. #define SW2 PORTCbits.RC1 // Tens up
  16. #define SW3 PORTCbits.RC2 // Hundreds up
  17. #define SW4 PORTCbits.RC3 // Hi/lo mode
  19. unsigned int iCurrent_Digit=0;
  21. const unsigned char iSegments_Code[11] = {
  22.     0b00111111, // 0
  23.     0b00000110, // 1
  24.     0b01011011, // 2
  25.     0b01001111, // 3
  26.     0b01100110, // 4
  27.     0b01101101, // 5
  28.     0b01111101, // 6
  29.     0b00000111, // 7
  30.     0b01111111, // 8
  31.     0b01101111, // 9
  32.     0b00000000, // 10 digit off
  33. };
  35. // Variables declarations stored on EEPROM
  36. unsigned char __eeprom iEEUnitsHi;
  37. unsigned char __eeprom iEETensHi;
  38. unsigned char __eeprom iEEHundredsHi;
  39. unsigned char __eeprom iEEUnitsLo;
  40. unsigned char __eeprom iEETensLo;
  41. unsigned char __eeprom iEEHundredsLo;
  43. // Because of the interrupt subroutine (ISR),
  44. // we working in main() AND in ISR with non EEPROM variables
  45. int iUnitsHi=0;
  46. int iTensHi=0;
  47. int iHundredsHi=0;
  48. int iUnitsLo=0;
  49. int iTensLo=0;
  50. int iHundredsLo=0;
  52. void TMR1_Init(void);
  54. void main(void) {
  56.     // Recall variables values from EEPROM
  57. iUnitsHi = iEEUnitsHi;
  58. iTensHi = iEETensHi;
  59. iHundredsHi = iEEHundredsHi;
  60. iUnitsLo = iEEUnitsLo;
  61. iTensLo = iEETensLo;
  62. iHundredsLo = iEEHundredsLo;
  64.     // In case EEPROM data is corrupted, display default 0 by corrupted digit
  65. if (iUnitsHi > 10) {iUnitsHi = 0;}
  66. if (iTensHi > 10) {iTensHi = 0;}
  67. if (iHundredsHi > 10) {iHundredsHi = 0;}
  68. if (iUnitsLo > 10) {iUnitsLo = 0;}
  69. if (iTensLo > 10) {iTensLo = 0;}
  70. if (iHundredsLo > 10) {iHundredsLo = 0;}
  72.     TMR1_Init();
  74.     TRISA = 0x00; // Segments on output
  75.     PORTA = iSegments_Code[0]; // All port A bits off
  76.     TRISB = 0x00; // Digit selection on output
  77.     PORTB = 0x00; // Digit selection on off
  78.     TRISC = 0b11111111; // Buttons on inputs, all port C
  79.     ADCON1 = 0x0F;
  81.    
  82.     while(1)
  83.     {
  84.         __delay_ms(2);
  85.    // First we check Hi / Lo mode
  86.     if(SW4==1)
  87.     {
  88.     // We are in High Mode
  89.         if(SW1==1)      //PRESSED SWITCH
  90.         {
  91.              while(SW1==1) // Waiting button release
  92.                 {
  93.                     NOP();
  94.                 }
  95.             // Increment Units
  96.             if (iUnitsHi < 10)
  97.             {
  98.              
  99.             iUnitsHi=iUnitsHi+1;
  100.             iEEUnitsHi=iUnitsHi ;
  101.             }
  102.     else
  103.     iUnitsHi=0;
  104.         iEEUnitsHi=iUnitsHi ;
  105.         }
  106.         if(SW2==1)      //PRESSED SWITCH
  107.         {
  108.             while(SW2==1) // Waiting button release
  109.                 {
  110.                     NOP();
  111.                 }
  112.             // Increment Tens
  113.             if (iTensHi < 10)
  114.             {
  115.                
  116.             iTensHi=iTensHi+1;
  117.             iEETensHi=iTensHi;
  118.             }
  119.     else
  120.     iTensHi=0;
  121.         iEETensHi=iTensHi;
  122.             }
  123.         if(SW3==1)      //PRESSED SWITCH
  124.         {
  125.             while(SW3==1) // Waiting button release
  126.                 {
  127.                     NOP();
  128.                 }
  129.             // Increment Hundreds
  130.             if (iHundredsHi < 10)
  131.             {
  132.                
  133.             iHundredsHi=iHundredsHi+1;
  134.             iEEHundredsHi=iHundredsHi;
  135.             }
  136.     else
  137.     iHundredsHi=0;
  138.         iEEHundredsHi=iHundredsHi;
  139.         }
  140.       }
  141.     else
  142.     {
  143.     // We are in Low mode
  144.         if(SW1==1)      //PRESSED SWITCH
  145.         {
  146.                while(SW1==1) // Waiting button release
  147.                 {
  148.                     NOP();
  149.                 }
  150.             // Increment Units
  151.             if (iUnitsLo < 10)
  152.             {
  153.            
  154.             iUnitsLo=iUnitsLo+1;
  155.             iEEUnitsLo=iUnitsLo;
  156.             }
  157.     else
  158.     iUnitsLo=0;
  159.         iEEUnitsLo=iUnitsLo;
  160.         }
  161.        
  162.         if(SW2==1)      //PRESSED SWITCH
  163.         {
  164.               while(SW2==1) // Waiting button release
  165.                 {
  166.                     NOP();
  167.                 }
  168.             // Increment Tens
  169.             if (iTensLo < 10)
  170.             {
  171.              
  172.             iTensLo=iTensLo+1;
  173.             iEETensLo=iTensLo;
  174.             }
  175.     else
  176.     iTensLo=0;
  177.         iEETensLo=iTensLo;
  178.             }
  179.        
  180.        
  181.         if(SW3==1)      //PRESSED SWITCH
  182.         {   
  183.             while(SW3==1) // Waiting button release
  184.                 {
  185.                     NOP();
  186.                 }
  187.             // Increment Hundreds
  188.             if (iHundredsLo < 10)
  189.             {
  190.          
  191.             iHundredsLo=iHundredsLo+1;
  192.             iEEHundredsLo=iHundredsLo;
  193.             }
  194.     else
  195.     iHundredsLo=0;
  196.         iEEHundredsLo=iHundredsLo;
  197.         }
  198.     }
  199.     }
  200.     return;
  201. }
  202. //===================================
  203. //--------------[ ISR ]--------------
  204. void __interrupt() ISR(void)
  205. {
  206.    if(TMR1IF)
  207.    {
  208.       //GIE=0;
  209.     __delay_ms(1);
  210.     // First we check Hi / Lo mode
  211.     if(SW4==1) // High Mode
  212.     {
  213.      if(iCurrent_Digit==0) // Units
  214.      {
  215.          // PORTA = iSegments_Code[10];
  216.  PORTA = iSegments_Code[iUnitsHi];
  217.          PORTB = 0b00000001;
  218.        
  219.      }
  220.      if(iCurrent_Digit==1) // Tens
  221.      {
  222.          //PORTA = iSegments_Code[10];
  223.  PORTA = iSegments_Code[iTensHi];
  224.  PORTB = 0b00000010;
  225.        
  226.      }
  227.      if(iCurrent_Digit==2) // Hundreds
  228.      {
  229.          //PORTA = iSegments_Code[10];
  230.  PORTA = iSegments_Code[iHundredsHi];
  231.          PORTB = 0b00000100;
  232.        
  233.      }
  234.       }
  235.       else // Low mode
  236.       {
  237.            if(iCurrent_Digit==0) // Units
  238.      {
  239.          // PORTA = iSegments_Code[10];
  240.  PORTA = iSegments_Code[iUnitsLo];
  241.          PORTB = 0b00000001;
  242.        
  243.      }
  244.      if(iCurrent_Digit==1) // Tens
  245.      {
  246.          //PORTA = iSegments_Code[10];
  247.  PORTA = iSegments_Code[iTensLo];
  248.  PORTB = 0b00000010;
  249.        
  250.      }
  251.      if(iCurrent_Digit==2) // Hundreds
  252.      {
  253.          //PORTA = iSegments_Code[10];
  254.  PORTA = iSegments_Code[iHundredsLo];
  255.          PORTB = 0b00000100;
  256.        
  257.      }
  258.       }
  259.      iCurrent_Digit++;
  260.      if(iCurrent_Digit==3)
  261.      {
  262.         iCurrent_Digit=0;
  263.       }
  264.      TMR1IF = 0;
  265.      TMR1 = PreLoad; // Preloading Timer1
  266.      //GIE=1;
  267.    }
  268. }
  269. //----------[ END OF ISR ]-----------
  270. //===================================
  271. // Functions' Definitions
  272. void TMR1_Init()
  273. {
  274.   // -- [[ Configure Timer1 To Operate In Timer Mode ]] --
  275.   // Preload TMR1 Register Pair
  276.   TMR1 = PreLoad;
  277.   // Choose the local clock source (timer mode)
  278.   TMR1CS = 0;
  279.   // Choose the desired prescaler ratio (1:8)
  280.   T1CKPS0 = 1;
  281.   T1CKPS1 = 1;
  282.   // Switch ON Timer1 Module!
  283.   TMR1ON = 1;
  284.   // Config TMR1 Interrupt
  285.   TMR1IE = 1;
  286.   TMR1IF = 0;
  287.   PEIE = 1;
  288.   GIE = 1;
  289. }


 
J'ai supprimé le le bouton STO inutile en fait et pas eu à gérer le code pour enregistrer en EEPROM le compilateur l'a fait pour moi  :love:
J'ai pour objectif de faire un speed display screen facile à programmer pour remplacer ceux des vieux PC années 80.
 
https://www.minuszerodegrees.net/le [...] isplay.htm
 
https://img.super-h.fr/images/2022/11/06/189df7c7886c8742bb680c8001001b46.th.jpg
 
Edit : J'ai oublié, j'ai un bug à résoudre, ça ne restore pas les zéros quand on boot le programme après extinction du courant...
Je fais une petite vidéo.
Edit : Optimisation de la gestion de l'appui du bouton, bug de la restauration des chiffres résolu, code mis à jour... Je vais refaire une vidéo de présentation non buguée ^^
 
Video : https://streamable.com/38pc2b


Message édité par iosys99 le 06-11-2022 à 17:21:06

---------------
Ce n'est pas parce que vous avez des certitudes que vous avez raison
Reply

Marsh Posté le 06-11-2022 à 17:48:31    

Bravo pour ta tenacité.
 
Quelque commentaires si tu veux bien:
-Il est très important d'avoir des résistances de tirage (pullup/down) sur les entrées, par contre il ne faut pas en mettre en série avec les boutons car tu fabriques un pont diviseur de tension. Selon les valeurs et les caractéristiques des entrées du PIC un appui risque de ne pas être enregistré (correctement). Aussi dans les AVR il y a des pullup intégrés qu'on peut activer, pour les PIC je ne sais pas.
-Tu devrais faire attention à la mise en page de ton code, ça évite les erreurs ou fait qu'on les trouve plus facilement et c'est plus facile à lire aussi.
-Le délai dans l'ISR c'est très moche, autant modifier la configuration du timer qui génère cette interruption.
-Tes variables globales ne sont pas _modifiés_ dans l'ISR, sinon il faudrait les déclarer "volatile".
-Pour l'EEPROM il faudrait voir si le code généré par le compilateur est suffisamment "intelligent" pour écrire l'EEPROM que quand une valeur change, car ce type de mémoire à un nombre de cycles limité.
 
A part ça le code est plutôt propre pour un débutant (ou pas débutant??). Après on peut toujours optimiser, genre utiliser switch() au lieu de tout ces if(), mais ce genre de choses c'est largement une question de goût aussi.
 

Reply

Marsh Posté le 06-11-2022 à 19:27:50    

J'ai fait de l'assembleur il y a 25 ans (HP 48 / Saturn) mais peu de C, et pas mal de Visual basic 6 debut 2000 (oui ça fait un grand écart !).
Pas totalement débutant mais après 20 ans, on va dire que pas loin du noob... !
 
en revanche certaines de tes explications sont un poil trop techniques pour moi ^^
 
- Les résistances pull-up/down :  
https://img.super-h.fr/images/2022/11/06/7becb787a75bb40016261b9a1d00b6bd.th.jpg
 
La je ne sais pas trop, j'ai des notions d'électronique (thevenin, norton, mailles, noeuds etc lol)
- Mise en page : alors là, je me base surtout sur l'indentation, si tu as un document sur la mise ne page de code... je prends :)
- Le délai dans l'ISR, là, totaly noob : J'ai pris un code à interrruption existant pour afficher un chiffre nombre sur un 3 segment LED Digit avec la persistance de la vision humaine ^^.
- Les variables alors j'ai de la chance, je ne les modifie pas dans l'interrruption... Mais je ne vois pas ce que signifie volatile dans un code C comme ça.
- Là je vais me renseigner chez microchip puisque j'utilise toute leur chaine de codage jusqu'au programmeur :)
 
Mais vraiment merci pour ta contribution !


Message édité par iosys99 le 06-11-2022 à 19:30:05

---------------
Ce n'est pas parce que vous avez des certitudes que vous avez raison
Reply

Marsh Posté le 08-11-2022 à 19:25:50    

Je ne connais pas les PIC, mais si j'interprête bien ce tableau il y a des pullup que sur le port B, or tes boutons sont branchés ailleurs. Pour quelque résistances autant laisser comme ça...
 
Pour l'indentation justement elle n'est pas cohérente dans ton code, mais c'est peut-être un soucis avec le forum. Il y a différents "styles" (et beaucoup de débats autour :o ), regarde simplement chez Wikipédia. Il y a des outils ("beautifier" ) pour automatiquement formater un code existant.
 
Le délai dans l'ISR: Comme je disais il faudrait modifier la configuration du Timer, mais je ne connais pas les PIC... Je laisse ma place à quelqu'un d'autre. De manière générale les interruption c'est pour des trucs critiques au niveau temps/délai/réactivité et on essaye de faire des interruptions les plus courtes possibles pour ne pas "bloquer" le processeur trop longtemps (sachant que l'interruption peut intervenir à n'importe quel moment et parfois - genre pour un timer - très souvent). Donc du délai dans une ISR c'est pas interdit et dans ton cas simple ça ne pose (probablement) pas de problème, mais c'est moche. :o  
 
Pour le "volatile", tu peux demander à ton moteur de recherche préféré ;). En bref ce mot clé indique au compilateur que la variable peut changer de valeur à n'importe quel moment (dans une ISR justement), du coup il faut récupérer la valeur "réelle" à chaque fois et ne pas utiliser une "copie" qu'on aurait stocké quelque part dans un registre du processeur ou similaire. Dans ce contexte il faut aussi faire attention au "atomic access" pour les variables >1 octet, je te laisse te renseigner si tu veux.

Reply

Marsh Posté le 08-11-2022 à 20:51:26    

[:onclecharlie:4]  
Je vais potasser tout cela car tu as a quand même des notions de coding bien avancées pour moi, mais comme je prends tout conseil et je me renseigne par la suite.
 
[:cyberad] Tu me montres le chemin, c'est à moi qu'il appartient de le suivre   [:apyrael:1]  
 
Merci beaucoup pour ton aide en tout cas !


Message édité par iosys99 le 08-11-2022 à 20:51:52

---------------
Ce n'est pas parce que vous avez des certitudes que vous avez raison
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed