EEPROM Address

[Ala]

hey there

I am using an EEPROM which is AT24C08. here is my code

//write
ee24_write((16*(3)),(unsigned char *)("[phone]"),13,500);
ee24_write((16*(4)),(unsigned char *)("aaa"),strlen("aaa"),100);
	
 ee24_write((16*(7)),(unsigned char *)("[phone]"),13,500);
 ee24_write((16*(8)),(unsigned char *)("bbb"),strlen("bbb"),100);
	
 ee24_write((16*(11)),(unsigned char *)("[phone]"),13,500);
 ee24_write((16*(12)),(unsigned char *)("ccc"),strlen("ccc"),100);
	
 ee24_write((16*(15)),(unsigned char *)("[phone]"),13,500);
 ee24_write((16*(16)),(unsigned char *)("ddd"),strlen("ddd"),100);
	
 ee24_write((16*(19)),(unsigned char *)("[phone]"),13,500);
 ee24_write((16*(20)),(unsigned char *)("eee"),strlen("eee"),100);
 
//read
 
ee24_read((16*(3)),(unsigned char *)SYS.User[0].PhoneNumber,13,500);
ee24_read((16*(4)),(unsigned char *)SYS.User[0].Name,3,500);
	
 ee24_read((16*(7)),(unsigned char *)SYS.User[1].PhoneNumber,13,500);
 ee24_read((16*(8)),(unsigned char *)SYS.User[1].Name,3,500);
	
 ee24_read((16*(11)),(unsigned char *)SYS.User[2].PhoneNumber,13,500);
 ee24_read((16*(12)),(unsigned char *)SYS.User[2].Name,3,500);

my problem is that the last write("[phone]"-"eee") is being read in the first arrays(SYS.User[0].PhoneNumber-SYS.User[0].Name)

where they should be ("[phone]"-"aaa") instead.

why is that?

here is the codes for ee24_write() and ee24_read()

#define		_EEPROM_SIZE_KBIT							8
#define		_EEPROM_I2C 								hi2c2		
#define		_EEPROM_USE_FREERTOS 0
#define		_EEPROM_ADDRESS 0xA0
#define		_EEPROM_USE_WP_PIN 1
#define _EEPROM_PSIZE 16
#if (_EEPROM_USE_WP_PIN==1)
#define		_EEPROM_WP_GPIO								GPIOB
#define		_EEPROM_WP_PIN								GPIO_PIN_2
 
