PIC16F628 ... VERY SIMPLE PROBLEM - Electronic Circuits Projects Diagrams Free

micro571 · 28th December 2004, 04:40 AM

Hi,

I'm trying to learn the assembler to program PICs. I'm doing alright, largely learning off of Nigel's tutorials.

I wrote a some-what complicated bit of code and it didn't work. So I keep trouble shooting it until I get down to really basic stuff ... and that won't even work.

Someone, please tell me why even this insanely simple bit of code won't work.

tone equ 0x31

sLoop
movlw 0xff
movwf tone
movlw tone ; <<<<<<
movwf PORTA
goto sLoop

I'm just trying to put 0xff into the register tone and then have tone put into PORTA.

If I replace tone with 0xff on the line with the arrows, it works fine. For some reason it won't let me load a value into tone and then use it later.

What am I doing wrong ?

Thanks.

csaba911 · 28th December 2004, 05:16 AM

Quote:

Originally Posted by micro571

Hi,

tone equ 0x31

sLoop
movlw 0xff
movwf tone
movlw tone ; <<<<<<
movwf PORTA
goto sLoop

What am I doing wrong ?

Thanks.

Code:

sLoop
     movlw 0xff
     movwf tone
     movf  tone,w     ; <<<<<<
     movwf PORTA
goto sLoop

csaba911 · 28th December 2004, 05:17 AM

Quote:

Originally Posted by micro571

Hi,

tone equ 0x31

sLoop
movlw 0xff
movwf tone
movlw tone ; <<<<<<
movwf PORTA
goto sLoop

What am I doing wrong ?

Thanks.

Code:

sLoop
     movlw 0xff
     movwf tone
     movf  tone,w     ; <<<<<<
     movwf PORTA
goto sLoop

**Nigel Goodwin** · 28th December 2004, 07:39 AM

As 'csaba911' has pointed out, you're using the totally wrong instruction, MOVLW means 'Move Literal' - a constant value, specified on the rest of the line. 'Tone' is just an equate name, and will have a numeric value (the address of the GPR it points to), this value will be copied to the W register.

What you need is MOVF - 'Move File', as already pointed out 'MOVF Tone, W' will move the contents of the GPR pointed to by Tone into the W register.

micro571 · 28th December 2004, 09:14 PM

Thanks Nigel and csaba911.

Just one more bit of code I can't figure out ...
After this I should be fine on my own.

PORTB are all INPUTS and PORTA are all OUTPUTS.

Code:

slit		equ	0x31
sLoop
call count
btfsc slit, 0x01     ;<<<<<<
	call turn_on
btfss slit, 0x01     ;<<<<<<
	call turn_off
goto sLoop


turn_on
	movlw	0xff
	movwf	PORTA			;set all bits on
return

turn_off
	movlw	0x00
	movwf	PORTA			;set all bits off
return

count	
clrf slit ;set slit to 0
	BTFSS  PORTB, 3
		incf slit,1
	BTFSS  PORTB, 4
		incf slit,1
	BTFSS  PORTB, 5
		incf slit,1
	BTFSS  PORTB, 6
		incf slit,1
	BTFSS  PORTB, 7
		incf slit,1
return

What I'd like it do is ... count up the number of switches that are on (grounding the pin) and if 0x0? number of switches are on, then put on the LEDs. So if it were 0x01 it would require 1 switch to be on.

What happens in reality with this code, is if there are 1 or 2 switches on ... the LEDs will be on. 3-5 and the lights are off ... doesn't make sense to me.

Any suggestions would be greatly appreciated.

**Nigel Goodwin** · 28th December 2004, 09:34 PM

BTFSC and BTFSS check a single bit for being high or low, this isn't what you want to happen!. To check for a particular value in a register you need to either use SUB or XOR on the two values, and check the zero flag to see if they were equal.

micro571 · 29th December 2004, 01:44 AM

BTFSC and BTFSS are what I want. I want to see how many switches are activated ... so if there are 4, slit will have a value of 4. If 2, then slit will have a value of 3. It doesn't matter which switches are activated, I just need to know how many there are.

**Nigel Goodwin** · 29th December 2004, 06:37 AM

Quote:

Originally Posted by micro571

BTFSC and BTFSS are what I want. I want to see how many switches are activated ... so if there are 4, slit will have a value of 4. If 2, then slit will have a value of 3. It doesn't matter which switches are activated, I just need to know how many there are.

I've already explained that BTFSC and BTFSS won't do what you want, they only test a single bit for high or low.

csaba911 · 29th December 2004, 07:15 AM

If you want to count the number of switches set _low regardless of their order, than your idea works.

Code:

MAIN	
	   BANKSEL	ADCON1
	   MOVLW	  0X06
	   MOVWF	  ADCON1
	   MOVLW	  B'00000000'
	   MOVWF	  TRISA
	   MOVLW	  B'11111111'
	   MOVWF	  TRISB
	   BANKSEL	PORTB
SLOOP CLRF      SLIT                       ;SET SLIT TO 0 
 	  BTFSS  	PORTB, 0 
      	INCF 	SLIT,F 
   	BTFSS  	PORTB, 1 
      	INCF 	SLIT,F 
   	BTFSS  	PORTB, 2 
      	INCF 	SLIT,F 
   	BTFSS  	PORTB, 3 
      	INCF 	SLIT,F 
   	BTFSS  	PORTB, 4 
      	INCF 	SLIT,F 
      MOVF	   SLIT,W
	   MOVWF	  PORTA
	   GOTO	   SLOOP

Good luck
STEVE

micro571 · 30th December 2004, 02:20 AM

Thanks for the code steve.

Here's where I'm stuck ...

The count procedure should go through the 5 switches and see how many are on (grounded). Say 3 are on, slit will have a value of 0x03.

Now how do I say if slit = 0x03 then call SuperLoop ?? Because like Nigel mentioned, btfss and btfsc only test a single pin and not actually the register value.

Nigel mentioned something about SUB or XOR? Do I have to have registers pre-filled with 0x01, 0x02, 0x03... then compare the values to slit ??

Any guidance would be greatly appreciated.
Nigel ... I think I misunderstood what you were originally saying. Sorry bout that.

williB · 30th December 2004, 02:43 AM

there was an earlier topic in here somewhere discussing If Then statments with assembler , but the search dosent seem to be working..

micro571 · 30th December 2004, 03:25 AM

Thanks williB.

