rrc124+@PITT.EDU wrote: > Here is my source code... > Now the important part is where i set ADCON1... Probably not, since it looks OK... Is this a *complete* working example, btw ? > If i set it to all zeros (which in the datasheet says > all PORTA becomes ADC in)... No it doesn't, see port RA4. > Reed Relay Outputs There is no RA6 or RA7 ! > movf ADRESH, W > movwf PORTA PORTA is just 5 bits, and one of them is used as input. Perhaps better to use a full 8 bit port ? Now, not much help, maybe... Jan-Erik. PS. Besides, your code (and mail in a whole) is formatted in a way that made it hard to read the code and reply to. Make sure you keep your line lenghts below 70-75 chars, either by using some feature in your mail tool, or just simply by using your ENTER key at regular intervals... DS. -- http://www.piclist.com hint: The PICList is archived three different ways. See http://www.piclist.com/#archives for details.