bool ee24_write(uint16_t address, uint8_t *data, size_t len, uint32_t timeout)
{
 if (ee24_lock == 1)
 return false;
 ee24_lock = 1; 
 uint16_t w;
 uint32_t startTime = HAL_GetTick();
	#if	(_EEPROM_USE_WP_PIN==1)
	HAL_GPIO_WritePin(_EEPROM_WP_GPIO, _EEPROM_WP_PIN,GPIO_PIN_RESET);
	#endif
 while (1)
 {
 w = _EEPROM_PSIZE - (address % _EEPROM_PSIZE);
 if (w > len)
 w = len; 
 #if ((_EEPROM_SIZE_KBIT==1) || (_EEPROM_SIZE_KBIT==2))
 if (HAL_I2C_Mem_Write(&_EEPROM_I2C, _EEPROM_ADDRESS, address, I2C_MEMADD_SIZE_8BIT, data, w, 100) == HAL_OK)
 #elif (_EEPROM_SIZE_KBIT==4)
 if (HAL_I2C_Mem_Write(&_EEPROM_I2C, _EEPROM_ADDRESS | ((address & 0x0100 >> 7)), (address & 0xff), I2C_MEMADD_SIZE_8BIT, data, w, 100) == HAL_OK)
 #elif (_EEPROM_SIZE_KBIT==8)
 if (HAL_I2C_Mem_Write(&_EEPROM_I2C, _EEPROM_ADDRESS | ((address & 0x0300 >> 7)), (address & 0xff), I2C_MEMADD_SIZE_8BIT, data, w, 100) == HAL_OK)
 #elif (_EEPROM_SIZE_KBIT==16)
 if (HAL_I2C_Mem_Write(&_EEPROM_I2C, _EEPROM_ADDRESS | ((address & 0x0700 >> 7)), (address & 0xff), I2C_MEMADD_SIZE_8BIT, data, w, 100) == HAL_OK)		
 #else
 if (HAL_I2C_Mem_Write(&_EEPROM_I2C, _EEPROM_ADDRESS, address, I2C_MEMADD_SIZE_16BIT, data, w, 100) == HAL_OK)
 #endif
 {
 HAL_Delay(10);
 len -= w;
 data += w;
 address += w;
 if (len == 0)
 {
 #if (_EEPROM_USE_WP_PIN==1)
 HAL_GPIO_WritePin(_EEPROM_WP_GPIO, _EEPROM_WP_PIN, GPIO_PIN_SET);
 #endif
 ee24_lock = 0;
 return true;
 }
 if (HAL_GetTick() - startTime >= timeout) 
 {
 ee24_lock = 0;
 return false;
 }
 }
 else
 {
 #if (_EEPROM_USE_WP_PIN==1)
 HAL_GPIO_WritePin(_EEPROM_WP_GPIO, _EEPROM_WP_PIN, GPIO_PIN_SET);
 #endif
 ee24_lock = 0;
 return false;		
 }
 }
}
//################################################################################################################
bool ee24_read(uint16_t address, uint8_t *data, size_t len, uint32_t timeout)
{
 if (ee24_lock == 1)
 return false;
 ee24_lock = 1;
 #if (_EEPROM_USE_WP_PIN==1)
 HAL_GPIO_WritePin(_EEPROM_WP_GPIO, _EEPROM_WP_PIN, GPIO_PIN_SET);
 #endif
 #if ((_EEPROM_SIZE_KBIT==1) || (_EEPROM_SIZE_KBIT==2))
 if (HAL_I2C_Mem_Read(&_EEPROM_I2C, _EEPROM_ADDRESS, address, I2C_MEMADD_SIZE_8BIT, data, len, 100) == HAL_OK)
 #elif (_EEPROM_SIZE_KBIT == 4)
 if (HAL_I2C_Mem_Read(&_EEPROM_I2C, _EEPROM_ADDRESS | ((address & 0x0100 >> 7)), (address & 0xff), I2C_MEMADD_SIZE_8BIT, data, len, 100) == HAL_OK)
 #elif (_EEPROM_SIZE_KBIT == 8)
 if (HAL_I2C_Mem_Read(&_EEPROM_I2C, _EEPROM_ADDRESS | ((address & 0x0300 >> 7)), (address & 0xff), I2C_MEMADD_SIZE_8BIT, data, len, 100) == HAL_OK)
 #elif (_EEPROM_SIZE_KBIT==16)
 if (HAL_I2C_Mem_Read(&_EEPROM_I2C, _EEPROM_ADDRESS | ((address & 0x0700>> 7)), (address & 0xff), I2C_MEMADD_SIZE_8BIT, data, len, 100) == HAL_OK)
 #else
 if (HAL_I2C_Mem_Read(&_EEPROM_I2C, _EEPROM_ADDRESS, address, I2C_MEMADD_SIZE_16BIT, data, len, timeout) == HAL_OK)
 #endif
 {
 ee24_lock = 0;
 return true;
 }
 else
 {
 ee24_lock = 0;
 return false;	
 } 
}

why is that?

[MM..1]

Maybe try change

((address & 0x0300 >> 7))

to

((address & 0x0300) >> 7)

[Ala]

what's the difference? seems just like my own code..

actually I tried a 64bit EEPROM and it worked!!!!

#define		_EEPROM_SIZE_KBIT							64
 
#define _EEPROM_PSIZE 32

are the only changes that I made.

[MM..1]

Sorry your new defines changes use code where you dont have & >> operators, and when priority operator >> is high as & , then your match result

(address & 0x06) 8)