A/D conversion with PIC18F67J50

[mister_e]

Side note... there's no real need to redefine ANCON1, ANCON0 as i did if you use ANCON1_ALT, ANCON0_ALT within your code. Those two are defined in PIC18EXT.BAS.

Having _ALT written at the right of the register name, may remember you to set/clear WDTCON.4 bit? Maybe

[ScaleRobotics]

Side note... there's no real need to redefine ANCON1, ANCON0 as i did if you use ANCON1_ALT, ANCON0_ALT within your code. Those two are defined in PIC18EXT.BAS.

Having _ALT written at the right of the register name, may remember you to set/clear WDTCON.4 bit? Maybe

Sweet! I did not see that either. May be time for glasses. And a lot cleaner than the way I was doing it, and I like the way you wrote the registry bits in the code. That would have kept me from having to divert my attention to and from tfm a lot!

Thanks for all the help.

Walter

[mister_e]

To me, comments are really important. So, on the long run... coding like that

        ADCON0 = %00000001
               '  0-------- VCFG1, Vref- 0 = AVss (1=AN2)
               '  -0------- VCFG2, Vref+ 0 = AVdd (1=AN3) 
               '  --0000--- CHS<3:0>, Channel select -- 0
               '  ------0-- Go/Done, don't care for now
               '  -------1- ADON, Enable ADC

May indeed avoid to open the datasheet all the time, one thing is sure, it's easier to modify/understand later.