ccp pic18f46k20 problem compile

[sebalitter]

Hello!

I have a problem trying to compile this program in ccs and pic18f46k20 in mplab 8 IDE.

[asmboy]

what compiler was this written for ?
and the repeating error :
Undefined identifier
is exactly spot on correct.

you don't declare any of those VAR/Register names that i can see,

in reading what you coded, i have to ask if
have you read the CCS manual,
ESPECIALLY including the section entitled:

General Purpose I/O ?

[Ttelmah]

It's a mix of code for about three different compilers.

CCS header, and some fuses.

Microchip fuses.

Some Microchip code, and a few unidentified bits.....

Even worse the settings differ. So the CCS part is selecting the HS oscillator, while the Microchip part selects XT.

There are occasions when you think a sledgehammer would be the best educational tool......

[asmboy]

There are a number of awesome expressions in this 'program'
but the logic of following is my hands down favorite:
keeping in mind that tiempo is a float ........

[sebalitter]

Thanks for the answers!


I didn't explain what i want to do.

I have a speedometer that makes a rectangular pulse with ups and downs.
The number of ups in a second * k gives the speed.
so, I have to count on the rise.
The signal is on pin_C2.


updated and compiling OK.

It has to display the result in the 7-segment leds but the disply doesn't show anything.
is the CCP declaration correct?
I think that is interrupting wrong in CCP1_isr() .


#include <18F46K20.h>
#fuses HS,NOWDT,NOPROTECT,NOLVP
#use delay(clock=10000000)
//#use rs232(baud=9600, xmit=PIN_C6, rcv=PIN_C7)



//definiciones del Programa
#define LEDRojo1 PIN_A0 //anodo1 del led rojo
#define LEDRojo2 PIN_A1 //anodo2 del led rojo
#define LEDVerde1 PIN_C2 //anodo2 del led rojo
#define LEDVerde2 PIN_C3 //anodo2 del led rojo
#define SENSOR PIN_A2
#define FOTO PIN_E0 //RELE para la camara de fotos
#define k 0.022 //constante para cambio de unidades (Km/hora, M/seg, etc)

#define INTS_PER_SECOND 10 // (10000000/(4*4*65536)) = 9.53


//funciones prototipo
void Print_LED(void); //imprime el LED
float Scan(void); //escanea el teclado de funciones

//Variables globales
int contador=0,duplicado=0,dup=0, dig1,dig2, repeticion, repet;
float tiempo=1,vel, medicion=0; //tiempo entre cada medicion. Debe ser menor o igual a 1 segundo
long rise,fall,pulse_width,count=0;
char digito[2];

int8 seconds; // A running seconds counter
int8 int_count; // Number of interrupts left before a second has elapsed

//Los eventos serán bajadas o subidas de los pines correspondientes a CCP1 y CCP2 que son, respectivamente, RC2 y RC1

#int_ccp1
void CCP1_isr()
{
count++;
}


#int_timer1 // This function is called every time
//void clock_isr() { // timer 1 overflows (65535->0), which is
void TIMER1_isr(void)
{ // approximately 19 times per second for
if(--int_count==0) { // this program.
++seconds;
int_count = INTS_PER_SECOND;
Print_LED();
}
}


void main(void)
{

SET_TRIS_B( 0xFF ); // pone B0, B1, B2 , B3 B4, B5, B6 y B7 como salidas
SET_TRIS_D( 0xFF ); // pone D0, D1, D2 , D3 D4, D5, D6 y D7 como salidas
SET_TRIS_A( 0xFF ); // pone A0 y A1 como salidas , A2 como entrada
SET_TRIS_E( 0xFF ); // pone E como salida
SET_TRIS_C( 0b01101111 ); // pone C como salida, C2 como entrada (sensor)

int_count = INTS_PER_SECOND;
setup_timer_1(T1_INTERNAL | T1_DIV_BY_4);
set_timer1(0);
enable_interrupts(INT_TIMER1);

enable_interrupts(INT_RTCC);

enable_interrupts(INT_CCP1); // Setup interrupt on falling edge
enable_interrupts(GLOBAL);

setup_ccp1(CCP_CAPTURE_RE); // Configure CCP1 to capture rise


while(true){}

}


