Watching for pin High

[GatorGuy]

I am having trouble getting a program to run If statements when a pin goes high.

I have a 16f628a for testing using three pins for outputs and four for inputs. Pins PortA.0 - PortA.3 are set low at the first of the program using:

Low PortA.0 ect.

The PCB has terminals when shorted send +5v to the inputs pulling them high.

The main loop is such:

:main
If Porta.0 = HIGH then goto example1
goto main

When I pull the pin high, it doesn't do anything different. The rest of the pic and program is running fine sending out data when appropriate.

[cncmachineguy]

Are you actually comparing porta.0 to "HIGH"? If so, try changing high to 1. High is a reserved word. You could also just say:
If porta.0 then example1

This will be true when the pin is high, and false when it is low.

[HankMcSpank]

not familiar with that pic, but RA0 is shared with an analogue comparator, and the pin defaults to analogue so probably worth slapping in an...

CMCON = 7

to turn the comparator module off. (http://www.svetelektro.com/upload/16F628A.pdf page 64 upper right box)

[HenrikOlsson]

What's been said and if you first do LOW PortA.0 the TRIS bit will be cleared so that the pin (RA0 in this case) is an output. When you then do If PortA.0 = 1 you aren't reading the state of the pin since it's not set to be an input.

[Heckler]

Just to expand on what others have posted...
Below is a schematic (sorry about the ugly resistors) that shows two different ways of wiring to detect a change on a port pin.

First... with PortA.0 pulled high and a press on the button will cause a LOW or 0.

Second... with PortA.1 pulled LOW and a press on the button will cause a HIGH or 1.

The resistors can be 1K to 10K...




It makes no difference and is not necessary to pre-set the state of the pin you are trying to test by "make port.pin LOW or HIGH" as that is an OUTPUT function not an INPUT function.

The correct way is to choose one of the wiring options as illistrated, set the corresponding TRIS BIT for that port.pin to an INPUT or "1" and then test for the opposite state, which will indicate a change or button press.

TrisA= 000011   'set PortA.0 and PortA.1 as input

IF PortA.0=0 THEN 'do something here when PortA.0 changes

IF PortA.1=1 THEN 'do something here when PortA.1 changes

as shown below... it is also correct to test for a "0" by testing for "NOT" and it is also correct to test for a "1" as shown...

IF NOT PortA.0 then '(this code tests for a "0" or "LOW")

IF PortA.1 THEN ' (this code tests for a "1" or "HIGH"

I hope this helps...

PS... if you can not see the schematic image above, click on it.

[GatorGuy]

Thank you for the help. So far it is still not working(I have not tried everything suggested though). I will try to add the pull down resistors as well. I already have the working PCB but I should be able to drill and add them with some creative soldering.

I didn't really intend this board to be an input board but figured I would use it for testing while I was ironing everything (output) else out.