Math question relating to AD input on PIC12F675

[rfetech]

Hello all,

This is my first PIC code in several years. Even back then, I only wrote a few simple programs and that will be obvious when you read my question.

I think that I'm OK with all the circuitry, and the main code isn't yet finished, but I would like for someone to help guide me with the math.

Let's assume that I have all the configurations correct.

I'll have a pot connected to AN2, which is configured for 8 bits. This should give me a value of 1 - 255 at AN2. I want to convert that range of numbers (1 - 255) into seconds and then use the new value in a variable for the PAUSE command. The code, I believe, should give me a range of 1 - 63 seconds. I just don't know how the decimal places will be handled and if the PAUSE command accepts non-whole numbers.

Thanks and regards,
Mitch

delayloop:
     ADCIN 2, adval                     ' read in value at AN2.
     IF adval <4 THEN                   ' make lowest value to be 4 which would equate
          adval = 4                     ' to a 1 second pause.
          delay = adval /4 * 1000       ' convert AD value of 1 - 255 into 1 - 63 second pause.
     ENDIF
     PAUSE delay                        ' pause from 1 - 63 seconds.

[LinkMTech]

Welcome Mitch,
I don't think you have to worry about the division remainders since it will be "rounded down".
But your code needs a little change to work as you want otherwise it won't give you a variable "Pause". So assuming this is just a snippet:

delayloop:
     ADCIN 2, adval                     ' read in value at AN2.
     IF adval <4 THEN  adval = 4    ' make lowest value to be 4 which would equate to a 1 second pause.
     delay = (adval /4) * 1000         ' convert AD value of 1 - 255 into 1 - 63 second pause.
     
     PAUSE delay                        ' pause from 1 - 63 seconds.

[aratti]

delay = adval * 250 will give a better linerity, unless you prefer increments of one second.

Al.

[rfetech]

Thanks Louie and Al.

Louie: Yes, I do need to use parenthesis. That was an oversight. I hope you are correct about the division remainders. That is one of my concerns.

Al: It's not critical, either way, to my application but what you suggest makes good sense. I will probably test it each way.

I've tentatively completed the code for my project if either of you, or anyone else, would care to critique it for me. I could really use your more experienced opinions. I hope to breadboard it tomorrow.

Thanks again,
Mitch

'****************************************************************
'*  Name    : Train_Autoreverser.pbp                            *
'*  Author  :                                                   *
'*  Notice  : Copyright (c) 2010                                *
'*          : All Rights Reserved                               *
'*  Date    : 10/27/2010                                        *
'*  Version : 1.0                                               *
'*  Notes   : Model train autoreversing circuit using           *
'*          : a PIC12F675 and reed switch sensors.              *
'****************************************************************

'   ---------Configuration--------

' PICBasic Pro v2.6a default configuration for PIC12F675
'   __config _INTRC_OSC_NOCLKOUT & _WDT_ON & _MCLRE_ON & _CP_OFF


'   ---------Constants/Defines-------

DEFINE OSCCAL_1K 1            ' Calibrate internal oscillator

DEFINE ADC_BITS 8             ' Set number of bits in result
DEFINE ADC_CLOCK 3            ' Set clock source (3=rc)
DEFINE ADC_SAMPLEUS 50        ' Set sampling time in uS

SW1 CON 0                     ' Reed switch 1 to GPIO.0
SW2 CON 1                     ' Reed switch 2 to GPIO.1
RY1 CON 4                     ' Relay 1 (Power) to GPIO.4
RY2 CON 5                     ' Relay 2 (Voltage reversing) to GPIO.5


'   ---------Variables---------

adval VAR BYTE                ' Store result of each A/D conversion.
delay VAR BYTE                ' Store result of math conversion to seconds.


'   ---------Initialization--------

ANSEL = %00000100             ' Set AN2 analog, all others digital.
CMCON = 7                     ' Turn off analog comparators.


'   ---------Main Code---------

PAUSE 2000                    ' Let everything stabalize for 2 seconds.
LOW RY2                       ' Make sure RY2 is off for forward movement.
                            
mainloop:
HIGH RY1                      ' Turn on Power relay to start forward movement.

  IF SW1 = 0 THEN             ' Keep moving in the forward direction until
    LOW RY1                   ' SW1 reed switch is closed. Then turn off 
  ELSE                        ' RY1 power relay to stop movement.
    HIGH RY1
  ENDIF
    
GOTO delayloop                ' Keep stopped for x number of seconds.(1-63)
    
HIGH RY2                      ' Turn on reversing relay for reverse movement.
HIGH RY1                      ' Turn on power relay to start reverse movement.
    
  IF SW2 = 0 THEN             ' Keep moving in the reverse direction until
    LOW RY1                   ' SW2 reed switch is closed. Then turn off
  ELSE                        ' RY1 power relay to stop movement.
    HIGH RY1
  ENDIF

LOW RY2                       ' Get RY2 ready for next forward movement.
    