void Print_LED(void)
{
vel= count * k;
//seteo de variables a cero
count=0;
//seconds=0;
//set_timer1(0);
//vel=contador;
if(vel>30){
//if(!duplicado)
// FOTO=1;
output_high(LEDVerde1);
output_high(LEDVerde2);
output_high(LEDRojo1);
output_high(LEDRojo2);
}
else if (vel<=30 && vel>8){
output_high(LEDVerde1);
output_high(LEDVerde2);
output_high(LEDRojo1);
output_high(LEDRojo2);
}
else{
output_high(LEDVerde1);
output_high(LEDVerde2);
output_high(LEDRojo1);
output_high(LEDRojo2);
duplicado=0;
return;
}

sprintf(digito,"%f",vel);
//dig1=(int)(vel/10);
//dig2=(int)(vel%10);

switch(digito[0])
{
case '0':
output_b(0b01111111);
break;
case '1':
output_b(0b00000110);
break;
case '2':
output_b(0b01011011);
break;
case '3':
output_b(0b01001111);
break;
case '4':
output_b(0b01100110);
break;
case '5':
output_b(0b01101101);
break;
case '6':
output_b(0b01111101);
break;
case '7':
output_b(0b00000111);
break;
case '8':
output_b(0b01111111);
break;
case '9':
output_b(0b01101111);
break;
}

switch(digito[1])
{
case '0':
output_d(0b01111111);
break;
case '1':
output_d(0b00000110);
break;
case '2':
output_d(0b01011011);
break;
case '3':
output_d(0b01001111);
break;
case '4':
output_d(0b01100110);
break;
case '5':
output_d(0b01101101);
break;
case '6':
output_d(0b01111101);
break;
case '7':
output_d(0b00000111);
break;
case '8':
output_d(0b01111111);
break;
case '9':
output_d(0b01101111);
break;
}



}

VAMOS RIVER Carajo!!

[temtronic]

Take your program and cut in half.
1st half the 'tachometer' section
2nd half the 'display results' section

For test purposes I'd do the 1st half and send the 'result' vai RS232 to a PC terminal program. This way you can easily confirm the correct operation of the 'tachometer' section.

Next, I'd just code the 'display section' and use 'dummy' variables to confirm the 'display section' is working correctly. Maybe have a 'counter' variable go from 0 to 9999 every 1/2 second.Slow enough to see, fast enough to confirm all segments of all digits are correct.


Once both of these 'sections' are working, 'merge' the codes and it _should_ work....

Jay

[ezflyr]

Hi,

First, learn to to use the 'Code' button when showing us your code. Without this, it's tedious to try to study your posted code......

You said this:

[asmboy]

IF you make digito[] actually be an INT8 of the character in question: ie: conform to absolute values 0-9 there is a good shortcut BUT
passing over a '0' to '9' will NOT work in any iteration of your code without parsing through atoi() to make it so.
you never read anything from pin_C2 either.

and of course you never SHOW digito[] receiving any display value anyway, so i have no clue what you expect it to do in the MESS you posted.

i can't help you unless you get better at showing the FULL code in a way we can read.........
B U T
this consolidates a lot of your code for output pattern IF digito contains binary values 0-9 ..

[sebalitter]

The only problem in the code is the count. It is not counting correctly.
The display shows the numbers correctly.
Now the code is updated with the code that Asmboy gave me.

Im trying to use the RS232 to send the result ( vel ) to the hyperterminal as temtronic proposed.
Can i send the result by pickit2 or the only way is to connect a MAX232 converter? i didn't find a good example.

Also I used the debugger of MPLABX with pickit2 and found that the program is not doing the CCP1_interrupt() so its not counting.
What im doing wrong with the ccp?

Thanks