Mega48-20 не пишет в EEPROM, Что я забыл? Опции

[smk]

Давно уже с АВРками дела не имел может что-то не так. Есть устройство с Mega48-20. Тактовая 4 МГц. Фуз еепрома стоит правильно, запись разрешена. Функции чтения-записи из даташита. Не пишет в еепром, в т.ч. и на симуляторе тоже. Что это может быть, что я забыл? Прерывания запрещаю. Пишется из главного цикла. Читать получается (0хFF).

Код

void EEPROM_write(unsigned int Address, unsigned char Data)
{
  while(EECR & (1<<EEPE)); /* Wait for completion of previous write */
  EEAR = Address; /* Set up address and Data Registers */
  EEDR = Data;
  EECR |= (1<<EEMPE); /* Write logical one to EEMPE */
  EECR |= (1<<EEPE); /* Start eeprom write by setting EEPE */
}

unsigned char EEPROM_read(unsigned int Address)
{
  while(EECR & (1<<EEPE)); /* Wait for completion of previous write */
  EEAR = Address; /* Set up address register */
  EECR |= (1<<EERE); /* Start eeprom read by writing EERE */
  return EEDR; /* Return data from Data Register */
}

    while(1)
    {
    if(mode)
    {
      PORTC = 0b00010000;
      delay(30000);
      PORTC = 0b00000000;
      delay(30000);
    }

    if(mem)
    {
      asm("cli");      
      EEPROM_write(0x0A, mode);
      EEPROM_write(0x0B, output);
      mem = 0;
      asm("sei");
    }
    //проверить питание
    }//while

[Сергей Борщ]

Код

EECR |= (1<<EEMPE); /* Write logical one to EEMPE */
EECR |= (1<<EEPE); /* Start eeprom write by setting EEPE */

Между установкой EEMPE и установкой EEPE должно пройти не более 4 тактов. Посмотрите, во что вылились эти операторы с вашими настройками компилятора. А чтобы работало железно - не используйте тут '|='. Вы ведь точно знаете сотояние всех остальных битов, вот и пишите каждый раз все биты через '='.

[smk]

Код

EECR |= (1<<EEMPE); /* Write logical one to EEMPE */
EECR |= (1<<EEPE); /* Start eeprom write by setting EEPE */

Между установкой EEMPE и установкой EEPE должно пройти не более 4 тактов. Посмотрите, во что вылились эти операторы с вашими настройками компилятора. А чтобы работало железно - не используйте тут '|='. Вы ведь точно знаете сотояние всех остальных битов, вот и пишите каждый раз все биты через '='.

Компилятор GCC. Кстати оптимизация О0. Если включить другую, то код ужимается раза в два и более при этом перестает работать. Что-то я упускаю в своем понимании. А как кстати тогда правильно если не использовать '|='?

[Сергей Борщ]

Компилятор GCC.

Тогда используйте <avr/eeprom.h> и описанные в нем eeprom_write_byte(), eeprom_read_byte(). Там все проблемы времянок решены для всех уровней оптимизации:

Код

#include    <avr/eeprom.h>

void test()
{
    eeprom_write_byte((uint8_t *)0x0A, mode);
}

А как кстати тогда правильно если не использовать '|='?

примерно так:

Код

void EEPROM_write(unsigned int Address, unsigned char Data)
{
  while(EECR & (1<<EEPE)) /* Wait for completion of previous write */
      ;
  EEAR = Address; /* Set up address and Data Registers */
  EEDR = Data;
  EECR = (1<<EEMPE); /* Write logical one to EEMPE */
  EECR = (1<<EEMPE)|(1<<EEPE); /* Start eeprom write by setting EEPE */
}

[smk]

Тогда используйте <avr/eeprom.h> и описанные в нем eeprom_write_byte(), eeprom_read_byte(). Там все проблемы времянок решены для всех уровней оптимизации:

Код

#include    <avr/eeprom.h>

void test()
{
    eeprom_write_byte((uint8_t *)0x0A, mode);
}

примерно так:

Код

void EEPROM_write(unsigned int Address, unsigned char Data)
{
  while(EECR & (1<<EEPE)) /* Wait for completion of previous write */
     ;
  EEAR = Address; /* Set up address and Data Registers */
  EEDR = Data;
  EECR = (1<<EEMPE); /* Write logical one to EEMPE */
  EECR = (1<<EEMPE)|(1<<EEPE); /* Start eeprom write by setting EEPE */
}

Спасибо. Только я теперь не пойму что они в даташитах пишут и зачем? Раньше, помню, работало.

[IgorKossak]

Спасибо. Только я теперь не пойму что они в даташитах пишут и зачем? Раньше, помню, работало.

Это зависит от Вашего понимания даташитов. Если там пишут "установить бит в 1", то это вовсе не означает, что это надо делать операцией |=.
А вот по поводу количества тактов между операциями сказано вполне ясно.

[smk]

Это зависит от Вашего понимания даташитов. Если там пишут "установить бит в 1", то это вовсе не означает, что это надо делать операцией |=.
А вот по поводу количества тактов между операциями сказано вполне ясно.

Ну а какой операцией?

[IgorKossak]

Ну а какой операцией?

Операцией присваивания, как и советовал Сергей Борщ.

Нашел старую программу для ATmega128, все было сделано по ДШ и именно с "|=", работает без нареканий.

Листинг кода посмотрите. В Вашем случае могла быть некоторая неявная оптимизация, например, битовыми операциями (не помню, попадают ли эти регистры для работы с ними битовыми операциями). Но в любом случае следует избегать непереносимого кода и зависеть от трюков конкретной версии компилятора.

[Bear_ku]

Но в любом случае следует избегать непереносимого кода и зависеть от трюков конкретной версии компилятора.

Переносимость меня абсолютно не волнует. А вот "зависеть от трюков" это видимо действительно было, примеры в ДШ даны для IAR и поэтому в IAR работают без нареканий.

Ну а по поводу того, что программа перестает работать при включении оптимизации - это нормально. Проблему можно легко вычислить воспользовавшись средствами отладки. Есть подозрение что виновата переменная consol, попробуйте в ее объявление добавить квалификатор volatile.

Сообщение отредактировал Bear_ku - Dec 17 2014, 03:55

[demiurg1978]

Я делал один проект. Сначала взял ATMEGA8535. Проект писал в IAR. Когда прошил программу, все работало, кроме записи в EEPROM. Стал разбираться. Залил давний проект, написанный на асме. Запись в EEPROM работает. Стал сверять в дизасме. Вроде все нормально, а запись EEPROM не работает. Решил проверить эту же программу, но скомпилированную под ATMEGA32. Запись в EEPROM нормально работает. Выходит, что библиотеку записи в EEPROM нужно написать на асме. Но я в IAR так и не смог пока разобраться, как правильно писать и использовать функции на асме. Да и в тот раз обошелся ATMEGA32A. И некогда мне было с асмом в IAR разбираться.
Еще один случай. Умер один человек, мне достались его проекты. Написаны в IAR. Его девайсы сделаны на ATMEGA1280. Для сохранения данных использовались внешние EEPROM. То есть, когда-то этот человек тоже испытывал трудности с записью в EEPROM в проектах, созданных в IAR, на некоторых МК AVR.