Below is the Problem that i am facing and i posted it too yesterday but still no response:

uint8_t Get_Version_Data()
{
 frame.data[0] = MINOR_VERSION;
 frame.data[1] = MAJOR_VERSION;
 frame.data[2] = 0; // Max packet size (256)
 frame.data[3] = 1;
 frame.data[4] = 0;
 frame.data[5] = 0;
 EEADR = 0x0006; // Get device ID
 EECON1bits.CFGS = 1;
 EECON1bits.RD = 1;
 NOP();
 NOP();
 frame.data[6] = EEDATL;
 frame.data[7] = EEDATH;
 frame.data[8] = 0;
 frame.data[9] = 0;
frame.data[10] = ERASE_FLASH_BLOCKSIZE;
 frame.data[11] = WRITE_FLASH_BLOCKSIZE;
EEADR = 0x0000;
 EECON1bits.CFGS = 1;
 for (uint8_t i= 12; i < 16; i++)
 {
 EECON1bits.RD = 1;
 NOP();
 NOP();
 frame.data = EEDATL;
 ++ EEADRL;
 }
return 25; // total length to send back 9 byte header + 16 byte payload
}
// **************************************************************************************
// Read Flash
// In: [<0x01><DLEN><ADDRL><ADDRH><ADDRU>]
// OUT: [<0x01><DLEN><ADDRL><ADDRH><ADDRU><DATA>...]
uint8_t Read_Flash()
{
 EEADRL = frame.address_L;
 EEADRH = frame.address_H;
 EECON1 = 0x80;
 for (uint8_t i = 0; i < frame.data_length; i += 2)
 {
 EECON1bits.RD = 1;
 NOP();
 NOP();
 frame.data = EEDATL;
 frame.data[i+1] = EEDATH;
 ++ EEADR;
 }
return (frame.data_length + 9);
}
// **************************************************************************************
// Write Flash
// In: [<0x02><DLENBLOCK><0x55><0xAA><ADDRL><ADDRH><ADDRU><DATA>...]
// OUT: [<0x02>]
uint8_t Write_Flash()
{
 EEADRL = frame.address_L;
 EEADRH = frame.address_H;
 EECON1 = 0xA4; // Setup writes
 if ((EEADR & 0x7FFF) < NEW_RESET_VECTOR)
 {
 frame.data[0] = ERROR_ADDRESS_OUT_OF_RANGE;
 return (10);
 }
 for (uint8_t i= 0; i < frame.data_length; i += 2)
 {
 if (((EEADRL & LAST_WORD_MASK) == LAST_WORD_MASK)
 || (i == frame.data_length - 2))
 EECON1bits.LWLO = 0;
 EEDATL = frame.data;
 EEDATH = frame.data[i+1];
StartWrite();
 ++ EEADR;
 }
 frame.data[0] = COMMAND_SUCCESS;
 EE_Key_1 = 0x00; // erase EE Keys
 EE_Key_2 = 0x00;
 return (10);
}
// **************************************************************************************
// Erase Program Memory
// Erases data_length rows from program memory
uint8_t Erase_Flash ()
{
 EEADRL = frame.address_L;
 EEADRH = frame.address_H;
 if ((EEADR & 0x7FFF) < NEW_RESET_VECTOR)
 {
 frame.data[0] = ERROR_ADDRESS_OUT_OF_RANGE;
 return (10);
 }
 for (uint16_t i=0; i < frame.data_length; i++)
 {
 if ((EEADR & 0x7F) >= END_FLASH)
 {
 frame.data[0] = ERROR_ADDRESS_OUT_OF_RANGE;
 return (10);
 }
 EECON1 = 0x94; // Setup writes
 StartWrite();
 EEADR += ERASE_FLASH_BLOCKSIZE;
 }
 frame.data[0] = COMMAND_SUCCESS;
 frame.EE_key_1 = 0x00; // erase EE Keys
 frame.EE_key_2 = 0x00;
 return (10);
}
// **************************************************************************************
// Read_EE_Data
//
// In: [<0x04><DataLengthL><DataLengthH> <unused><unused> <ADDRL><ADDRH><ADDRU><unused>...]
// OUT: [<0x04><DataLengthL><DataLengthH> <unused><unused> <ADDRL><ADDRH><ADDRU><unused><DATA>...]
//
uint8_t Read_EE_Data()
{
 EEADRL = frame.address_L;
 EEADRH = frame.address_H | 0x70; // 0x70 needed for new style ee access
 EECON1 = 0x40; // sets PGD bit for old style eeaccess
 for (uint8_t i = 0; i < frame.data_length; i++)
 {
 EECON1bits.RD = 1;
 NOP();
 NOP();
 frame.data = EEDATL;
 ++ EEADRL;
 }
 return (frame.data_length+9);
}
// **************************************************************************************
// Write_EE_Data
// In: [<0x05><DLEN><ADDRL><ADDRH><0x00><DATA>...]
// OUT: [<0x05>]
uint8_t Write_EE_Data()
{
 EEADRL = frame.address_L;
 EEADRH = frame.address_H | 0x70;
 EECON1 = 0x44;
 for (uint8_t i = 0; i < frame.data_length; i++)
 {
 EEDATL = frame.data;
 StartWrite ();
 while (EECON1bits.WR == 1); // wait until previous write complete
 ++ EEADRL;
 }
 frame.data[0] = 0x01;
 return 10;
}
// **************************************************************************************
// Read Config Words
// In: [<0x06><DataLengthL><unused> <unused><unused> <ADDRL><ADDRH><ADDRU><unused>...]
// OUT: [9 byte header + 4 bytes config1 + config 2]

