[SOLVED] Unable to program PIC controller

[Rickooo]

Hi, I want to program PIC18F4520.
I am using PIC Kit 3. I am using MPLAB X IDE and XC8 compiler.

I have tried the programing by:
1. Giving PIC kit power.
2. Giving external power.
3. Using 1k - 10k resistor at MCLR pin.

I think, that it may be a issue of configuration.

Here is the configuration:

/*
* File:   configBits.h
* Author: LENOVO
*
* Created on 21 October, 2020, 1:37 PM
*/

#ifndef CONFIGBITS_H
#define    CONFIGBITS_H


// PIC18F4520 Configuration Bit Settings

// 'C' source line config statements
#define ON  1
#define OFF 0
// CONFIG1H
#pragma config OSC = HS         // Oscillator Selection bits (HS oscillator)
#pragma config FCMEN = OFF      // Fail-Safe Clock Monitor Enable bit (Fail-Safe Clock Monitor disabled)
#pragma config IESO = OFF       // Internal/External Oscillator Switchover bit (Oscillator Switchover mode disabled)

// CONFIG2L
#pragma config PWRT = OFF       // Power-up Timer Enable bit (PWRT disabled)
#pragma config BOREN = SBORDIS  // Brown-out Reset Enable bits (Brown-out Reset enabled in hardware only (SBOREN is disabled))
#pragma config BORV = 3         // Brown Out Reset Voltage bits (Minimum setting)

// CONFIG2H
#pragma config WDT = ON         // Watchdog Timer Enable bit (WDT enabled)
#pragma config WDTPS = 32768    // Watchdog Timer Postscale Select bits (1:32768)

// CONFIG3H
#pragma config CCP2MX = PORTC   // CCP2 MUX bit (CCP2 input/output is multiplexed with RC1)
#pragma config PBADEN = ON      // PORTB A/D Enable bit (PORTB<4:0> pins are configured as analog input channels on Reset)
#pragma config LPT1OSC = OFF    // Low-Power Timer1 Oscillator Enable bit (Timer1 configured for higher power operation)
#pragma config MCLRE = ON       // MCLR Pin Enable bit (MCLR pin enabled; RE3 input pin disabled)

// CONFIG4L
#pragma config STVREN = ON      // Stack Full/Underflow Reset Enable bit (Stack full/underflow will cause Reset)
#pragma config LVP = OFF        // Single-Supply ICSP Enable bit (Single-Supply ICSP disabled)
#pragma config XINST = OFF      // Extended Instruction Set Enable bit (Instruction set extension and Indexed Addressing mode disabled (Legacy mode))

// CONFIG5L
#pragma config CP0 = OFF        // Code Protection bit (Block 0 (000800-001FFFh) not code-protected)
#pragma config CP1 = OFF        // Code Protection bit (Block 1 (002000-003FFFh) not code-protected)
#pragma config CP2 = OFF        // Code Protection bit (Block 2 (004000-005FFFh) not code-protected)
#pragma config CP3 = OFF        // Code Protection bit (Block 3 (006000-007FFFh) not code-protected)

// CONFIG5H
#pragma config CPB = OFF        // Boot Block Code Protection bit (Boot block (000000-0007FFh) not code-protected)
#pragma config CPD = OFF        // Data EEPROM Code Protection bit (Data EEPROM not code-protected)

// CONFIG6L
#pragma config WRT0 = OFF       // Write Protection bit (Block 0 (000800-001FFFh) not write-protected)
#pragma config WRT1 = OFF       // Write Protection bit (Block 1 (002000-003FFFh) not write-protected)
#pragma config WRT2 = OFF       // Write Protection bit (Block 2 (004000-005FFFh) not write-protected)
#pragma config WRT3 = OFF       // Write Protection bit (Block 3 (006000-007FFFh) not write-protected)

// CONFIG6H
#pragma config WRTC = OFF       // Configuration Register Write Protection bit (Configuration registers (300000-3000FFh) not write-protected)
#pragma config WRTB = OFF       // Boot Block Write Protection bit (Boot block (000000-0007FFh) not write-protected)
#pragma config WRTD = OFF       // Data EEPROM Write Protection bit (Data EEPROM not write-protected)

// CONFIG7L
#pragma config EBTR0 = OFF      // Table Read Protection bit (Block 0 (000800-001FFFh) not protected from table reads executed in other blocks)
#pragma config EBTR1 = OFF      // Table Read Protection bit (Block 1 (002000-003FFFh) not protected from table reads executed in other blocks)
#pragma config EBTR2 = OFF      // Table Read Protection bit (Block 2 (004000-005FFFh) not protected from table reads executed in other blocks)
#pragma config EBTR3 = OFF      // Table Read Protection bit (Block 3 (006000-007FFFh) not protected from table reads executed in other blocks)

// CONFIG7H
#pragma config EBTRB = OFF      // Boot Block Table Read Protection bit (Boot block (000000-0007FFh) not protected from table reads executed in other blocks)

// #pragma config statements should precede project file includes.
// Use project enums instead of #define for ON and OFF.

#include <xc.h>


#define _XTAL_FREQ 10000000

#endif

Here is message is

Device Erased...

Programming...

The following memory area(s) will be programmed:
program memory: start address = 0x0, end address = 0x7fff
program memory
Address: 0 Expected Value: 50 Received Value: 0
configuration memory
EEData memory
User Id Memory
Failed to program device

 

[betwixt]

I suspect the reason is the program is zero bytes long!
There is no program code in it at all, only configuration bits.

Brian.

 

[Rickooo]

Can't understand, what you mean. Zero byte long as in ?

 

[betwixt]

Well, all the code you have shown is to set the configuration bits and it is probably doing that correctly. The configuration bits are not in program memory space.

There is no actual code to compile so it is probably correct in saying it can't put it into the program memory.

Try adding a short 'main()' function with a few lines of code in it and see what it does.

Brian.

 

[Aussie Susan]

Also DO NOT define ON and OFF.
For one thing, the 'off' config settings usually have the bit set a '1' (the config settings are all defined in the data sheet and erase to '1' as the default settings).
However I don't think the XC8 compiler actually does text substitution on the #pragma lines (although I could be wrong which makes your defining ON and OF even more of a no-no) so you are probably getting away with it.
I generally recommend that you put the CONFIG pragma lines in the 'main.c' file. That way they are only ever specified once and you don't need to put a conditional around them.
Further, the 'xc.h' include should be in all of the .c files so that the register definitions are available.
Susan

 

[Rickooo]

Thanks. Configuration bits solve the problem.