Need help with 16f819 10 bit ADC

[modifyit]

Don't you need to change your command to ADCIN 4, adval to get AN4?

I personally have been using Melanies direct method reading analog values that can be seen in this thread

http://www.picbasic.co.uk/forum/show...ghlight=adcon0

Its pretty straight forward and I can understand what is going on.

[Ingvar]

Defines should be in uppercase.

DEFINE ADC_BITS 10
DEFINE ADC_CLOCK 3
DEFINE ADC_SAMPLEUS 50

[bannrikae]

Thanks modifyit, I'll try this when I get back to work on Monday and let you know how I get on.

Ingvar, I didn't think these were case sensitive, it worked OK lower case for 8 bit ADC but I will also try this on Monday.

Thanks both.

[Ingvar]

Your defines didn't do anything atall since they were the default values.

[bannrikae]

So if they are default values Then shouldn't the prog work?
The rest of the code works when set up for 8 Bits, can't see what else is wrong, confused!.

BG

[Ingvar]

It worked before you changed it to 10 bits. 8 bits is default. That's why it worked even though you wrote the defines in lowercase.

[bannrikae]

OK ingvar, see what your saying, will try as soon as I get the time,
(work gets in the way of projects)

Thanks

BG