uint8_t Read_Config ()
{
 EEADRL = frame.address_L;
 EEADRH = frame.address_H;
 EECON1 = 0x40; // can these be combined?
 for (uint8_t i= 0; i < frame.data_length; i += 2)
 {
EECON1bits.RD = 1;
 NOP();
 NOP();
 frame.data = EEDATL;
 frame.data[i+1] = EEDATH;
 ++ EEADR;
 }
 return (13); // 9 byte header + 4 bytes config words
}
// **************************************************************************************
// Write Config Words
uint8_t Write_Config ()
{
 EEADRL = frame.address_L;
 EEADRH = frame.address_H;
 if ((EEADR & 0x7FFF) < NEW_RESET_VECTOR)
 {
 frame.data[0] = ERROR_ADDRESS_OUT_OF_RANGE;
 return (10);
 }
 EECON1 = 0xC4; // Setup writes
 for (uint8_t i = 0; i < frame.data_length; i += 2)
 {
 EEDATL = frame.data;
 EEDATH = frame.data[i+1];
StartWrite();
 ++ EEADR;
 }
 frame.data[0] = COMMAND_SUCCESS;
 EE_Key_1 = 0x00; // erase EE Keys
 EE_Key_2 = 0x00;
 return (10);
}

// **************************************************************************************
// Calculate Checksum
// In: [<0x08><DataLengthL><DataLengthH> <unused><unused> <ADDRL><ADDRH><ADDRU><unused>...]
// OUT: [9 byte header + ChecksumL + ChecksumH]

 
uint8_t Calc_Checksum()
{
 EEADRL = frame.address_L ; //<< 1 not present in original
 EEADRH = frame.address_H ; //<< 1 not present in original
 // if(frame.address_L & 0x08) //Not present in original
 // EEADRH +=1; //Not present in original
 EECON1 = 0x80; //0x195 //0x80; //0x08
 check_sum = 0; //0xF359; //0xFFFF; //0
 
for (uint16_t i = 0;i < frame.data_length; i += 2)
 {
 EECON1bits.RD = 1;
 // while(EECON1bits.RD = 1) //Not present in original
 NOP();
 NOP();
 check_sum += (uint16_t)EEDATA;
 ++ EEADR;
 }
 frame.data[0] = (uint8_t) (check_sum & 0x00FF);
 frame.data[1] = (uint8_t)((check_sum & 0xFF00) >> 8);
 return (11);
}
 
 
// *****************************************************************************
// Unlock and start the write or erase sequence.
void StartWrite()
{
 CLRWDT();
 EECON2 = EE_Key_1;
 EECON2 = EE_Key_2;
 EECON1bits.WR = 1; // Start the write
// had to switch to assembly - compiler doesn't comprehend no need for bank switch
 asm ("movf " str(_EE_Key_1) ",w");
 asm ("movwf " str(BANKMASK(EECON2)));
 asm ("movf " str(_EE_Key_2) ",w");
 asm ("movwf " str(BANKMASK(EECON2)));
 asm ("bsf " str(BANKMASK(EECON1)) ",1"); // Start the write
NOP();
 NOP();
 return;
}

PIC16F1788
Hi Thank you for your reply.
 
Can you explain it in simple words and in detail please. As i am new to this that's why i am not familiar with many things.

JANickerson
PIC16F1788
  Please do make use of "code tags" when posting code.
use "[ c o d e ] before the code and [ / c o d e ] after.
Do type the code tags without the spaces ( if I removed the spaces you would no longer be able to see the tags).

PIC16F1788
Hi Thank you for your reply.

Can you explain it in simple words and in detail please. As i am new to this that's why i am not familiar with many things.
 

JANickerson
PIC16F1788
  Please do make use of "code tags" when posting code.
use "[ c o d e ] before the code and [ / c o d e ] after.
Do type the code tags without the spaces ( if I removed the spaces you would no longer be able to see the tags).