>    movlw 0x07           ;assuming all digital
>    movwf cmcon

Sorry, that should be (still in bank1)

         movlw   b'00000111'  ;set Port I/O configuration
         movwf   adcon1

         bank0
         clrf         adcon0          ;or at least clr bit0  (ADON)

Deeply ensconced in F628s at the moment and cmcon is a
similar PortA register

--
http://www.piclist.com hint: The list server can filter out subtopics
(like ads or off topics) for you. See http://www.piclist.com/#topics