microchip, C18, variable folle Fermé

Question

Bonjour,
Je suis en train de faire un afficheur pour afficher les horaires de bus de Rennes dans mon appart, j'ai déjà réalisé avec succès ce projet sur un arduino. J'ai cependant besoin de libérer mon arduino pour un autre projet.
J'ai donc sortit un 18f2580 pour l'occasion. Je travaille avec mplabX et C18.

J'ai un problème qui me semble défier les lois de la programmation, je vous explique :

var = 38%10;//Récupérer '8'printCar(var+48); //+48 pour la conversion en ascii, et afficher '8'// --> mon écran affiche '8', pas de problème!
Maintenant voilà quelque chose de plus utile, mais qui tourne pas :
autreVar = 38;var = autreVar%10;//Récupérer '8'printCar(var+48); //l'afficher '8'// --> mon écran affiche '0' peut importe la valeur de "autreVar"

J'hésite entre passer le karcher sur le pic ou mon ordi...

Merci de votre analyse, j'attend patiemment une réponse, voir solution?

Titus.

fiddy · Answer

Bonjour,

Tout d'abord, plutôt que mettre 48, je te conseille de mettre '0'. C'est plus lisible et plus portable :-).

Sinon, le problème est bizarre. Ca sent le bon vieux stack overflow. Peux-tu donner ton code en entier ? 

Cdlt,

titusIII · Answer

char tmp,hourNextL3;   char next3[] = "Next 3 : ";   cursor(0,1);   ecr_string(next3);   hourNextL3 = 28;   tmp = hourNextL3 % 10;   ecr_data(tmp + 48);

Mon code est pas beaucoup plus compliqué.
Les drivers de mon LCD est fait maison, il a fait ses preuves, je l'écarte donc.

titusIII · Answer

J'ai eu pas mal de problème pas très cohérents avec C18... Une autre histoire de variable que j'ai résolu en appliquant un genre de décalage de bit un peu bourrin...

Si tu avais un autre compilateur moins nerveux?

titusIII · Answer

excuse moi :
char tmp,hourNextL3;char next3[] = "Next 3 : ";void fct(){   cursor(0,1);   ecr_string(next3);   hourNextL3 = 28;   tmp = hourNextL3 % 10;   ecr_data(tmp + '0');}

Et toujours 0 en sortie.
Et tout mon code est consacré au drivers LCD pour l'instant, mais je l'écarte.

Utilisateur anonyme · Answer

Bonjour

Je n'ai jamais eu de problèmes avec C18.
Vois-tu bien l'affichage de la chaîne "Next 3 : " ?
Peux-tu donner le prototype de ecr_data ? Pourquoi n'as tu pas gardé la fonction printCar que tu utilisais dans ton post initial ?

titusIII · Answer

Parce que j'ai changé le nom de ma fonction, ça reste la même.

void ecr_data(char data){  PORTBbits.RB2 = 1; hbit = data; hbit >>=4; PORTBbits.RB4 = hbit&0x01;  PORTCbits.RC6 = (hbit>>1)&0x01; PORTCbits.RC5 = (hbit>>2)&0x01; PORTCbits.RC4 = (hbit>>3)&0x01; PORTBbits.RB3 = 1; PORTBbits.RB3 = 0;   Delay10TCYx(1); PORTBbits.RB4 = data&0x01;  PORTCbits.RC6 = (data>>1)&0x01; PORTCbits.RC5 = (data>>2)&0x01; PORTCbits.RC4 = (data>>3)&0x01;  PORTBbits.RB3 = 1; PORTBbits.RB3 = 0;  Delay10TCYx(5);}
C'est la fonction qui me permet d'écrire un caractère sur mon afficheur, elle tourne bien, "Next 3 : " est bien affiché!

titusIII · Answer

une autre exemple :
   hourNextL3 = 28;   tmp = hourNextL3 % 10;   if (tmp == 8) ecr_data(8 + '0'); // n'affiche rien

Alors que 
   ecr_data(8 + '0'); // affiche bien 8

Utilisateur anonyme · Answer

Je viens de tester chez moi, ça se passe bien (en simulation de 18F4550).
Peux-tu vérifier dans le .map à quelles adresses sont tes variables hourNextL3  et tmp ?

titusIII · Answer

