Hi,

CMCON = 7 is correct for the 18F4620 but it's also the POR default so it's not REALLY needed.
ADCON1 = 15 is correct if all pins should be digital, if you set it to 0 all 12 inputs will be analog.
ADCON2 doesn't really matter since the ADC is off anyway.

I've never used Darrels AllDigital but remember that when you INCLUDE a file like that the content of the file will be put in at that very spot - just as if it was actually typed in right there. Since you include it first and THEN set CMCON etc manually YOU will overwrite whatever it is IT does to the relevant registers. Had YOU set the registers first and THEN included AllDigital IT would overwrite anything you've set manually - in the register it believes are important only of course.

/Henrik.