ASM - Problem at first step!

[Megahertz]

Hi, I am having problem in compiling the following code:

	LIST P=16F676
	__CONFIG H'2007', B'11000000000100'


	ORG	0
	GOTO	5
	ORG	4
	GOTO	5
	ORG	5

	END

The config line is causing the error. The error is the following:

Error[108] C:\D_DRIVE\ASM_LEARNING\16F676\676_ASM.ASM 2 : Illegal character (,)
Halting build on first failure as requested.
----------------------------------------------------------------------
Debug build of project `C:\D_DRIVE\ASM_Learning\16F676\676_ASM.mcp' failed.
Preprocessor symbol `__DEBUG' is defined.
Sat Jul 23 19:48:06 2011
----------------------------------------------------------------------
BUILD FAILED

I am using MPLAB IDE V8.15. I am trying to learn ASM and this is my first shot at it.
Thanks

[Bruce]

There are .asm template files in your MPASM directory that show basic templates for beginners.

Look in C:\Program Files\Microchip\MPASM Suite\Template\Code

For a template for the 16F676.

[Megahertz]

Thanks Bruce
Really useful information.

[Megahertz]

I am able to compile it now but it is not working. I am using Proteus 7.9 software to simulate it but
Is something wrong with the code?

	LIST P=16F676
	#include <p16F676.inc>
	__CONFIG   _CP_OFF & _CPD_OFF & _BODEN_OFF & _MCLRE_ON & _WDT_OFF & _PWRTE_ON & _INTRC_OSC_NOCLKOUT

	ORG	0
	GOTO	5
	ORG	4
	GOTO	5
	ORG	5

	CLRF	H'07'
	BSF	H'03',5
	CLRF	H'07'
	CLRF	H'91'
	BCF	H'03',5

	BSF	H'07',4
	BSF	H'07',5
	BSF	H'07',2
	BSF	H'07',3

	END

[mister_e]

couple of mistakes in there... no problem it's your first I guess

You haven't cleared the TRISC register
no real need for multiple ORG and GOTO at the begining
If you want to use GOTOs, you want to use/define some label and jump to them.
It's really more readable to use register name instead of their addresses.
Make suire you're always in the good BANK when writing/writing to a registers. Datasheet show everything 'bout that
Maybe not a bad idea to retrieve the oscillator calibration at the begining.

Like Bruce said, check out your MPASM\Template\CODE folder.

[Megahertz]

couple of mistakes in there... no problem it's your first I guess

You haven't cleared the TRISC register
no real need for multiple ORG and GOTO at the begining
If you want to use GOTOs, you want to use/define some label and jump to them.
It's really more readable to use register name instead of their addresses.
Make suire you're always in the good BANK when writing/writing to a registers. Datasheet show everything 'bout that
Maybe not a bad idea to retrieve the oscillator calibration at the begining.

Like Bruce said, check out your MPASM\Template\CODE folder.

Thanks. I was under the impression that the below statements clear the TRISC register.

	CLRF	H'07' ; clear portC
	BSF	H'03',5  ; Jump to bank 1
	CLRF	H'07'  ; Clear Trisc
	CLRF	H'91' ; Clear ANSEL register
	BCF	H'03',5  ; Jump to Bank 0

[Megahertz]

Even tried this now. But

	......
	ORG	5

	CLRF	PORTC
	BSF	STATUS,RP0
	CLRF	TRISC
	CLRF	ANSEL
	BCF	STATUS,RP0
	BSF	PORTC,4
	BSF	PORTC,5
	BSF	PORTC,2
	BSF	PORTC,3

	END

[cncmachineguy]

Compiles and works fine in MPLAB SIM. PORTC bits 4,5,2,3 all get set. Post your Protous file and we can have a look for you.

[Megahertz]

16f676.zip

My DSN file attached.

[mister_e]

I don't use ISIS/Proteus, but as per your configuration fuses, you must tie you MCLR pin to Vdd... make sure it is done.

Another thing... It's nogt a bad Idea to finish your program with an endless loop, so the LED will stay on.

I bet this will work, if you haven't tied you MCLR pin to Vdd

    LIST P=16F676
    #include <p16F676.inc>
    __CONFIG   _CP_OFF & _CPD_OFF & _BODEN_OFF & _MCLRE_OFF & _WDT_OFF & _PWRTE_ON & _INTRC_OSC_NOCLKOUT
    
    CLRF    PORTC
    BSF    STATUS,RP0
    CLRF    TRISC
    CLRF    ANSEL
    BCF    STATUS,RP0
    BSF    PORTC,4
    BSF    PORTC,5
    BSF    PORTC,2
    BSF    PORTC,3
    goto $
    END

And from what I do remember of my venture with ISIS, you need to specify the osc speed... set it to 4MHz and try the above without specifying the configuration word value. If it's still not working, tie your MCLR pin to Vdd. Ought to work ... or throw away ISIS

[cncmachineguy]

Steve is correct You need to either turn off MCLR or tie it high. As a side note, you have some hardware issues that will make you think the program doesn't work. It has to do with tying all the LED's together then through 1 resistor to ground. Also I think you have to edit the properties for the GND symble and assign it the name "GND" to make it be ground.

I am Very new to ISIS, but that has caused me trouble in the past.

Sorry bout taking so long to reply, had trouble getting on the forum yesterday.

[mister_e]

..., had trouble getting on the forum yesterday.

Me too, and I guess everybody has too. Bah, it happen once in a while. No problem, nobody will die of it

[Megahertz]

Thanks for the help. I tied MCLR to VDD & named the ground terminal 'GND'. Worked fine. I almost started having suicidal thoughts becoz of this. Thanks again.

[mister_e]

I almost started having suicidal thoughts becoz of this.

Don't do that again, no SIM worth that much attention