Hournextl3 à 0x74
Et tmp à 0x78
Je vois pas le probleme encore une fois!

Je précise que j'ai déja changé de variable pour voir, de type de variable également.

Sinon j'ai une piste, toute petite et autant improbable... J'ai un timer qui génère une interruption toutes les secondes (je m'en sers pour faire horloge, heures minutes secondes), c'est assez improbable que ça fasse planter mon code, vu qu'il y a un système de PUSH-POP en arrière plan je suppose.
Ma fonction avec le modulo est appelée toutes les secondes et j'ai JAMAIS le résultat attendu. Ca serai un sacré manque de chance que mon timer la perturbe à ce point. J'essayerai sans pour voir, demain, quand j'aurai les yeux sur mon matériel.

Utilisateur anonyme · Answer

Tes variables déclarées char tmp,hourNextL3; se retrouvent à 4 octets d'écart ? Étonnant, non ? Tu as une explication ?

J'avais pensé aussi à une modification due à une interruption cyclique, mais si elle n'a lieu que toutes les secondes, elle ne peut pas fausser ton résultat à tous les coups.

As-tu essayé :
   hourNextL3 = '8'; 
   tmp = hourNextL3 ;
   ecr_data(tmp);

titusIII · Answer

Oui je les déclarent pas sur la meme ligne, bizarre ça me ressemble pas.
Je vais essayer le code que tu me proposes, je connais je pense déjà le résultat! Ca va marcher

Utilisateur anonyme · Answer

As-tu essayé ton code avec le simulateur de MPLABX pour voir à partir de quand les valeurs diffèrent de celles que tu attends ?

D'autre part, peux-tu mettre ici le code assembleur généré par le compilateur pour ton opération qui ne marche pas ?

titusIII · Answer

Ouaip, 
Le code en C :

cursor(0,1);
   ecr_string(next3);
   testT = 17;
   tmp = testT%10;
   ecr_data(tmp + '0');


Et en asm j'arrive pas à mettre la main sur le .asm, y'a un moyen particulier sur mplabX ?

J'ai testé avec le simulateur intégré et aucun problème

Utilisateur anonyme · Answer

Pas facile de te suivre : ton code change à chaque fois, ainsi que les noms de tes fonctions et de tes variables... Et tu n'as pas répondu à l'essai que je t'avais demandé.

Quand tu dis qu'il n'y a aucun problème avec le simulateur, qu'est-ce que ça veut dire ? Le simulateur n'a pas d'afficheur ! Et quand tu simules, exécutes-tu tout bien depuis le reset du PIC ?

Est-il possible d'avoir le programme complet ?

titusIII · Answer

48:               cursor(0,1);
0516  6EE6     MOVWF POSTINC1, ACCESS
0518  6AE6     CLRF POSTINC1, ACCESS
051A  EC8E     CALL 0x31C, 0
051C  F001     NOP
051E  52E5     MOVF POSTDEC1, F, ACCESS
0520  52E5     MOVF POSTDEC1, F, ACCESS
49:               ecr_string(next3);
0522  0EB6     MOVLW 0xB6
0524  6EE6     MOVWF POSTINC1, ACCESS
0526  0E01     MOVLW 0x1
0528  6EE6     MOVWF POSTINC1, ACCESS
052A  EC6F     CALL 0x2DE, 0
052C  F001     NOP
052E  52E5     MOVF POSTDEC1, F, ACCESS
0530  52E5     MOVF POSTDEC1, F, ACCESS
50:               testT = 17;
0532  0E11     MOVLW 0x11
0534  0100     MOVLB 0x0
0536  6F79     MOVWF testT, BANKED
0538  6B7A     CLRF 0x7A, BANKED
51:               tmp = testT%10;
053A  C079     MOVFF testT, __tmp_0
053C  F010     NOP
053E  C07A     MOVFF 0x7A, 0x11
0540  F011     NOP
0542  0E0A     MOVLW 0xA
0544  6E0D     MOVWF 0xD, ACCESS
0546  6A0E     CLRF 0xE, ACCESS
0548  C010     MOVFF __tmp_0, 0x8
054A  F008     NOP
054C  C011     MOVFF 0x11, 0x9
054E  F009     NOP
0550  ECB8     CALL 0x570, 0
0552  F002     NOP
0554  C004     MOVFF 0x4, __tmp_0
0556  F010     NOP
0558  C005     MOVFF 0x5, 0x11
055A  F011     NOP
055C  C010     MOVFF __tmp_0, tmp
055E  F078     NOP
52:               ecr_data(tmp + '0');
0560  0E30     MOVLW 0x30
0562  0100     MOVLB 0x0
0564  2578     ADDWF tmp, W, BANKED
0566  6EE6     MOVWF POSTINC1, ACCESS
0568  EC0F     CALL 0x21E, 0
056A  F001     NOP
056C  52E5     MOVF POSTDEC1, F, ACCESS

titusIII · Answer

0532  0E11     MOVLW 0x11
0534  0100     MOVLB 0x0
0536  6F79     MOVWF testT, BANKED
0538  6B7A     CLRF 0x7A, BANKED

Quelque chose qui va pas ici me semble t'il...
On déplace 0x11 vers W, on sélectionne la banque 0x0 pour mettre mon 0x11 ?
Et on efface la banque 0x7A ?

Ca fait longtemps que j'ai pas fait d'assembleur, je dois pas être loin de la vérité

Utilisateur anonyme · Answer

J'étais INSAlien de 1974 à 1979 et c'était déjà la ligne 3 :)

