Hi,
ANSEL = 0 for sure.
CMCON = 7 why not, but it's the default POR value so really shouldn't be needed (as long as the datasheet isn't lying to us).
ADCON1 = 7 don't know how you came up with that value or how it would help (if it acually did). The four lower bits are unimplemented and does nothing so Writing 7 to that regiser does exactly that - nothing.
So, as far as I can see, clear ANSEL and clear TRIS bits for your outputs should do it.
/Henrik.
Bookmarks