Won't go back to SLEEP after 1st Interrupt


Closed Thread
Results 1 to 33 of 33

Hybrid View

  1. #1
    Join Date
    Jul 2003
    Location
    Colorado Springs
    Posts
    4,959


    Did you find this post helpful? Yes | No

    Default

    Quote Originally Posted by jellis00 View Post
    (Note: all PortAB pullups were already off, as suggested by Darrell, in my original code by virtue of the OPTION_REG = %00111000 setting in my Initialization code)
    Actually, when OPTION_REG.7 = 0, the pull-ups are ENABLED.
    A single pin with a 0V input can easily draw 250uA from the pull-up.

    Neither of your comments mentioned anything that appeared in my code to be preventing return to the SLEEP mode in the MAIN loop after the 1st interrupt service.
    That's what my ANSEL point was about.

    By turning ON/OFF the analog function on PORTA.3 (flush), you were inadvertently triggering PORTA change interrupts when no changes were made to the input state of the RA3 pin.
    That alone would cause it to lock up in the ISR.

    Perhaps there are more problems as well.
    For sure, there's an awful lot of stuff that doesn't need to be there.

    I will look further.
    <br>
    DT

  2. #2
    Join Date
    Jul 2003
    Location
    Colorado Springs
    Posts
    4,959


    Did you find this post helpful? Yes | No

    Default

    And if AN3 had been on RA3 ... That might have made sense.

    But it isn't
    On a 690, it's on RA4, so my ANSEL point was stupid.

    Still looking though.
    <br>
    DT

  3. #3
    Join Date
    Mar 2009
    Location
    Colorado
    Posts
    378


    Did you find this post helpful? Yes | No

    Default Here are the current pinout assignments for your info

    To hopefully assist you in identifying why my code doesn't put the MCU into sleep after the execution of the 1st interrupt service, I am including here the pinouts in my current configuration. These are the same pinouts I had for the powere consumption summary of 4 cases I previously posted.
    Code:
    '------------------------- 16F690 Port Assignments ----------------------------
    ' I/O pin connections to the PIC16F690 MCU are as follows:
    ' PORTA.0 (19) RA0 connected as ICSPDAT when used for ICSP & also to a low
    '              battery monitor R/C circuit for normal ops.
    ' PORTA.1 (18) RA1 connected as ICSPCLK when used for ICSP.
    ' PORTA.2 (17) RA2 connected to external Flush contact limit switch which  
    '              connects to ground when activated and serves as On-change Interup   
    '              During testing connects to a grounded momentary-on switch on the
    '              PICkit2 board to simulate external limit switch On-change Interup
    ' PORTA.3 (04) RA3 connected during testing to an external flow meter to detect  
    '              flow meter pulse inputs. 
    '              During testing it connects to a grounded momentary-on switch  
    '              on the PICkit2 board as a clock input to TMR0 for counting of  
    '              pulses and as an overflow interrupt.   
    ' PORTA.4 (03) RA4 connected to pin 6 of the ICSP header for use in programming. 
    ' PORTA.5 (02) RA5 not connected.  
    ' PORTB.0 (  ) RB0 not connected. 
    ' PORTB.1 (  ) RB1 not connected.  
    ' PORTB.2 (  ) RB2 not connected. 
    ' PORTB.3 (  ) RB3 not connected.
    ' PORTB.4 (13) RB4 connected via a 10 ohm resistor to power the Wireless option. 
    ' PORTB.5 (12) RB5 connected to Hall Effect sensor circuit #2. 
    ' PORTB.6 (08) RB6 connected to Hall Effect sensor circuit #1.
    ' PORTB.7 (09) RB7 connected to pin R/T SEL on the TRM-XXX-LT transceiver.      
    ' PORTC.0 (11) RC0 connected as an output to an LED & is on when the valve 
    '              is opened.
    ' PORTC.1 (12) RC1 connected as an output to an LED & is on when battery is low.
    ' PORTC.2 (13) RC2 connected as an output to relay RL1 that actuates the   
    '              + latching line of a solenoid to open the valve.  
    '              During program testing on the PICkit2 demo board it is connected 
    '              to an LED to show when the pulse is sent.
    ' PORTC.3 (14) RC3 connected as an output to relay RL2 that actuates the   
    '              - latching line of a solenoid to close the valve.
    '              During program testing on the PICkit2 demo baord it is connected
    '              to an LED to show when the pulse is sent. 
    ' PORTC.4 (15) RC4 not connected.
    ' PORTC.5 (16) RC5 connected to pin PDN on the TRM-XXX-LT transceiver. 
    ' PORTC.6 (17) RC6 connected to pin RSSI on the TRM-XXX-LT transceiver. 
    ' PORTC.7 (18) RB7 connected to pin R/T SEL on the TRM-XXX-LT transceiver.
    
    ' -----[ Device Declaration ]----------------------------------------------
    ' For MSASM assembler use following:
    @ __config _INTRC_OSC_NOCLKOUT & _BOR_SBODEN  & _WDT_OFF & _MCLRE_OFF & _CP_OFF

  4. #4
    Join Date
    Mar 2009
    Location
    Colorado
    Posts
    378


    Did you find this post helpful? Yes | No

    Default Descriptions for RA2 & RA3 reversed in previous post

    Sorry for having posted incorrect info in previous post, but I noticed that I had the descriptions reversed on RA2 and RA3. In the actual code you will see that I am using RA3 as the flush On-change interrupt and RA2 as the pulse count input to TMR0 overflow interrupt....just the reverse of the pinout descriptions.

  5. #5
    Join Date
    Mar 2009
    Location
    Colorado
    Posts
    378


    Did you find this post helpful? Yes | No

    Default May have found problem but don't know why??

    Darrel,
    I did some experimenting with TRISA to follow up on Joe's suggestion that didn't work when I tried it. However, I discovered that if I place the following TRISA statement ahead of the @ SLEEP in the MAIN loop
    TRISA = %00001101 ' Set RA0, RA2 & RA3 as inputs
    then it appears the MCU does go into sleep after the 1st interrupt with the same power as was shown at PowerUp....in this case, 350 microamps.
    Unfortunately I don't know why. I still have the following statement in the Int_Handlr that makes all PortA inputs during the ISR.
    TRISA = %11111111 ' Set all PORTA pins to inputs...RA0, RA2 & RA3 are used in the ISR
    Can you explain why this change to TRISA in the MAIN loop makes it work and without it, it won't??
    So that you and anyone else who ever referes to this thread can see the final solution, here is the code that appears to work. Power consumed during apparent SLEEP (350 microamps) is still higher than I would like, but possibly I am not seeing accurate values on my ampmeter???
    Code:
    ' -----[ Device Declaration ]----------------------------------------------
    ' For MSASM assembler use following:
    @ __config _INTRC_OSC_NOCLKOUT & _BOR_SBODEN  & _WDT_OFF & _MCLRE_OFF & _CP_OFF
    ' For PM assembler only use following instead
    '@device pic16F690, intrc_osc_noclkout, _bor_sboden, pwrt_off, wdt_off, mclr_off,
    ' protect_off
    ' The above config  is inteded to set CONFIG = %0000110111010100 for hex 0DD4 or
    ' FCMEN on (1), IESO on(1), BOREN (01), CPD off (1), CP off (1), /MCLR is  
    ' digital input (0), /PWRTE off (1), WDTE off (0), INTOSCIO (100).   
      
    ' -----[ Revision History ]------------------------------------------------
    ' Version 1.0.6 on 6/19/09: Replaced 2nd IF-THEN block with REPEAT-UNTIL to
    '                           flash leds correctly during flow and low battery
    ' Version 1.0.7 on 6/21/09: Eliminated @ NOP in MAIN; Reformated Port
    '                           Assignments list; Changed IntCloc from 4 to 1 MHz;
    '                           WDT_OFF and comparators off to save power.
    ' Version 1.0.8 on 6/22/09: Changes to MAIN to minimize power during Sleep:  
    '                           Set BOR_SBODEN; set PCON.4 =0; ANSEL =%11110011, 
    '                           TRISA = %00001000 in MAIN to reduce Sleep power.                                                                                                                                             
    
    ' -----[ Declare Variables & Aliases ]-----------------------------
    '
    diff    VAR Byte      ' Difference between bat_mon and Vthr
    dummy   VAR Byte      ' For read of on-interrupt port & clear mismatch condition
    flush   VAR PORTA.3   ' Set RA3 as input for sensing flush switch closure Int
    i       VAR Byte      ' Index used in Gallon counter loop
    led1    VAR PORTC.0   ' Set RC0 as LED indicator of valve open..water flowing
    led2    VAR PORTC.1   ' Set RC1 as LED indicator of low battery
    bat_mon VAR PortA.0   ' Set RA0 as battery low power monitor
    meter   VAR PORTA.2   ' Set RA2 as input for Hall Sensor meter pulse Interrupt
    open_valve    VAR PORTC.2   ' Set RC2 as valve solenoid open command
    close_valve   VAR PORTC.3   ' Set RC3 as valve solenoid close command
    
    ' -----[ Declare Constants ]----------------------------------------
    '
     k             CON 10   ' Calibration factor for flow meter...# pulses per gal
     Vthr          CON 3    ' Assumes Low Battery Monitor threshold = 3 volts
     
    ' -----[ Initialization ]-------------------------------------------  
    Init: 
    ' Setup Timer0 as an 8-bit counter with the clock input on RA2.
    ' 1:1 TMR0 prescaler
    ' TMR0 counts on high-to-low transitions
      OPTION_REG = %00111000   ' PORTA/B pullups enabled, Interrupt on falling
                               ' edge of RA2/INT, TMR0 clock source is RA2,
                               ' increment on high-to-low transitions, prescaler to 
                               ' TMR0, TMR0 Rate 1:2
      TMR0 = 256 - k           ' preload TMR0 to overflow after k counts
      
    ' Initialization of inputs/outputs
      open_Valve = 0    ' Initialize RC2 (open_valve) at Low value
      close_valve = 0   ' Intialize RC3 (close_valve) at Low value
      TRISA.2 = 1       ' Set RA2 as input port for clock to TMR0
      TRISA.3 = 1       ' Set RA3 as input port for sensing Flush switch closure
      meter = 1         ' Initialize RA2 (meter) at High for METER pulse inputs
                        ' RA2 = TMR0 clock input for simulated meter pulse inputs
      flush = 1         ' Initialize RA3 (flush) at High value for flush interrupt
      
    ' Interrupts Settings
    FLUSH_INT_FLAG VAR INTCON.0 ' Alias RA3(FLUSH) On-change-interrupt flag bit
    TMR0_INT_FLAG VAR INTCON.2  ' Alias Timer0 overflow flag bit
    INTCON = %10101000          ' Enable interrupts: global, TMR0 overflow, & RABIE
    IOCA = %00001000            ' Enable RA3 as on-change-INT
    
    ' Set INT Handler
    ON INTERRUPT GOTO Int_handler   
    
    '-----[ Main Code Starts Here ]------------------------------------------------
    MAIN:
      ' Perform following steps to save power during Sleep mode
        OPTION_REG.7 = 1    ' Disable PortA/PortB pull-ups 
        OSCCON = %0100011   ' Switch to 1 MHz internal oscillator
        VRCON = %00100000   ' Set Voltage Reference for minimum power consumption
                            ' Disable CVref
        ANSEL= %11110011    ' Set PortA to Analog I/O to save power during Sleep but
                            ' leave Bits 2 & 3 as digital for RA2 pulse count
                            ' TMR0 overflow interrupt & RA3 On-Change intterupt.
        ANSELH= %11111111   ' Analog module enabled to save power during Sleep
        CM1CON0.7 = 0       ' Turn off comparators during Sleep
        CM2CON0.7 = 0
        PCON.4 = 0          ' Turn off BOR during Sleep
        WDTCON = %00010110  ' Turn WDT off to SLEEP indefinitely
        TRISA = %00001101   ' Set RA0, RA2 & RA3 as inputs
        TRISB = %11111111   ' Set all PORTB pins to inputs during Sleep
        TRISC = %11111111   ' Set all PORTC pins to input during Sleep
        PortA = %11111111   ' Write Port A all High to clear mismatch and save
                            ' power during SLEEP  
        PortB = %11111111   ' Set all Port B and C pins High to save power
        PortC = %11111111   ' in Sleep mode
        @ sleep                  
        ' Microcontroller is in Sleep State waiting for external FLUSH Interrupt
            ' Valve should be closed at this point and no water flowing
       GOTO Main    ' Loop to Main to wait for next Flush interrupt on RA3 change
     
    '------{ Begin Interrupt Handler }---------------------------------------------
      DISABLE       ' Disable interrupts during interrupt handler
    Int_handler:                     
        OPTION_REG.7 = 0    ' Enable PortA/PortB Pullups                   
        WDTCON = %0001000   ' Turn WDT on for ops
        'Initialize registers for interrupt ops
      ' A/D & Comparators disabled
        ANSEL=0             ' Set PortA to digital I/O for use with RA2 and RA3
        ANSELH=0            ' Analog module disabled
        CM1CON0=0           ' Disable comparators
        CM2CON0=0 
      ' Port Settings
        TRISA = %11111111   ' Set all PORTA pins to inputs...RA0, RA2 & RA3 are used
        TRISB = %00000000   ' Set all PORTB pins to outputs
        TRISC = %11110000   ' Set lower 4 pins of PartB as outputs for LEDs
        PORTA = %00000000   ' PortA pins all set to Low
        PORTC = %00000000   ' LEDs off, PULSOUT RC3 provides a high-going pulse 
       
      High bat_mon      ' For testing..simulate battery monitor input is Low or High
      DEFINE WRITE_INT 1
      'Write 13, bat_mon   ' Remove comments on Write statements for test only
      'Write 15, Vthr
      diff = Vthr - bat_mon
      'Write 17, diff
      IF FLUSH_INT_FLAG = 1 Then  ' Interrupt was from RA3 on change
        REPEAT
            ' Wait until the external Flush interrupt is at high level...limits
            ' interrupt to switch closure only and not also for switch opening 
        Until flush = 1 
        PULSOUT open_valve,2000*2   ' Generate 20 msec pulse to RC3 to open valve
        'HIGH led1              ' Light indicator that valve is open & water flowing  
        Write  7, flush        ' Write FLUSH value..remove comment for test only  
        ' Put code here to start a timer to run for 50 secs as a fail safe 
        ' to prevent overflow of toilet tank in case of sensor failure.
            'PULSOUT close_valve,2000 ' Generate 20 msec
                                      ' pulse to RC4 to close valve      
      Endif
      ' Valve is open and water is flowing
      REPEAT
            ' Wait for flow to reach 1.6 gallons
            ' Flash flow light while water flowinguntil pulse counter overflows
            High led1
            Pause 125/10
            Low led1
            Pause 500/10
            If diff > 2 Then   ' Battery is low..flash the low battery monitor light
                HIGH led2      ' while water is flowing
                PAUSE 125/10        
                Low led2
                Pause 500/10
            ENDIF                                            
            'Write 5, bat_mon  ' Remove comment for test only 
      Until TMR0_INT_FLAG = 1  ' Until flow meter pulse counter overflows
      PULSOUT close_valve,2000*2 ' Generate 20 msec pulse to RC4 to close valve  
      WRITE 11, TMR0_INT_FLAG ' Write TMR0 value..remove comment for test only
      dummy = flush            ' Clear mismatch condition                   
      FLUSH_INT_FLAG = 0       ' Clear interrupt flag & enable RA3 on interrupt
      TMR0_INT_FLAG = 0        ' Clear overflow flag
      TMR0 = 256 - k           ' Reload TMR0 to overflow after k counts
      RESUME Main              ' Resume Main Program                 
      ENABLE   
      '-------{ End of Interrupt Handler }-----------------------------------------
          
        ' If the user program ends by getting to the last statement of the program
        ' at an END instruction, the MCU will SLEEP and await a wakeup.
    END

  6. #6
    Join Date
    Nov 2005
    Location
    Bombay, India
    Posts
    967


    Did you find this post helpful? Yes | No

    Default

    Jellis

    This may be totally irrelevant. How do you measure the sleep current? Are you measuring only the CPU current or the total circuit including all its peripherals. Maybe you are measuring the total current of all peripherals included. In that case, you should also look at the possibility that some peripheral is getting biased due to the active state of your PIC output to it.

    I have many devices in the field that use the sleep mode and the total system current is in the region of 40-100uA. 40uA is typical for the PIC. Rest should be related to your circuit and its peripherals.

    It will be helpful if you can post a schematic to assist in identifying your issue.

    Jerson

  7. #7
    Join Date
    May 2004
    Location
    NW France
    Posts
    3,653


    Did you find this post helpful? Yes | No

    Wink

    Hi,Jerson

    You're right ...

    Code:
    ' Perform following steps to save power during Sleep mode
        OPTION_REG.7 = 1    ' Disable PortA/PortB pull-ups 
        OSCCON = %0100011   ' Switch to 1 MHz internal oscillator
        VRCON = %00100000   ' Set Voltage Reference for minimum power consumption
                            ' Disable CVref
        ANSEL= %11110011    ' Set PortA to Analog I/O to save power during Sleep but
                            ' leave Bits 2 & 3 as digital for RA2 pulse count
                            ' TMR0 overflow interrupt & RA3 On-Change intterupt.
        ANSELH= %11111111   ' Analog module enabled to save power during Sleep
        CM1CON0.7 = 0       ' Turn off comparators during Sleep
        CM2CON0.7 = 0
        PCON.4 = 0          ' Turn off BOR during Sleep
        WDTCON = %00010110  ' Turn WDT off to SLEEP indefinitely
        TRISA = %00001101   ' Set RA0, RA2 & RA3 as inputs
        TRISB = %11111111   ' Set all PORTB pins to inputs during Sleep
        TRISC = %11111111   ' Set all PORTC pins to input during Sleep
        PortA = %11111111   ' Write Port A all High to clear mismatch and save
                            ' power during SLEEP  
        PortB = %11111111   ' Set all Port B and C pins High to save power
        PortC = %11111111   ' in Sleep mode
        @ sleep
    seen any ADC power down in those lines ???

    Alain
    ************************************************** ***********************
    Why insist on using 32 Bits when you're not even able to deal with the first 8 ones ??? ehhhhhh ...
    ************************************************** ***********************
    IF there is the word "Problem" in your question ...
    certainly the answer is " RTFM " or " RTFDataSheet " !!!
    *****************************************

  8. #8
    Join Date
    Mar 2009
    Location
    Colorado
    Posts
    378


    Did you find this post helpful? Yes | No

    Default Measuring SLEEP Power

    Quote Originally Posted by Jerson View Post
    Jellis

    This may be totally irrelevant. How do you measure the sleep current? Are you measuring only the CPU current or the total circuit including all its peripherals. Maybe you are measuring the total current of all peripherals included. In that case, you should also look at the possibility that some peripheral is getting biased due to the active state of your PIC output to it.

    I have many devices in the field that use the sleep mode and the total system current is in the region of 40-100uA. 40uA is typical for the PIC. Rest should be related to your circuit and its peripherals.

    It will be helpful if you can post a schematic to assist in identifying your issue.

    Jerson
    I am measuring the SLEEP power by inserting in series my ampmeter at the Vcc input to the MCU. I am not measuring the circuit power to peripherals because they are either not connected during my test or for those that are powered by a pin from the PIC, they are shut down.

    I am also enclosing a copy of the schematic for my design, including peripherals, per your request. If you see any precautions I should take as far as setting interfaces to peripherals before SLEEP to save power, I would greatly appreciate your suggestions.
    Attached Images Attached Images  

Similar Threads

  1. Pin won't stay high during Repeat...Until loop??
    By jellis00 in forum mel PIC BASIC Pro
    Replies: 1
    Last Post: - 16th August 2009, 23:57
  2. Battery powered applications
    By NavMicroSystems in forum Off Topic
    Replies: 7
    Last Post: - 22nd June 2009, 07:12
  3. Can't ID interrupt source with this IntHandler??
    By jellis00 in forum mel PIC BASIC Pro
    Replies: 7
    Last Post: - 3rd June 2009, 02:35
  4. Using Sleep
    By elec_mech in forum mel PIC BASIC Pro
    Replies: 2
    Last Post: - 17th August 2008, 04:05
  5. USART interrupt not interrupting right
    By Morpheus in forum mel PIC BASIC Pro
    Replies: 12
    Last Post: - 6th March 2005, 01:07

Members who have read this thread : 0

You do not have permission to view the list of names.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts