Programma contagiri... ****!

[DeMonio83924EL]

Salve a tutti!

Avrei bisogno di un aiuto... Purtroppo il programma che ho fatto per il contagiri della moto (in allegato lo schema circuitale) Non da segni di vita... e dal momento che non riesco a fare la simulazione con "MikroC pro demo", forse perché è la versione demo, non riesco a capire dove si impianta il programma... ho provato pure ad accendere i led in più punti del programma e provando direttamente sul circuito ma nessun segno di vita!

Se qualcuno di voi ha un po' di tempo per dare un'occhio per darmi una mano, metto a disposizione il progetto che allego qui sotto. Per il progetto è stato impiegato il PIC18F2450 con il programmatore PicKit 2.

Il datasheet del PIC18F24 è scaricabile da QUESTO link!

Ringrazio anticipatamente tutti!!

[RenzoDF]

Sono certo che qualche "nostro" esperto ti dara' una mano! ...

DeMonio83924EL e' un mio "raccomandato"

[DeMonio83924EL]

Grazie mille RenzoDF!!

Comunque allego pure una foto del circuito che ho realizzato!

back_circuit

foto_mand02.jpg (302.84 KiB) Osservato 2915 volte

front_circuit

foto_mand01.jpg (324.03 KiB) Osservato 2916 volte

[TardoFreak]

Gli ho dato uno sguardo solo che e' scritto male.
Riscrivirlo per bene e cioe':
- La chiamata alla funzione di inizializzazione non serve ad un tubo di niente perche' viene fatta una volta sola all' inizio del programma. Quindi scrivila nel main.
- L' unica variabile globale non inizializzarla durante la dichiarazione ma all' interno del main. E' una questione di stile che pero' permette una maggiore linearita' e leggibilita' del programma. Sei agli inizi ed e' bene prendere le buone abitudini.
- Devi usare un solo timer! Te ne basta uno per avere la base tempi a 100ms.
- Scrivi le inizializzazioni bit per bit, una riga per volta e commentando cosa fa quel bit. In questo modo il programma e' piu' leggibile ed e' piu' facile individuare gli errori.
Esempio con MPLAB-C18 ma che vale anche per mikroC.

Codice: Seleziona tutto

  //Timer 0 utilizzato a 16 bit
  T0CONbits.T08BIT = 0;

...

  // Start Timer 0
  T0CONbits.TMR0ON = 1;

[c1b8]

Oltre a quanto detto da TardoFreak posso aggiungere, ad uno sguardo veloce, che manca l'impostazione delle porte come digitali, se non ricordo male bisogna muovere il valore esadecimale 0F nel registro ADCON1.

[RenzoDF]

Grazie ragazzi

... solo una domanda
su Eagle5 vers. Free non trovo in libreria il PIC 18F2450 ... son io che son "cecato" o non ce sta?

Trovato !

PIC18F2*_28

[DeMonio83924EL]

Grazie mille TardoFreak!!

Ok lo inizio subito a riscrivere bene! Però una cosa... ho ben utilizzato solo il timer 0 per il conteggio dei 100ms e il timer 1 per il conteggio asincrono giusto no?! e un'altra cosa... per esempio i bit del registro T0CON sono normalmente impostati a valore 0 giusto?! quindi se un valore deve rimaner 0 non lo scrivo giusto?!

Grazie pure a te c1b8!
Ora cerco sul datasheet!

[DeMonio83924EL]

Trovato !

Io ho una versione precedente di Eagle ora lo cerco con quel nome magari lo becco!!

[TardoFreak]

... e il timer 1 per il conteggio asincrono giusto no?! ...

Giusto si, ma se non scrivi bene il programma, se non ci metti commenti bisogna andarsi a cercare i bit uno per uno ed immaginare quello che volevi fare.
Abbonda di commenti UTILI, servono a te e a chi dovra' eventualmente metterci le mani sopra.

[DeMonio83924EL]

Ok grazie!! rimedio subito e posto il tutto, appena ho finito!!!
Grazie!!