PIC18F2550 - Problem vid skrivning till internt EEPROM minne

[kabelhaerva]

Hej,

Jag håller på att försöka spara lite variabler i det interna EEPROM-minnet på en PIC18F2550.
Använder mig av MPLAB X 3.10 och en PICKIT3.

Programmet skriver in lite variabler på 10 st adresser i minnet men när jag ska läsa av dem så får jag värdet 0xFF på samtliga.
I simulatorn fungerar detta utmärkt och jag kan se i EEPROM-minnet fönster att värdena finns där, men när jag skall köra koden på riktigt via PICKIT3 så fungerar detta inte utan alla värden är 0xFF.
Har gått igenom min CONFIG-bits för att säkerställa att ingen code-protect/write-protect är på.
Antagligen är det något basalt jag missat( det brukar alltid vara så) och därför ber jag om hjälp ifrån färscha ögon som inte låst fast sig i tankebanor.
Kan ju tillägga att som ni säkert ser så är jag nybörjare i C och tar gärna emot tips på saker som ni ser kan göras mycket bättre.
Har ej tagit med min .h filer här som innehåller prototyperna till funktioner och variabler, hojta till dem de behövs.

Här är mina CONFIG BITS:

Kod: Markera allt

// PIC18F2550 Configuration Bit Settings

// 'C' source line config statements

#include <xc.h>

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

// CONFIG1L
#pragma config PLLDIV = 5       // PLL Prescaler Selection bits (Divide by 5 (20 MHz oscillator input))
#pragma config CPUDIV = OSC1_PLL2// System Clock Postscaler Selection bits ([Primary Oscillator Src: /1][96 MHz PLL Src: /2])
#pragma config USBDIV = 1       // USB Clock Selection bit (used in Full-Speed USB mode only; UCFG:FSEN = 1) (USB clock source comes directly from the primary oscillator block with no postscale)

// CONFIG1H
#pragma config FOSC = HS        // Oscillator Selection bits (HS oscillator (HS))
#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 BOR = ON         // 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)
#pragma config VREGEN = OFF     // USB Voltage Regulator Enable bit (USB voltage regulator disabled)

// CONFIG2H
#pragma config WDT = OFF        // Watchdog Timer Enable bit (WDT disabled (control is placed on the SWDTEN bit))
#pragma config WDTPS = 32768    // Watchdog Timer Postscale Select bits (1:32768)

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

// CONFIG4L
#pragma config STVREN = OFF     // Stack Full/Underflow Reset Enable bit (Stack full/underflow will not 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) is not code-protected)
#pragma config CP1 = OFF        // Code Protection bit (Block 1 (002000-003FFFh) is not code-protected)
#pragma config CP2 = OFF        // Code Protection bit (Block 2 (004000-005FFFh) is not code-protected)
#pragma config CP3 = OFF        // Code Protection bit (Block 3 (006000-007FFFh) is not code-protected)

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

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

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

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

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

Här är mina EEPROM funktioner:

Kod: Markera allt

/*
 * File:   eeprom.c
 * Author: snake
 *
 * Created on den 8 oktober 2015, 16:22
 */


#include <xc.h>
#include "eeprom.h"
#include "main.h"

void save_program(unsigned char program_number) {
    

    if (program_number == 1){
        mem_write(0x00, MOTOR_A.SPEED);
        mem_write(0x04, MOTOR_A.LENGTH);
        mem_write(0x08, MOTOR_A.DIR);
        
    }
    
   
    

}

void load_program(unsigned char program_number){
 
        MOTOR_A.SPEED =     (mem_read(0x00)* 100) + mem_read(0x01);
        MOTOR_A.LENGTH =    (mem_read(0x04)* 100) + mem_read(0x05);
        MOTOR_A.DIR =       mem_read(0x08);

        
    }    

}

void mem_write(int addr, int data){
    EECON1bits.EEPGD = 0;                                                       // flash or eeprom memory (0=eeprom)
    EECON1bits.CFGS  = 0;                                                       // choose to write to data mem, 1 = config.registers
    EECON1bits.WREN  = 1;                                                       // write to eeprom mem. enabled
    
        
   
    byte_splitter(data);
    
    

    TMR0IE = 0;                                                         // disable interrupts
    
    EEADR = addr;
    EEDATA = temp_data0;   
    EECON2 = 0x55;
    EECON2 = 0x0AA;
    EECON1bits.WR = 1;
    while (!EECON1bits.WR);  
    
    
    EEADR = addr + 1;
    EEDATA = temp_data1;    
    EECON2 = 0x55;
    EECON2 = 0x0AA;
    EECON1bits.WR = 1;  
    while (!EECON1bits.WR);    
    
    EECON1bits.WREN = 0;                                                        // write to eeprom mem. disabled
    
    TMR0IE = 1;                                                         // enable interrupts again
    return;    
}

int mem_read(int addr){
    
    unsigned int mem_location = addr;
    unsigned int data = 0;
    EEADR = mem_location;
    EECON1bits.EEPGD = 0;
    EECON1bits.CFGS = 0;
    EECON1bits.RD = 1;
    data = EEDATA;
    //bt_tx(data);



        return data;
    
}


int byte_splitter(unsigned int data){
    

    temp_data0 = data / 100;
    temp_data1 = data - (temp_data0 * 100);
    return;
}

Min main.c fil som den ser ut just nu när jag debuggar:

Kod: Markera allt

void main(void) {
    
    init();
    
    MOTOR_A.DIR = 1;
    MOTOR_A.SPEED = 333;
    MOTOR_A.LENGTH = 444;

    save_program(1);

    MOTOR_A.DIR = 0;
    MOTOR_A.SPEED = 0;
    MOTOR_A.LENGTH = 0;

    load_program(1);

    while(1){};

}

[baron3d]

Så här ser min EEPROM skrivning ut:

Kod: Markera allt

void EEPROM_put(auto int addr, auto unsigned char data) { 
	EECON1bits.EEPGD = 0;  
	EECON1bits.CFGS = 0;  
	EECON1bits.WREN = 1;   
	EEADR = addr;          
	EEDATA = data;        
	INTCONbits.GIE = 1; // Disable interrupts
	EECON2 = 0x55;         
	EECON2 = 0xAA;         
	EECON1bits.WR = 1;     
	INTCONbits.GIE = 0; // Enable interrupts
	while (!PIR2bits.EEIF);
	PIR2bits.EEIF = 0;     
	EECON1bits.WREN = 0;   
}
//--------------------------------------------------------

[Marta]

Har Du kontrollerat vad kompilern verkligen genererar för kod?

Denna sekvens:

MOVLW 55h
MOVWF EECON2
MOVLW 0AAh
MOVWF EECON2
BSF EECON1, WR

Måste ovillkorligen vara exakt på detta sätt utan t.ex. banksel o.dyl. instoppat någonstans.

----
baron3d: INTCONbits.GIE = 1; // Disable interrupts

GIE=1 betyder interrupt ENABLE.

[kabelhaerva]

Kan nog vara en bra ide att köra just skrivningen i assembler kod. Får prova detta ikväll och se om det fungerar bättre. Återkommer med resultat!

[baron3d]

Ops,

[kabelhaerva]

Och nu verkar vi ha löst problemet.
Gick igenom .lst filen och kikade på assemblyn som XC8 skapar och den såg korrekt ut så där var inte felet.
Felet var helt enkelt att vid varje programmering så raderade min PICKIT3 EEPROM-minnet så jag fick välja att klicka i boxen "Preserve EEPROM memory" under inställningar för PICKIT3.
Tack för alla svar och hoppas att denna tråd kan hjälpa någon annan som missat denna simpla inställning!