Проблема записи в EEPROM. Из 128-й на ATmega2560

[mehanik_all]

Приветствую. Пытаюсь перенести кусок рабочего кода с ATmega128 на ATmega2560, и возникает проблема с записью из SRAM в EEPROM, а конкретно записывает одну переменную из семи, и дальше зависает. AVR Studio 4.18, оптимизация O3/Os

#define F_CPU 8000000UL // Я использую кристалл 8 МГц

#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <avr/eeprom.h>

struct config_t {
  uint8_t cwl;				
  uint8_t cwh;				
  uint8_t tpsacold; 		
  uint8_t divider;			
  uint8_t alternate; 		
  uint8_t injpwm;			
  uint8_t injpwmt;
};

struct config_t config;
struct config_t config_ee EEMEM;


volatile uint8_t eeprom_store_idx;     /* счетчик обрабатываемых байт данных*/
volatile uint8_t eeprom_store_busy;    /* флаг, eeprom занято*/

/*
  Эта функция запускает процесс сохранение структуры конфигурации из sram в 
  eeprom и немедленно возвращает управление,позволяя обработчику прерывания 
  EEPROM продолжить работу по сохранению данных.
*/
uint8_t storeConfig(void) {

  if (eeprom_store_busy) {
    return -1; 				/* eeprom занято, подождите! */
  }

  eeprom_store_idx = 0;		/* обнулить счетчик обрабатываемых байт данных*/
  eeprom_store_busy = 1;	/* установить флаг, eeprom занято*/
  
  EECR |= _BV(EERIE);		/* разрешить прерывание от eeprom */
  return 0;
}


/*
  время выполнения этого прерывания может быть достаточно высоким в случае 
  самой плохой ситуации, это должно быть обязательно исследовано!
*/
SIGNAL(SIG_EEPROM_READY) {
  uint8_t *pe, *ps;
  uint8_t ebyte;

  pe = (uint8_t *)&config_ee + eeprom_store_idx; /* адрес данных в eeprom */
  ps = (uint8_t *)&config + eeprom_store_idx;    /* адрес данных в sram */
  ebyte = eeprom_read_byte(pe);

  /* перезаписывать данные в eeprom будем только если это необходимо */
  /* 
    сравним содержимое структур в sram и eeprom, выход из цикла 
	осуществляем по первому найденному различию в данных
  */	 
  while( (ebyte == *ps) && (eeprom_store_idx != (sizeof(struct config_t) - 1)) ) {
    eeprom_store_idx++;
    pe++;
    ps++;
    ebyte = eeprom_read_byte(pe);
  }

  /* записываем новое значение в eeprom */
  if (ebyte != *ps)
    eeprom_write_byte(pe, *ps);

  /*
    если последняя запись в структуре не достигнута переходим к поиску 
	следующего различия данных в sram и eeprom 
  */
  if (eeprom_store_idx == sizeof(struct config_t)-1) { 	/* запись закончена */
      EECR &= ~_BV(EERIE);     							/* запретить прерывание от eeprom */
      eeprom_store_busy = 0;							/* сбросить флаг, eeprom занято*/
      PORTB &= ~_BV(PB7); 							/* погасить светодиод */
  } else { 												/* переходим к следующему различию */
      eeprom_store_idx++;
      PORTB |= _BV(PB7); 							/* зажечь светодиод */
  /* 
    необходимо отметить, если светодиод горит более 5 сек следует снять питание 
	с контроллера иначе возможно повреждение eeprom
  */
  }
}


int main(void){

	DDRB |= _BV (PB7);		//установить как выход LED
	PORTB &=~ _BV (PB7);	//установить 0 LED

	config.cwl=50;			//0x32
	config.cwh=65;			//0x41
	config.tpsacold=155;	//0x9B
	config.divider=48;		//0x30
	config.alternate=4;		//0x04
	config.injpwm=94;		//0x5E
	config.injpwmt=223;		//0xDF

	uint8_t i=0;

	if (i==0)
	{
		eeprom_write_byte(0x11, config.injpwm);
		i++;
	}

	sei();

	while(1){
	
	storeConfig();

	}
}

В EEPROM записывает только переменную cwl, и переменную injpwm по указанному адресу, далее цикл в программе не продолжается.

:10000000 32000000000000FFFFFFFFFFFFFFFFFF C7
:10001000 FF5EFFFFFFFFFFFFFFFFFFFFFFFFFFFF 91

Для записи остальных переменных нужно шесть раз сбросить работу МК. Что я делаю не так? Где может быть ошибка?

[Реклама]

Реклама: ООО ТД Промэлектроника, ИНН: 6659197470, Тел: 8 (800) 1000-321

[Alex]

Про отладку что-нибудь слышали ? Подобное на раз-два вылавливается.

[Реклама]

20% скидка на весь каталог электронных компонентов в ТМ Электроникс!

Акция "Лето ближе - цены ниже", успей сделать выгодные покупки!

