Do PICs goto sleep?

[J_Brittian]

Is MaskSwitch ending up outside your parameters? What happens if you add a case else statement?

[Tissy]

Hmm, got me thinking now.

If i only wanted to look at RC0 , RC1 & RC2, i have now changed the TRISC to read:

TRISC = %00000111	' Set PORTC (0-2 input) rest Outputs

Is this correct?

Also, would any change need to be done for this line? What does the $07 mean?

MaskSwitch=PORTC & $07

What would you suggest i put in the CASE ELSE statement. A return back to Main? It may be worth puttin in this statement to capture any other event.

Thank you.

[J_Brittian]

Actually I was going to suggest an output of some kind, maybe an led. If MaskSwitch gets outside of your parameters in someway you could effectively end up in a main ... goto main loop.

[Tissy]

OK, I've added the following

Main:
    MaskSwitch=PORTC & $07
        Select Case MaskSwitch
          Case 1 '%00000001 or SW1
            Gosub Procedure_SW1

          Case 2 '%00000010 or SW2
            Gosub Procedure_SW2

          Case 4 '%00000100 or SW3
            Gosub Procedure_SW3
          Case Else
            Gosub Error
    End Select
Goto Main

Error:
    For loop = 1 to 5
        Red = 1
        Pause 100
        Red = 0
        Pause 100
    Next Loop
Return

Now the LED is flashing all the time, so i presume the CASE ELSE is true, which i guess means something else is happening.

I think it may be due to the following:

MaskSwitch=PORTC & $07

What does the $07 represent. Is this saying all of the PORTS on C. If so, what will it be if i want just PORTC.0, PORTC.1 and PORTC.2.

Thank you, i think i'm getting there!!

[Dwayne]

I agree with IngVar,

You have no way of knowing if the case statement is something other than 1,2,4. Put a default case in there with a blinking light....If that blinking light comes on... there is your problem.

Dwayne

[Tissy]

Dwayne,

Is what i've done above your posting wrong then? If so, could you post the modification to the code you are suggesting.

Many thanks.

[J_Brittian]

$07 = %00000111

Steve,
If you push two buttons at the same time or if you're not using pull down/up resistors you end up with MaskSwitch = %00000110 which isn't an option.

[Tissy]

Bugger, i think it might be due to no pull-down resistors.

I've attahced my circuit, would this be the case?

What values are recommended ?

Hopefully that is the problem which is reasonably easy to fix.

Regards,

Steve

[J_Brittian]

Try 5K. Hopefully that fixes your problem.

[Tissy]

The pull-down resistors didn't make any difference, the LED keeps in the 'error' flash routine. I didn't have and 5K, so i used a 2k2, would this still perform the task? If not i have 47k?

However, i have inserted:

Main:
    MaskSwitch=PORTC & $07
        Select Case MaskSwitch
          Case 1 '%00000001 or SW1
            Gosub Procedure_SW1

          Case 2 '%00000010 or SW2
            Gosub Procedure_SW2

          Case 4 '%00000100 or SW3
            Gosub Procedure_SW3
          Case Else
            Goto Main
    End Select
Goto Main

Adding the Goto Main routine now seems to work.

Although this is a work around, its still strange that some other signal is being received into the PIC which i would like to find.

Cheers.