GOTO delayloop                ' Keep stopped for x number of seconds.(1-63)
  
GOTO mainloop                 ' Keep repeating until user terminated.
    
END
  
    
delayloop:
ADCIN 2, adval                ' Read A/D channel 2 into adval variable.
  IF adval <4 THEN            ' Make 1 second the shortest time delay.
    adval = 4                 
    delay = (adval /4) * 1000 ' Divide adval value (1-255) by 4 and multiply
  ENDIF                       ' by 1000 to get millisecond value for PAUSE
PAUSE delay                   ' command. Place this value in delay variable.

[rfetech]

Never mind looking over my code as I have found gross errors and will be attempting to correct them will I get time over the next couple days. I will re-post when I think that I have the problems resolved.

Thanks,
Mitch

[rfetech]

OK.... that was a long night!!

I've finished the code and tested the circuit on a breadboard. After getting my head on straight, I was able to revise the main code quite easily, even for me!!

The headache and long hours occurred when I just couldn't get the A/D working correctly. I spent hours going over the data sheet and dozens of forum messages, tried different combination's of Defines and register settings but never got it working... UNTIL... I saw that the PAUSE statement was expecting a 16 bit argument. At least that is the way that I understand it. So, based upon that, I changed my code from being setup for 8 bit resolution to 10 bit and everything purred like a kitten. Got to bed at 6am with a smile on my face!!

Bruce, if you happen to read this, I strongly believe that your proposed book would have saved me many hours of head-scratching by translating the data sheet into easier to understand terminology. So, don't give up on it.

Thanks to all who post on this forum for giving me the use of your knowledge and helping us all learn from your own trials and your successes.

My extremely complex, yet simple code is available for scrutiny.

p.s. Special thanks to Louie and Al for responding to my original post and helping me much more than they probably realize. THANKS!!!

'****************************************************************
'*  Name    : Train_Auto-Reverser.pbp                           *
'*  Author  :                                                   *
'*  Notice  : Copyright (c) 2010                                *
'*          : All Rights Reserved                               *
'*  Date    : 10/27/2010                                        *
'*  Version : 1.0                                               *
'*  Notes   : Model train auto-reversing circuit using          *
'*          : a PIC12F675 and reed switch sensors.              *
'****************************************************************


'   ---------Configuration--------

' PICBasic Pro v2.6a default configuration for PIC12F675
'   __config _INTRC_OSC_NOCLKOUT & _WDT_ON & _MCLRE_ON & _CP_OFF
' Changed MCLR function to Input Pin during programming of PIC.


'   ---------Defines/Constants-------

DEFINE OSCCAL_1K 1              ' Calibrate internal oscillator

DEFINE ADC_BITS 10              ' Set number of bits in A/D result
DEFINE ADC_CLOCK 3              ' Set clock source (3=rc)
DEFINE ADC_SAMPLEUS 50          ' Set sampling time in uS

RY1 CON 4                       ' Relay 1 (Power) to GPIO.4
RY2 CON 5                       ' Relay 2 (Voltage reversing) to GPIO.5


'   ---------Variables---------

SW1 VAR GPIO.0                  ' Reed switch 1 to GPIO.0
SW2 VAR GPIO.1                  ' Reed switch 2 to GPIO.1
adval VAR WORD                  ' Store result of each A/D conversion
delay VAR WORD                  ' Store result of math conversion


'   ---------Initialization--------

ANSEL = %00000100               ' Set AN2 analog, all others digital
CMCON = 7                       ' Turn off analog comparators
ADCON0.7 = 1                    ' Right justify A/D conversion output


'   ---------Main Code---------

PAUSE 2000                      ' Let everything stabalize for 2 seconds.
LOW RY2                         ' Make sure RY2 is off for forward movement.

                            
mainloop:

HIGH RY1                        ' Turn on Power relay to start forward movement.

  WHILE SW1 = 1                 ' Keep RY1 active until reed switch SW1
    HIGH RY1                    ' is closed.
  WEND
                      
LOW RY1                         ' Removes power and stops movement.

GOSUB delayloop                 ' Keep stopped for x number of seconds.(0-63)
    
HIGH RY2                        ' Prepare RY2 for reverse movement.
PAUSE 500                       ' Let RY2 settle before activate RY1.
HIGH RY1                        ' Turn on power relay to start reverse movement.
    
  WHILE SW2 = 1                 ' Keep RY1 active until reed switch SW2
    HIGH RY1                    ' is closed.
  WEND

LOW RY1                         ' Removes power and stops movement.
LOW RY2                         ' Prepare RY2 for next forward movement.
    
GOSUB delayloop                 ' Keep stopped for x number of seconds.(0-63)
  
GOTO mainloop                   ' Keep repeating until user terminated.
    
END
  
    
delayloop:
ADCIN 2, adval                  ' Read A/D channel 2 into adval variable.        
delay = adval * 60              ' Creates values for PAUSE command that                     
PAUSE delay                     ' allow delays from approx. 0 to 61 seconds.
RETURN