I found the topic you mentioned.
(http://www.electro-tech-online.com/v...ic.php?t=12304)

In that topic, Nigel posted a link to here:
http://www.piclist.com/techref/microchip/codeflow.htm

... I've gone through that page and I might be able to figure it out. Tricky stuff this assembler programming.

micro571 · 30th December 2004, 06:37 AM

Ok. I've got it all figured out. SUB was used.

For future reference ... anyone doing PIC programming, here is how you do an IF REGISTER = ??? THEN ...

In this case ... if slit = ?? then ...

Quick explanation:
The value you are checking is loaded into W and subtracted from your register. If the result is zero, then the Z bit of the status flag becomes '1'. Then we can tell it to call a subroutine. If it isn't equal to the value we just checked, the old value is restored (by adding W to it) and we continue to check it against another value.

Code:

movlw 0x05
subwf slit
btfsc STATUS, Z
	call its_five
addwf slit

movlw 0x04
subwf slit
btfsc STATUS, Z
	call its_four
addwf slit

movlw 0x03
subwf slit
btfsc STATUS, Z
	call its_three
addwf slit

movlw 0x02
subwf slit
btfsc STATUS, Z
	call its_two
addwf slit

**Nigel Goodwin** · 30th December 2004, 06:42 AM

Quote:

Originally Posted by micro571

Nigel mentioned something about SUB or XOR? Do I have to have registers pre-filled with 0x01, 0x02, 0x03... then compare the values to slit ??

You have a register called Slit, which contains the number of switches that are turned ON - you want to check if either the value equals a certain number, or is greater than a certain number.

Try looking at http://www.piclist.com/techref/micro...pcon.htm#8_bit. These 8 bit examples use ADDLW rather than SUBLW, simply because 12 bit PIC's don't have SUBLW, so they use complement subtraction.

If you check my tutorials, various ones do similar comparisons, check the IR receiver ones - these check the incoming IR code and turn on the respective LED's.

**Nigel Goodwin** · 30th December 2004, 10:51 AM

One thing I didn't think to mention earlier, you could also do this with a simple 'jump table' - the table would have five entries, each jumping to the routine required for that value.

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode

28th December 2004, 04:40 AM	(permalink)
micro571	PIC16F628 ... VERY SIMPLE PROBLEM Hi, I'm trying to learn the assembler to program PICs. I'm doing alright, largely learning off of Nigel's tutorials. I wrote a some-what complicated bit of code and it didn't work. So I keep trouble shooting it until I get down to really basic stuff ... and that won't even work. Someone, please tell me why even this insanely simple bit of code won't work. tone equ 0x31 sLoop movlw 0xff movwf tone movlw tone ; <<<<<< movwf PORTA goto sLoop I'm just trying to put 0xff into the register tone and then have tone put into PORTA. If I replace tone with 0xff on the line with the arrows, it works fine. For some reason it won't let me load a value into tone and then use it later. What am I doing wrong ? Thanks.

28th December 2004, 07:39 AM	(permalink)
Nigel Goodwin	As 'csaba911' has pointed out, you're using the totally wrong instruction, MOVLW means 'Move Literal' - a constant value, specified on the rest of the line. 'Tone' is just an equate name, and will have a numeric value (the address of the GPR it points to), this value will be copied to the W register. What you need is MOVF - 'Move File', as already pointed out 'MOVF Tone, W' will move the contents of the GPR pointed to by Tone into the W register. __________________ PIC programmer software, and PIC Tutorials at: http://www.winpicprog.co.uk

28th December 2004, 09:14 PM	(permalink)
micro571	Thanks Nigel and csaba911. Just one more bit of code I can't figure out ... After this I should be fine on my own. PORTB are all INPUTS and PORTA are all OUTPUTS. Code: slit equ 0x31 sLoop call count btfsc slit, 0x01 ;<<<<<< call turn_on btfss slit, 0x01 ;<<<<<< call turn_off goto sLoop turn_on movlw 0xff movwf PORTA ;set all bits on return turn_off movlw 0x00 movwf PORTA ;set all bits off return count clrf slit ;set slit to 0 BTFSS PORTB, 3 incf slit,1 BTFSS PORTB, 4 incf slit,1 BTFSS PORTB, 5 incf slit,1 BTFSS PORTB, 6 incf slit,1 BTFSS PORTB, 7 incf slit,1 return What I'd like it do is ... count up the number of switches that are on (grounding the pin) and if 0x0? number of switches are on, then put on the LEDs. So if it were 0x01 it would require 1 switch to be on. What happens in reality with this code, is if there are 1 or 2 switches on ... the LEDs will be on. 3-5 and the lights are off ... doesn't make sense to me. Any suggestions would be greatly appreciated.

28th December 2004, 09:34 PM	(permalink)
Nigel Goodwin	BTFSC and BTFSS check a single bit for being high or low, this isn't what you want to happen!. To check for a particular value in a register you need to either use SUB or XOR on the two values, and check the zero flag to see if they were equal. __________________ PIC programmer software, and PIC Tutorials at: http://www.winpicprog.co.uk

29th December 2004, 01:44 AM	(permalink)
micro571	BTFSC and BTFSS are what I want. I want to see how many switches are activated ... so if there are 4, slit will have a value of 4. If 2, then slit will have a value of 3. It doesn't matter which switches are activated, I just need to know how many there are.

30th December 2004, 02:20 AM	(permalink)
micro571	Thanks for the code steve. Here's where I'm stuck ... The count procedure should go through the 5 switches and see how many are on (grounded). Say 3 are on, slit will have a value of 0x03. Now how do I say if slit = 0x03 then call SuperLoop ?? Because like Nigel mentioned, btfss and btfsc only test a single pin and not actually the register value. Nigel mentioned something about SUB or XOR? Do I have to have registers pre-filled with 0x01, 0x02, 0x03... then compare the values to slit ?? Any guidance would be greatly appreciated. Nigel ... I think I misunderstood what you were originally saying. Sorry bout that.

30th December 2004, 02:43 AM	(permalink)
williB	there was an earlier topic in here somewhere discussing If Then statments with assembler , but the search dosent seem to be working..

30th December 2004, 03:25 AM	(permalink)
micro571	Thanks williB. I found the topic you mentioned. (http://www.electro-tech-online.com/v...ic.php?t=12304) In that topic, Nigel posted a link to here: http://www.piclist.com/techref/microchip/codeflow.htm ... I've gone through that page and I might be able to figure it out. Tricky stuff this assembler programming.

30th December 2004, 06:37 AM	(permalink)
micro571	Ok. I've got it all figured out. SUB was used. For future reference ... anyone doing PIC programming, here is how you do an IF REGISTER = ??? THEN ... In this case ... if slit = ?? then ... Quick explanation: The value you are checking is loaded into W and subtracted from your register. If the result is zero, then the Z bit of the status flag becomes '1'. Then we can tell it to call a subroutine. If it isn't equal to the value we just checked, the old value is restored (by adding W to it) and we continue to check it against another value. Code: movlw 0x05 subwf slit btfsc STATUS, Z call its_five addwf slit movlw 0x04 subwf slit btfsc STATUS, Z call its_four addwf slit movlw 0x03 subwf slit btfsc STATUS, Z call its_three addwf slit movlw 0x02 subwf slit btfsc STATUS, Z call its_two addwf slit

30th December 2004, 10:51 AM	(permalink)
Nigel Goodwin	One thing I didn't think to mention earlier, you could also do this with a simple 'jump table' - the table would have five entries, each jumping to the routine required for that value. __________________ PIC programmer software, and PIC Tutorials at: http://www.winpicprog.co.uk