CCS C Software and Maintenance Offers
FAQFAQ   FAQForum Help   FAQOfficial CCS Support   SearchSearch  RegisterRegister 

ProfileProfile   Log in to check your private messagesLog in to check your private messages   Log inLog in 

CCS does not monitor this forum on a regular basis.

Please do not post bug reports on this forum. Send them to support@ccsinfo.com

Restriction of "delay_us" when using return value

 
Post new topic   Reply to topic    CCS Forum Index -> General CCS C Discussion
View previous topic :: View next topic  
Author Message
SchimmelSchumi



Joined: 05 May 2018
Posts: 3

View user's profile Send private message

Restriction of "delay_us" when using return value
PostPosted: Sat May 05, 2018 1:41 pm     Reply with quote

HI guys,


I came across a very challenging problem on my current project.

I am building some sort of frequency generator with the PIC12F863 utilizing the "delay_us" integrated function.

As per the functional description, 'delay_us' can be used with variables up to int16.

However, if I read an analog input (10 bit), do some easy math and put it into the 'delay_us' function as parameter, it seems not to respond to any changes on the analogue input.

I get the frequency pattern I am looking for, but I cant change the frequency with the help of the ADC.

Are they any restrictions on the "delay_us" function I am not aware of?

Here a sneak into my code.


Code:


.......



   setup_adc_ports (sAN0|VSS_VDD);
   setup_adc(ADC_CLOCK_INTERNAL);
   setup_oscillator(OSC_8MHZ);

int16 Frequency, HalfDelay;

.....

set_adc_channel (sAN0);
Frequency = read_adc (ADC_START_AND_READ);
HalfDelay=Frequency*5;
HalfDelay=HalfDelay+300;

   FOR (i = 0;i < 24; i++)
   
   {
      output_bit(PIN5, 0);
      delay_us(HalfDelay);
      output_bit(PIN5, 1);
      delay_us(HalfDelay);
   }
   
   delay_us(HalfDelay);
   delay_us(HalfDelay);
   
      FOR (i = 0;i < 35; i++)
   
   {
      output_bit(PIN5, 0);
      delay_us(HalfDelay);
      output_bit(PIN5, 1);
      delay_us(HalfDelay);
   }

.....




Many thanks in advance for any input.

Brdgs, Mike.
PCM programmer



Joined: 06 Sep 2003
Posts: 21708

View user's profile Send private message

PostPosted: Sat May 05, 2018 4:40 pm     Reply with quote

You need to see what the ADC values are. If you had a larger PIC, you
could easily add printf capability. You can use the debugger to see the
values. Set a breakpoint and put a watch window on the variables.

Also, what is "PIN5" ? That's not a CCS pin number, and not the official
name of any pin.


There should be a delay between these two lines. In the 12F683 data sheet
it's listed as about 5 usec. Add a
Quote:
set_adc_channel (sAN0);
Frequency = read_adc (ADC_START_AND_READ);



Regarding restrictions or bugs for delay_us(), we need to know your CCS
version number. It's a 4-digit number given at the top of your .LST file,
which will be in your project directory after a successful compilation.
SchimmelSchumi



Joined: 05 May 2018
Posts: 3

View user's profile Send private message

PostPosted: Sat May 05, 2018 4:45 pm     Reply with quote

Thank you for your reply.

I wasn't aware of the delay. I will try it out.

I have a bunch of bigger PICs laying around, will also try other chips.

I am using CCS PCM C Compiler, Version 5.008, 5967

The PIN5 I made up as my real pin definition was a bit "strange".

I will let you know if this solves the problem.
temtronic



Joined: 01 Jul 2010
Posts: 9081
Location: Greensville,Ontario

View user's profile Send private message

PostPosted: Sat May 05, 2018 5:54 pm     Reply with quote

this
setup_adc(ADC_CLOCK_INTERNAL);

is usually NOT correct for 99% of the PICs. Some say the 'wizard' tosses it in well the 'wizard' should read the datasheet !
PIC12F863 ? hmm should that be 12f683 ??
If so, the ADC section says '..for clocks >1MHz, adc_internal for sleep mode only...' or words to that effect.

Jay
SchimmelSchumi



Joined: 05 May 2018
Posts: 3

View user's profile Send private message

PostPosted: Sat May 05, 2018 5:59 pm     Reply with quote

Hi Jay,

thanks for your comment.

Even if I use the internal oscillator? I always believed I have to use

Code:
setup_adc(ADC_CLOCK_INTERNAL);


When using the internal osciliator of the PIC.

Using the PIC12F683, correct. Smile
temtronic



Joined: 01 Jul 2010
Posts: 9081
Location: Greensville,Ontario

View user's profile Send private message

PostPosted: Sat May 05, 2018 6:07 pm     Reply with quote

yes ! It's in table 9-1, there's two notes (1,4) . #4 says if clock >1MHz ONLY use internal ADC clock while in sleep mode. I was going to copy/paste the PDF page but silly Acrobat version I have won't do that.
I just checked again as I'm using the 12F683 for a project and the PDF is on my screen ! Now you may get 'lucky' and have it work, say in the lab, buit in the real World where it gets HOT or k-k-k-kold, the ADC may given wrong or random results.
I never use the 'wizard' or any 'simulator', so I can only blame myself for any mistakes or typos !
Jay
Ttelmah



Joined: 11 Mar 2010
Posts: 19195

View user's profile Send private message

PostPosted: Sun May 06, 2018 12:30 am     Reply with quote

For your chip using the internal oscillator at 8MHz, the recommended ADC settings are Fosc/16 or Fosc/32. These are:
ADC_CLOCK_DIV_16 or ADC_CLOCK_DIV_32

The ADC_CLOCK_INTERNAL uses the separate RC oscillator for the ADC Except for 'modern' PIC 24's and similar chips, this is never recommended, unless you actually stop the CPU to do the conversion. It has a tendency to give 'beat' signals as it's frequency is not sychronised to the master oscillator. Result poor accuracy....

Now on your delays. There are some other problems:

Code:

set_adc_channel (sAN0);
Frequency = read_adc (ADC_START_AND_READ);

Wrong. The ADC needs at least Tacq between the channel being selected, and the value being read. About 5uSec assuming your source has an impedance below 10KR.

Then you don't show the code ever going back to read the ADC?. 24 cycles based on the single reading, then another 35 cycles still based on this reading. Are you actually ever going to see the value change?.

There are limitations with a variable. There is a minimum delay of perhaps 30 cycles using a variable, but your +300 should avoid this, and some granularity, but you should see the value change fairly well.
Display posts from previous:   
Post new topic   Reply to topic    CCS Forum Index -> General CCS C Discussion All times are GMT - 6 Hours
Page 1 of 1

 
Jump to:  
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum


Powered by phpBB © 2001, 2005 phpBB Group