PIC12F683 with VRef in ADC [SOLVED]

lcs · Joined: 10 Mar 2020 Posts: 6

Hi!
I'm trying to make a code to use the lm35 with pic12f683, using Vref (AN1) = 1.2V (with potentiometer). For testing, I'm just trying to light a led when the temperature exceeds 27C. What could be wrong?

PCM programmer · Joined: 06 Sep 2003 Posts: 21708

1. The Vref module in the 12F683 is for the comparator, not the ADC.

2. The ADC allows you to use an External Vref on a pin. You have to
generate it. If your Vdd is constant, then you could just two resistors or
a trimpot to make a voltage divider and get the external Vref from that
circuit.

3. You can't use a Vref of 1.2v with the ADC. The minimum is 2.2v,
and the preferred minimum is 2.7v. See this section in the data sheet:

Ttelmah · Joined: 11 Mar 2010 Posts: 19545

Start with setup_vref.

This is wrong. This is _not_ for the ADC VREF. This sets up the internal
comparator VREF. This module sets up the comparator Vref as a level
of the supply. If you look in the .h file for the processor, the setup_vref
'mode' settings are VREF_LOW and VREF_HIGH or'ed with a number 0-15.
If you look in the data sheet for the Cvref module. section 8.11, tells you that
the comparator voltage reference has two ranges each with 16 levels. The
VREF_HIGH and VREF_LOW setting gives control of the ranges and the value
0-15 control the levels. However this is completely and totally for the
comparator _not_ the ADC.
So start by getting rid of this setting....

Then the second problem. ADC VREF. If you again check the data sheet
and look in the A/D converter (ADC) Characteristics. you find AD06
'Reference voltage'. This has values of 2.2 and 2.7V 'minimum'.
The value of 2.2v, is the absolute minimum value that the ADC can
use for the Vref. 2.7v is the minimum it can use and retain 1bit
accuracy. So you can not use 1.2v....

I see PCM_programmer has pointed out both problems while I've been
typing..... Smile

As a general 'comment', most of the PIC ADC's require a minimum of
about 2.5v for the Vref.

temtronic · Posted: Wed Mar 11, 2020 4:33 am

along with the other comments....
this line...

setup_adc(ADC_CLOCK_INTERNAL);

is incorrect.
Have a look at table 9-1, ADC clock vc clock frequency...

you'll see that 'internal' is wrong for clk>4MHz.... you have 2 choices, /8 or /16.
I'll assume you used a 'Wizard' to come up with 'setup_adc(ADC_CLOCK_INTERNAL); ' ?
PIC rule #2... Never EVER trust Wizards.

lcs · Joined: 10 Mar 2020 Posts: 6

Thanks for the answers.
I made some changes to my code, but the led doesn't turn on:

PCM programmer · Joined: 06 Sep 2003 Posts: 21708

Ttelmah · Joined: 11 Mar 2010 Posts: 19545

A simple set of maths to give a reasonable answer with your Vref would
be:

temp=(r*30)/100;

For 1C, this will give '1'. For 10C, 10 etc..

So at 20C, 0.2v in. ADC should read 0.2/(3/1024)=68. 68*30=2040. 2040/100
= 20.

Understand the ADC reads 1023, just below it's reference voltage, so the
division needed to get the ADC step is /1024, not /1023.
Use a 2.5v Vref. Then the conversion becomes

temp=(r*25)/100

and the actual resolution will be slightly better.

Also if you need to read below about 2 to 3C, you do need the -ve
pull down resistor on the signal.

lcs · Joined: 10 Mar 2020 Posts: 6

In the last code of my answer, I directly tested the ADC value ... Apparently, read_adc() did not return a value greater than 0. Is the Adc module having problems?
When I get home, I will test the conversion constants that have passed on to me.

temtronic · Posted: Thu Mar 12, 2020 6:51 am

That could mean the LM34/35 sensor is defective or a shorted input to ADC pin.
In 3+ decades of using those LMs, I've never had a defective one.

Ttelmah · Joined: 11 Mar 2010 Posts: 19545

In your original code, there is a major issue, you are setting the Vref pin
as an output. No. It needs to be an input. Input, and 3v.
Honestly get rid of the #use fixed_IO, and just use output_high/ output_low
on PIN_A2 for your test. It should always be high with the code as posted,
since unless the ADC input is totally shorted to 0v, you will always see
a value above 0.
Honestly better to use

#use delay(INTERNAL=4MHz)

lcs · Joined: 10 Mar 2020 Posts: 6

The code finally worked! I thank everyone for their help and the time they spent helping me.
The changes:

PCM programmer · Joined: 06 Sep 2003 Posts: 21708

Your code still has several problems:

1. You forgot to set the adc channel. You got away with it because the
channel select register bits default to 0. But it should be set.

2. There should be a 5us delay after setting the channel. This in the
12F683 data sheet in this section:

temtronic · Posted: Sat Mar 14, 2020 4:27 am

I'd also delete this line....
set_tris_a(0b00000011);

and let the compiler handle the I/O configuration.

Unless you NEED very,very fast I/O just use 'standard I/O, the compiler default.
In 3 decades of programming PICs, I've only needed set_tris.. and fast_io... 2 or 3 times. In each case is was early and for a custom interface with propriatory I/O. Once the 20MHz barrier was broken, 'speed' hasn'y been a problem.

Also
try to get into the habit of adding comments at the end of each line. While code looks obvious, now, for you... 3 days,or months from now you or a friend will wonder WHAT is this suppose to do ??
Comments don't take up any codespace and as long as you can type are easy to add.

PCM programmer · Joined: 06 Sep 2003 Posts: 21708

I agree it's not necessary for his program to work, but one reason for
including that line is if he doesn't want floating inputs on the unused pins.

He is not setting the output level for each pin, but the GPIO register bits
default to 0 at power-up, except pin A3 (input-only) and pin A5.