Hi, I do not speak English very well, sorry :(.

My program in C:

----------------------------------------------------------------------------------------------

#include <m8c.h>        

#include "PSoCAPI.h"    

#include "Stdlib.h"

#include "m8c.h"

#include "math.h"

int cadena;

float datos;

int t;

int menu;

int aux;

void main(void)

{

    M8C_EnableGInt;

PGA_Start(PGA_HIGHPOWER);

ADCINC_Start(ADCINC_HIGHPOWER);

ADCINC_GetSamples(0);

Character_LCD_Start();               

while(1)

    //infinite loop.

{

if(menu==3){

menu=0;

}

switch(menu){

case 0:

if (ADCINC_fIsDataAvailable() != 0){    

if ((ADCINC_bGetData() <= 255)&&(ADCINC_bGetData() >0)){

AMUX4_1_InputSelect(AMUX4_1_PORT0_7); 

                datos=(float)ADCINC_wGetData()*5/255;

a=datos*100;

datos=floor(a);

datos=datos/100;

Character_LCD_Init();                           

for(t=0;t<50;t++){

Character_LCD_Position(1,2);               

Character_LCD_PrString(ftoa(datos,&cadena));

Character_LCD_Position(1,6);

Character_LCD_PrCString("      ");

Character_LCD_Position(1,12);

Character_LCD_PrCString("V");

Character_LCD_Position(0,1);

Character_LCD_PrCString("Selector 1:");

}

ADCINC_fClearFlag();                            

}

}

break;

case 1:

for(t=0;t<50;t++){

Character_LCD_Position(0,1);

Character_LCD_PrCString("Temperatura");

Character_LCD_Position(1,1);

Character_LCD_PrCString("Temperatura de referencia:");

}

ADCINC_fClearFlag();

break;

case 2:

for(t=0;t<50;t++){

Character_LCD_Position(0,1);

Character_LCD_PrCString("Configuracion:");

}

ADCINC_fClearFlag(); 

break;

}

aux=PRT0DR&&0x01;

if(aux!=0&&x==0){

menu=menu++;

x=1;

}else if(aux==0&&x==1){

x=0;

}

if(menu==3)menu=0;

}

}

 Ah and the port 0[0] is configurated:

Globalinputeven, High Z and interruptions disabled.

Hi Clemen,

do not rely on a variable beeing initialized when you do not initialize it yourself.
You rely on menu equal to 0 (zero).

And it is better programmer's style to write

if (menu >= 3) menu = 0;

which in fact may help against some errors.

Have fun

Bob 

... and I miss the #pragma ioport declaration (have a look into the c-language manual)

Bob

... and another one (too much code to understand quickly)
It looks like you are using a pushbutton on port0[0]. so what is the port like, when the button is not pushed? Floating??

I use very often resistive output high, write a one to it and then read and debounce the pushbutton.

Bob

Yes, you have to have debounce of the key first.

Here is the modification I do

  aux=PRT0DR&&0x01;

  if(aux == 1)
  {
   /* only do this if x is 0*/
   if (x == 0)
   {
 
    /* add a delay here */
    /* Here is a quick but ugly way to do it */
    /* Add declare of i at the beginning of the code */
    /* you may need to experiment with the delay_count number */
    
    for(i = 0; i <= DELAY_COUNT; i ++);
   
    aux = PRT0DR && 0x01;
    
    /* only if it is still pressed */
    if(aux == 1)
    {
    
     if (++menu >= 3)
     {
      menu = 0;
     }
 
     x = 1;
    }
   }
  }
  else
  {
   x = 0;
  }

Rather than using a crude for loop for debounce delay,since youre already using an LCD,use the delay functions that it provides.

use LCD_Delay50uTimes(BYTE bTimes); where bTimes is the number of times to delay 50 μSec.

This is a clock independent delay.
 

PS:Hit 'Verify Answer' if my reply helped you.

yes,

it would be good to reuse the existing delay function.

Hi again!

Thank you very much to all!

The psoc is in the lab, on wednesday i will go to the lab and test all :)