problema con push button

[Superboy]

Hola buena tarde, estoy probando comunicacion serial con un push button y una lcd, por medio del pic maestro yo presiono un push button y este envia un caracter en concreto '1' y el pic esclavo debe aumentar un variable contador de 1 en 1, el problema que me surge es que al oprimir el boton el contador aumenta drasticamente su valor sin detenerse, no aumenta de 1 en 1 conforme yo oprimo el botón solo pulso una vez y de ahi aumenta y sin detenerse en algun valor.
Alguien que me pueda decir, ¿que es lo que debo hacer para que aumente el contador cada que yo pulse el boton?
Trabajo con pic 16f887 y pic c compiler.

Código: C++

1. ///////////// esclavo ///////////////
2.  
3. #INCLUDE <16F887.H>
4. #FUSES INTRC_IO,NOWDT,NOMCLR,NOLVP
5. #USE DELAY(INTERNAL=4M)
6.  
7. #USE RS232 (baud=57600,rcv=pin_c7,xmit=pin_c6,bits=8,parity=N)
8.  
9. #use fast_io(D)
10.  
11. #priority rda
12.  
13. #define LCD_RS_PIN      PIN_D0
14. #define LCD_RW_PIN      PIN_D1
15. #define LCD_ENABLE_PIN  PIN_D2
16. #define LCD_DATA4       PIN_D3
17. #define LCD_DATA5       PIN_D4
18. #define LCD_DATA6       PIN_D5
19. #define LCD_DATA7       PIN_D6
20.  
21. #include <lcd.c>
22.  
23.  
24. int8 conteo=0;
25. int k;
26.  
27. #int_rda
28. void serial_isr(void)
29. {
30.  
31.     K=getc();
32.  
33. }
34.  
35. void main()
36. {
37.      
38.    disable_interrupts(global);
39.    
40.    setup_comparator(NC_NC_NC_NC);
41.    
42.    set_tris_D(0b0);
43.  
44.    lcd_init();  
45.    
46.    lcd_gotoxy(4,1);
47.    lcd_putc("Proyecto");
48.    
49.    lcd_gotoxy(6,2);
50.    lcd_putc("VEOX");
51.    
52.    delay_ms(2000);
53.    
54.    lcd_putc("\f"); ///// limpia la LCD
55.    
56.    enable_interrupts(int_rda);
57.    enable_interrupts(global);
58.    
59.    while(TRUE)
60.    {
61.      
62.          lcd_gotoxy(1,1);
63.          printf(lcd_putc,"Pasos=");//imprimir entero
64.          lcd_gotoxy(7,1);
65.          printf(lcd_putc,"%u",conteo);//imprimir entero
66.           delay_ms(5);
67.    
68.          if(k=='1')
69.             {
70.               conteo++;
71.             }        
72.                  
73.    }
74.  
75.  
76. }

 

Código: C++

1. ////////////// maestro ///////////////////
2.  
3. #INCLUDE <16F887.H>
4. #FUSES INTRC_IO,NOWDT,NOMCLR,NOLVP
5. #USE DELAY(INTERNAL=4M)
6.  
7. #USE RS232 (baud=57600,rcv=pin_c7,xmit=pin_c6,bits=8,parity=N)
8.  
9. #use fast_io(A)
10. #use fast_io(D)
11.  
12.  
13. #define LCD_RS_PIN      PIN_D0
14. #define LCD_RW_PIN      PIN_D1
15. #define LCD_ENABLE_PIN  PIN_D2
16. #define LCD_DATA4       PIN_D3
17. #define LCD_DATA5       PIN_D4
18. #define LCD_DATA6       PIN_D5
19. #define LCD_DATA7       PIN_D6
20.  
21. #include <lcd.c>
22.  
23. #DEFINE B1  PIN_A0
24.  
25. void main()
26. {
27.      
28.    disable_interrupts(global);
29.    
30.    setup_comparator(NC_NC_NC_NC);
31.    
32.    set_tris_D(0b0);
33.    set_tris_A(0b1);
34.    
35.    lcd_init();  
36.    
37.    lcd_gotoxy(4,1);
38.    lcd_putc("Proyecto");
39.    
40.    lcd_gotoxy(6,2);
41.    lcd_putc("VEOX");
42.    
43.    delay_ms(2000);
44.    
45.    lcd_putc("\f"); // limpia la LCD
46.    
47.    while(TRUE)
48.    {
49.    
50.          if(input(B1)==1)
51.             {
52.               PUTC('1');
53.             }
54.        
55.                  
56.    }
57.  
58.  
59. }

Muchas gracias.

[KILLERJC]

Esclavo/Receptor:

C es case sensitive, asi que K mayuscula (en la interrupcion) no es lo mismo que k minuscula.

El otro problema es que una ves recibido un '1', k mantiene ese valor hasta que no llegue otra cosa... entonces siempre va a aumentar. Dentro del k == '1' pone k igual a 0, asi no hay problemas. Y por otro lado declara a k como volatile.

Maestro/Transmisor:

Deberias tener una especie de bloquo para que no envie muchas veces el '1', asi se envia una sola ves.

Código: C

1. if(input(B1)==1)
2.             {
3.               PUTC('1');
4.               delay_ms(10);   // Pequeño antirebote
5.               while(input(B1)==1); //Espera aca hasta qeu se suelte el boton.
6.             }

[Robert76]

Hola, yo utilizo ésta subrutina para detectar sólo una vez una pulsación, también funciona cómo antirrebotes.
Ejem:

if(!boton)          'detecta botón pulsado
  {
    if(Var<250)Var++;
    if(Var==200)"realiza la acción deseada"
  }
else Var=0;

[Superboy]

Esclavo/Receptor:

C es case sensitive, asi que K mayuscula (en la interrupcion) no es lo mismo que k minuscula.

El otro problema es que una ves recibido un '1', k mantiene ese valor hasta que no llegue otra cosa... entonces siempre va a aumentar. Dentro del k == '1' pone k igual a 0, asi no hay problemas. Y por otro lado declara a k como volatile.

Maestro/Transmisor:

Deberias tener una especie de bloquo para que no envie muchas veces el '1', asi se envia una sola ves.

Código: C

1. if(input(B1)==1)
2.             {
3.               PUTC('1');
4.               delay_ms(10);   // Pequeño antirebote
5.               while(input(B1)==1); //Espera aca hasta qeu se suelte el boton.
6.             }

muchas gracias por las respuestas solo tengo duda en como pongo eso de k como volatile osea asi lo escribo volatile k; ¿?

[Superboy]

Hola, yo utilizo ésta subrutina para detectar sólo una vez una pulsación, también funciona cómo antirrebotes.
Ejem:

if(!boton)          'detecta botón pulsado
  {
    if(Var<250)Var++;
    if(Var==200)"realiza la acción deseada"
  }
else Var=0;

+

gracias por tu respuesta, tengo duda sobre si eso lo usas con el adc o ¿como? o ¿por que el valor de 250?

[Robert76]

Ésta subrutina no tiene nada que ver con el ADC.
El valor 250 lo elegí para usar una simple variable int. Porque el valor máximo es 255.
Puedes usar un valor menor si quieres, todo depende de la velocidad cíclica del programa. Ahora, si reduces el número 250, también deberás hacerlo en el 200. Pues a un valor menor, mayor velocidad de respuesta de detección de botón pulsado.