ta ligne 0538  6B7A     CLRF 0x7A, BANKED montre que ta variable testT est en fait un int. J'ai fait l'essai, quand on déclare un char, il n'essaye pas de mettre à 0 le poids fort d'un char, il ne le fait que pour un int. Mais passons.

Deux questions :

Est-il possible d'avoir le programme en entier ? (via cjoint.com ou autre)
Peux -tu afficher deux fois consécutives le chiffre :
ecr_data(tmp + '0');ecr_data(tmp + '0');

titusIII · Answer

Effectivement je l'avais laissée en int pour mes tests...

Je peux t'envoyer le programme, par mail? Peux-tu m'envoyer une adresse mail?

titusIII · Answer

Rectification, en sortie avec :

testT = 17;
   tmp = testT%10;
   ecr_data(tmp + '0');


J'ai 'T'.
Soit 84-48=36

36?
17%10=36 d'après mon pic

Utilisateur anonyme · Answer

Heureusement que quand je t'avais fait remarquer qu'une de tes variables était un int, tu m'avais confirmé que c'était un char. Maintenant, tu te rends compte que c'est un int. Passons encore, mais essaye de donner des réponses exactes.
As-tu fait l'essai que je t'avais demandé de faire deux affichages consécutifs :
   
  ecr_data(tmp + '0');
  ecr_data(tmp + '0');
Pour l'envoi du code, zippe tout en un seul fichier et utilise le site cjoint.com ou un autre si tu as une préférence.

titusIII · Answer

J'ai deux fois le caractère T, j'ai remis ma variable en char.

Je zip le tout ce soir et te l'envoie.

titusIII · Answer

Voila mon code :
https://www.cjoint.com/?3BtuY6zxFoC

Utilisateur anonyme · Answer

Je vois que le code a encore changé : tu as ajouté des remises à 0 des sorties après chaque impulsion : tu remets ton driver LCD en cause, finalement ?
Et tes variables ont changé de banque. Je ne peux malheureusement pas compiler ni désassembler, je n'ai pas MPLABX à la maison et je n'ai pas envie de l'installer.
Mais je vois que tu es supposé afficher l'heure sur la première ligne. Mis à part que tu oublies, il me semble, de faire sl++ au début et que ton test if (sl > 10) est incorrect (il faudrait >=), est-ce que cet affichage-là marche ?

Ceci étant dit, j'avoue que je ne trouve toujours pas d'explication

titusIII · Answer

Cet affichage marche très bien! L'heure est correcte et s'affiche bien.

J'ai en effet remi mon drivers en question, et il marche aussi comme ça.
Pas de changement sur mon résultat de modulo.

Passer sur un pic 16F ou un autre compilateur tel que HITECH (je doute de la prise en charge des 18F) résoudrait peut etre le problème.
J'essayerai demain. ca m'embete d'arreter le projet sur un problème comme celui là.

Le sl++ se fait en interruption

Utilisateur anonyme · Answer

