Try not using your ADCON statements. I've never used them & always worked for me. Also try the internal oscillator @ 4MHZ. Your program looks like it should work.