18F1220 fuse help

[mackrackit]

First I do not see how you are handling MCLRE in the configs or hardware.

Second, take a look at the P18F1220.inc in the MPASM directory. That will show the correct way of setting the configs. They should be in this format

    @ __CONFIG    _CONFIG1H, _OSC_HS_1H
    @ __CONFIG    _CONFIG2H, _WDT_OFF_2H & _WDTPS_128_2H
    @ __CONFIG    _CONFIG4L, _LVP_OFF_4L

[mr.sneezy]

Thanks Dave, yes the MCLRE was the magic missing ingredient. I thought I'd done that last night but noooo....

I did look in the P18F1220.INC and followed it's advice. It actually says this, and shows the word 'Important' in loud capitals...

;================================================= =========================
;
; IMPORTANT: For the PIC18 devices, the __CONFIG directive has been
; superseded by the CONFIG directive. The following settings
; are available for this device.
;
; Oscillator Selection:
; OSC = LP LP Oscillator
; OSC = XT XT Oscillator
; OSC = HS HS Oscillator

etc etc for about 50 lines of Config choices.

I added this to get my PIC alive.
'Config3H register
@ CONFIG MCLRE=OFF
'Reset pin disabled

It works fine for me, no PBP compile errors, so is it really the wrong way to write the configs ?

I have a new issue though, I sped up the INT OSC to 8Mhz with the OSCCON and set 'Define Osc 8' to match, but the code is executing now twice as fast as it should, like the Define Osc 8 is not being used by PBP. Can that happen for some reason ?

[Acetronics2]

Hi,

Both methods work ...

the old one still is in "old" devices .inc files :

; To embed the Configuration Bits in your source code, paste the
; following lines into your source code in the following format,
; and change the configuration value to the desired setting (such
; as WDT_OFF to WDT_ON).
; These lines are commented out - each __CONFIG line should have the
; preceding semicolon (;) removed when pasted into your source code.
; __CONFIG _CONFIG1H, _IESO_ON_1H & _FSCM_OFF_1H & _RC_OSC_1H
; __CONFIG _CONFIG2L, _PWRT_ON_2L & _BOR_OFF_2L & _BORV_27_2L
; __CONFIG _CONFIG2H, _WDT_OFF_2H & _WDTPS_32K_2H
; __CONFIG _CONFIG3H, _MCLRE_ON_3H
; __CONFIG _CONFIG4L, _DEBUG_OFF_4L & _LVP_ON_4L & _STVR_ON_4L
; __CONFIG _CONFIG5L, _CP0_OFF_5L & _CP1_OFF_5L
; __CONFIG _CONFIG5H, _CPB_OFF_5H & _CPD_OFF_5H
; __CONFIG _CONFIG6L, _WRT0_OFF_6L & _WRT1_OFF_6L
; __CONFIG _CONFIG6H, _WRTC_OFF_6H & _WRTB_OFF_6H & _WRTD_OFF_6H
; __CONFIG _CONFIG7L, _EBTR0_OFF_7L & _EBTR1_OFF_7L
; __CONFIG _CONFIG7H, _EBTRB_OFF_7H

from the "up to date" .inc file of the '1320 ... ( MPLAB 8.50 )

using those, we only Get an "Info message" from MPASM ... telling there's a new ( simplified ??? ) way ...

The old way advantage ??? ... there are templates available in the MPASM .inc files !!!

Alain

PS:

in MPLAB, for Pic 18 ... there's a tool named Visual initializer ... that could help.
see :

; Initialization Code for PIC18F2520, Family: GP control, Package: 28-Pin SDIP 28pins
#include P18F2520.inc
　
　
; Filter Controls used to Generate Code:
; POR Match Filter OFF
; Provisioned Feature Filter OFF
; Masks are Ignored and uses UnMasked Register Writes
; Feature=fuses - fuses (DCR) configuration
; B7=IESO B6=FCMEN B3:0=FOSC3:0
CONFIG1H=0x02
; B4:3=BORV1:0 B2:1=BOREN1:0 B0=PWRTEN-L
CONFIG2L=0xEB
; B4:1=WDTPS3:0 B0=WDTEN
CONFIG2H=0xE0
; B7=MCLRE B2=LPT1OSC B1=PBADEN B0=CCP2MX
CONFIG3H=0xFF
; B7=DEBUG-L B6=XINST B2=LVP B0=STVREN
CONFIG4L=0xFF
; B3:0=CP3:0
CONFIG5L=0xFF
; B7=CPD B6=CPB
CONFIG5H=0xFF
; B3:0=WR3:0
CONFIG6L=0xFF
; B7=WRTD B6=WRTB B5=WRTC
CONFIG6H=0xFF
; B3:0=EBTR3:0
CONFIG7L=0xFF
; B6=EBTRB
CONFIG7H=0xFF
 
...

isn't life marvellous ???

[mr.sneezy]

Thanks Alain, glad I was doing something OK.
I like the new method, I was never very happy with all the _underscore stuff going on with the other method so I'll stick with it.
(Perhaps there is a need for a new template library in this forum somewhere for the new syntax).

I'll have a look at the tool you pointed out, thanks. I got to say I'm having information overload with this particular project, I've changed PIC's twice now from my original choice, and it caused headaches both times. Hoping to settle down to writing useful code real soon.
Cheers,
Martin