Плюс весь апрель действует скидка 10% по промокоду APREL24 + 15% кэшбэк и бесплатная доставка!

Перейти на страницу акции

Реклама: ООО ТМ ЭЛЕКТРОНИКС, ИНН: 7806548420, info@tmelectronics.ru, +7(812)4094849

[mehanik_all]

Не вылавливается. Висит разрешение EERIE и SREG I, но обработчик прерывания не запускается. Куда смотреть? Отладчиком почти не пользуюсь.

[Реклама]

Выбираем схему BMS для корректной работы литий-железофосфатных (LiFePO4) аккумуляторов

 Обязательным условием долгой и стабильной работы Li-FePO4-аккумуляторов, в том числе и производства EVE Energy, является применение специализированных BMS-микросхем. Литий-железофосфатные АКБ отличаются такими характеристиками, как высокая многократность циклов заряда-разряда, безопасность, возможность быстрой зарядки, устойчивость к буферному режиму работы и приемлемая стоимость. Но для этих АКБ, также как и для других, очень важен контроль процесса заряда и разряда, а специализированных микросхем для этого вида аккумуляторов не так много. Инженеры КОМПЭЛ подготовили список имеющихся микросхем и возможных решений от разных производителей. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[mehanik_all]

Выкинул прерывание, запилил всё через функцию, вроде зафурычило как надо, осталось погонять хорошо на железе. Но вопрос остаётся, как это работало в оригинале? Или это такую свинью подкинули в исходнике. Я хз.

#define F_CPU 8000000UL // Я использую кристалл 8 МГц

#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <avr/eeprom.h>

struct config_t {
  uint8_t cwl;            
  uint8_t cwh;            
  uint8_t tpsacold;       
  uint8_t divider;         
  uint8_t alternate;       
  uint8_t injpwm;         
  uint8_t injpwmt;
};

struct config_t config;
struct config_t config_ee EEMEM;

volatile uint8_t c=0;

volatile uint8_t eeprom_store_idx;     /* счетчик обрабатываемых байт данных*/
volatile uint8_t eeprom_store_busy;    /* флаг, eeprom занято*/
/*
  Эта функция запускает процесс сохранение структуры конфигурации из sram в 
  eeprom и немедленно возвращает управление,позволяя обработчику прерывания 
  TIMER0 продолжить работу по сохранению данных. */
uint8_t storeConfig(void) {

  if (eeprom_store_busy) {
    return -1;             /* eeprom занято, подождите! */
  }

  eeprom_store_idx = 0;      /* обнулить счетчик обрабатываемых байт данных*/
  eeprom_store_busy = 1;   /* установить флаг, eeprom занято*/
/*
  время выполнения этого прерывания может быть достаточно высоким в случае 
  самой плохой ситуации, это должно быть обязательно исследовано! */
  uint8_t *pe, *ps;
  uint8_t ebyte;

	while (eeprom_store_idx != sizeof(struct config_t)-1)
			{
  pe = (uint8_t *)&config_ee + eeprom_store_idx; /* адрес данных в eeprom */
  ps = (uint8_t *)&config + eeprom_store_idx;    /* адрес данных в sram */
  ebyte = eeprom_read_byte(pe);

  /* перезаписывать данные в eeprom будем только если это необходимо */
  /* сравним содержимое структур в sram и eeprom, выход из цикла 
   осуществляем по первому найденному различию в данных */    
  while( (ebyte == *ps) && (eeprom_store_idx != (sizeof(struct config_t) - 1)) ) {
    eeprom_store_idx++;
    pe++;
    ps++;
    ebyte = eeprom_read_byte(pe);
  }

  /* записываем новое значение в eeprom */
  if (ebyte != *ps)
    eeprom_write_byte(pe, *ps);
  /*
    если последняя запись в структуре не достигнута переходим к поиску 
   следующего различия данных в sram и eeprom */
  if (eeprom_store_idx == sizeof(struct config_t)-1) {    /* запись закончена */
      eeprom_store_busy = 0;                     /* сбросить флаг, eeprom занято*/
      PORTB &= ~_BV(PB7);                      /* погасить светодиод */
  } else {                                     /* переходим к следующему различию */
      eeprom_store_idx++;
      PORTB |= _BV(PB7);                      /* зажечь светодиод */
  /* 
    необходимо отметить, если светодиод горит более 5 сек следует снять питание 
   с контроллера иначе возможно повреждение eeprom */
  }
  			}
			return 0;
}

int main(void){

	DDRB |= _BV (PB7);      //установить как выход LED
	PORTB &=~ _BV (PB7);   //установить 0 LED

	config.cwl=50;         //0x32
	config.cwh=65;         //0x41
	config.tpsacold=155;   //0x9B
	config.divider=48;      //0x30
	config.alternate=4;      //0x04
	config.injpwm=94;      //0x5E
	config.injpwmt=223;      //0xDF

	sei();

   while(1){

		if (c==0)
		{
		c=1;
		storeConfig();
		}
   }
}