Le sl++ se fait en interruption
Effectivement, j'aurais dû le voir. Mais ça n'est pas logique : elle devrait être incrémentée dans time() comme les autres variables de comptage de temps. Et il faudra que tu m'expliques comment tu fais pour avoir un affichage correct du temps alors que tu n'incrémentes les dizaines que toutes les 11 secondes ! Pendant la dernière de chaque 11 secondes, tu dois afficher : (deux points) qui est le code ASCII qui suit '9'.

Personnellement, j'ai fait tourner des applications écrites avec C18 sur des 18F, j'utilise l'opérateur modulo et je n'ai jamais rien remarqué.

Aux dernières nouvelles, ça affiche un 'T' avec testT=17 : peux-tu essayer d'autres valeurs pour essayer de dégager une loi ?

titusIII · Answer

Pour :
testT = 17 affichage de 'T'
testT = 18 affichage de 'T'
testT = 19 affichage de 'T'
testT = 0 affichage de 'T'
Bon... Visiblement il s'en fou de ma variable.

Une autre piste? Je n'utilise pas de picKit ou ICD. Je suis dans le non-officiel avec cette chose : https://login.aliexpress.com/?from=sm&return_url=http://www.aliexpress.com/item/536517358.html

Ça marche plutôt bien, seul inconvénient, il faut régler les fuses à chaque fois dans le logiciel fourni avec. Je pense les avoirs bien réglés.
Je t'envoie un screenshot de mes réglages.

titusIII · Answer

Mes réglages : http://cjoint.com/data3/3BunwvW9v3R.htm

Utilisateur anonyme · Answer

Encore une idée : n'as-tu pas mélangé les modèles (small et large) dans ton projet ? Utilises-tu bien la bonne bibliothèque ? Il me semble (je n'ai pas de doc où je suis) qu' il y a plusieurs façons de gérer la mémoire et il faut bien utiliser à l'édition de liens la bibliothèque compatible avec le modèle choisi.

titusIII · Answer

Alors là, je suis perdu! J'ai jamais entendu parler de ça!
Peux-tu m'en dire plus?

Utilisateur anonyme · Answer

Hors sujet, après vérification, ça concerne les pointeurs sur le code.
J'ai horreur de l'avouer, mais je sèche...
Sinon, peux-tu confirmer que ton comptage de secondes a bien un problème de comptage  ?

titusIII · Answer

Oui, effectivement il y avait bien un problème, j'avais pas remarqué.
C'est des vraies 10 secondes maintenant, merci!

J'aimerai également faire quelques chose pour mon drivers LCD :
ecrire("bonjour");

au lieu de :
char var[] = "bonjour";
ecrire(var);

J'ai essayé quelques truc mais rien de s'affiche, il doit y avoir une solution?
J'ai pensé à faire un prototype dans ce genre :
void ecrire(const char p*);
Mais rien!

Utilisateur anonyme · Answer

Pour ce dernier point, je devrais pouvoir t'aider.
Dans les PIC, les bus d'adresse programme et de données, ainsi que les mémoires correspondantes, sont totalement séparés.
Il existe des pointeurs RAM et des pointeurs ROM. Ces deux types sont totalement incompatibles : une adresse de la mémoire programme n'a absolument rien qui lui correspond dans la mémoire vive, et lycée de Versailles.
Quand tu définis :
char var[] = "bonjour";

Tu définis une variable (initialisée) en RAM et ta fonction "ecrire" attend un pointeur de type ram : ça tombe bien.
Quand tu appelles directement ecrire("bonjour"); "bonjour" n'est pas affecté à une variable, il est "logé" en mémoire non volatile, donc la mémoire programme. donc tu essayes de passer à ecrire un pointeur de type rom : je suppose que le compilateur refuse.

Ré-écris une autre version, copie quasi-conforme de "ecrire",  mais avec un paramètre rom char * et ça devrait marcher.
Tu peux toujours garder tes messages prédéfinis sous la forme 
const rom char var[] = "bonjour";
comme ça ton texte "bonjour" est et reste en mémoire programme, alors que tel que tu le définis actuellement, il est défini dans la mémoire vive, ce qui oblige à en avoir une copie dans la mémoire programme (recopiée vers la RAM au lancement du programme).

Microchip, C18, variable folle

32 réponses

Discussions similaires