Can't get switch to work

[JeffnDana]

Bruce:

Thanks for your help. Those changes solved it. Which line was responsible for keeping it from working?

CMCON = 7 ' all digital - comparators disabled

OR

The RETURN in the IF THEN block?

Thanks again for your help!

Jeff

[mackrackit]

Bruce:

Thanks for your help. Those changes solved it. Which line was responsible for keeping it from working?

CMCON = 7 ' all digital - comparators disabled

OR

The RETURN in the IF THEN block?

Thanks again for your help!

Jeff

That is why Skimask did not do a re-write for you, you did not learn or listen to him.

Blocks of code, GOSUBS/RETURNS,etc and CMCON.

Print you code and the code Bruce did, study side by side - line by line.

Your first try was not far off for a first timer pun intended

[Bruce]

Hi Jeff,

CMCON was the show stopper for the switch input, but GOSUB's without the
RETURN (as already mentioned) was another problem.

You might also want to include a conversion routine just after your GOSUB
Read_1307 to convert before displaying the results.

Something like this;

Temp var byte ' for conversion routine

Then just after GOSUB Read_1307 insert GOSUB Convert.

Here's Convert:

Convert:
    IF hour >=$10 THEN
       Temp = (hour>>4)&$0F
       Temp = (temp*10)+(hour&$0F)
       hour=temp
    ENDIF
    IF minute >=$10 THEN
       Temp = (minute>>4)&$0F
       Temp = (temp*10)+(minute&$0F)
       minute=temp
    ENDIF
    IF second >=$10 THEN
       Temp = (second>>4)&$0F
       Temp = (temp*10)+(second&$0F)
       second=temp
    ENDIF
    RETURN

I just happened to be working on a few routines for the SLED-C4, and figured
I would give you a boost.

And, yes, you should also get cozy with the 16F62xA datasheet so you'll
know what each I/O-pin can do & what you'll need to write to certain
registers to make things work right...;o}