i found the problem
i had to set bits 6:4 in ADCON1 to match the setting in the
Define ADC_CLOCK
after that it is working well now

so what about ANSEL ?
does ADCIN handle that automatically ?
it seems to work the same whether or not i explicitly set the ANSEL bits