Форум разработчиков электроники ELECTRONIX.ru > Проблема: спящий режим+ USART

Ruslan Konovalov

Oct 7 2008, 03:33

Всем добрый день.

Возникла проблема при использовании спящего режима(Idle mode) в Atmega8.
Вот код(не полный):

Код

void main(void)
{
  MCUCR |= (0<<SM2)|(0<<SM1)|(0<<SM0); //Idle mode
  init_Usart();
  __enable_interrupt();

for(;;)
{
   MCUCR |= (1<<SE); //Разрешение спящего режима
   asm("sleep");  //Сон
   Usart_read(); //Читаем строку
   Usart_Send_Str(UsartBuff); //Возвращаем принятое
}
}

Проблема заключается в том, что при использовании спящего режима Usart читает входящие данные нормально, за исключением первого байта.
Я так понимаю он пропускает его из-за того что процессор просыпается и начинает нормально работать в момент прихода 2-го байта.
Как решить эту проблему ?

А что если соеденить ножку RX и INT0 и будить процессор по этому прирыванию ?
Может кто нибудь уже с этим сталкивался ?
Заранее спасибо.

Николай Иванович Приходько

Oct 7 2008, 06:36

Цитата(Ruslan Konovalov @ Oct 7 2008, 07:33)

Проблема заключается в том, что при использовании спящего режима Usart читает входящие данные нормально, за исключением первого байта.

Ну так организуйте протокол так, чтобы первый байт не нёс никакой инфы, а использовался только для "побудки" микроконтроллера. В чём проблема-то?

Ruslan Konovalov

Oct 7 2008, 07:06

Цитата(Николай Иванович Приходько @ Oct 7 2008, 09:36)

Ну так организуйте протокол так, чтобы первый байт не нёс никакой инфы, а использовался только для "побудки" микроконтроллера. В чём проблема-то?

К сожалению изменить протокол нет возможности.

VladimirYU

Oct 7 2008, 08:07

Цитата(Ruslan Konovalov @ Oct 7 2008, 07:33)

Всем добрый день.

Возникла проблема при использовании спящего режима(Idle mode) в Atmega8.
Вот код(не полный):

Код

void main(void)
{
  MCUCR |= (0<<SM2)|(0<<SM1)|(0<<SM0); //Idle mode
  init_Usart();
  __enable_interrupt();

for(;;)
{
   MCUCR |= (1<<SE); //Разрешение спящего режима
   asm("sleep");  //Сон
   Usart_read(); //Читаем строку
   Usart_Send_Str(UsartBuff); //Возвращаем принятое
}
}

Проблема заключается в том, что при использовании спящего режима Usart читает входящие данные нормально, за исключением первого байта.
.

Idle mode enables the MCU to wake up from external triggered interrupts as well as
internal ones like the Timer Overflow and USART Transmit Complete interrupts. If
wake-up from the Analog Comparator interrupt is not required, the Analog Comparator
can be powered down by setting the ACD bit in the Analog Comparator Control and Status
Register – ACSR. This will reduce power consumption in Idle mode. If the ADC is
enabled, a conversion starts automatically when this mode is entered.

Мне кажется функцию приема нужно организовать по прерыванию от RXC.

Цитата(VladimirYU @ Oct 7 2008, 12:04)

Idle mode enables the MCU to wake up from external triggered interrupts as well as
internal ones like the Timer Overflow and USART Transmit Complete interrupts. If
wake-up from the Analog Comparator interrupt is not required, the Analog Comparator
can be powered down by setting the ACD bit in the Analog Comparator Control and Status
Register – ACSR. This will reduce power consumption in Idle mode. If the ADC is
enabled, a conversion starts automatically when this mode is entered.

Мне кажется функцию приема нужно организовать по прерыванию от RXC.

Немного не так выделил.

Ruslan Konovalov

Oct 7 2008, 23:53

Цитата(VladimirYU @ Oct 7 2008, 11:07)

Мне кажется функцию приема нужно организовать по прерыванию от RXC.

Все верно.
Функция Usart_read(); выглядит примерно так:

Код

void Usart_read(void)
{
  while(!received);
}

//А вот само прерывание

#pragma vector = USART_RXC_vect
__interrupt void Recieve()
{
  unsigned char temp = 0;

  temp = UDR;
  UsartBuff[UsartBuffPtr] = temp;
  UsartBuffPtr++;
  if(temp == 0x0D) received = 1; //0x0D - конец принимаемых данных.
}

VladimirYU

Oct 8 2008, 05:01

Цитата(Ruslan Konovalov @ Oct 8 2008, 03:53)

Все верно.
Функция Usart_read(); выглядит примерно так:

Код

void Usart_read(void)
{
  while(!received);
}

//А вот само прерывание

#pragma vector = USART_RXC_vect
__interrupt void Recieve()
{
  unsigned char temp = 0;

  temp = UDR;
  UsartBuff[UsartBuffPtr] = temp;
  UsartBuffPtr++;
  if(temp == 0x0D) received = 1; //0x0D - конец принимаемых данных.
}

Действительно, я примерно делаю также. Попробуйте убрать спячку (заменить, например, на while с условием выхода по приему первого байта пакета ), и убедиться, что потеря 1-го байта связана именно с ней.

Ruslan Konovalov

Oct 9 2008, 01:43

Цитата(VladimirYU @ Oct 8 2008, 08:01)

Действительно, я примерно делаю также. Попробуйте убрать спячку (заменить, например, на while с условием выхода по приему первого байта пакета ), и убедиться, что потеря 1-го байта связана именно с ней.

Уже пробовал.
Без спячки все прекрасно работает.
И еще, как я понял процессор просыпается от прерывания по RXC только в режиме Idle.
Пробовал другие режимы. В них процессор спит.

VladimirYU

Oct 9 2008, 05:11

Цитата(Ruslan Konovalov @ Oct 9 2008, 05:43)

Уже пробовал.
Без спячки все прекрасно работает.
И еще, как я понял процессор просыпается от прерывания по RXC только в режиме Idle.
Пробовал другие режимы. В них процессор спит.

Idle mode enables the MCU to wake up from external triggered interrupts as well as
internal ones like the Timer Overflow and USART Transmit Complete interrupts.

Доки от Атмела, как русский язык по Черономырдину. Ни слова про RXC, является ли оно тем, что выделено, или речь только об интах? Ипользовал сам этот режим, но не помню подробностей, вроде делал в принципе все как у Вас, попробую паоднять старый проект, правда он был на меге128. Но это не принципиально. Может быть задействовал еще одну ногу.

Ruslan Konovalov

Oct 9 2008, 12:59

Спасибо всем, кто откликнулся.
Оказалось, что это я как всегда напартачил.
Вообщем стыдно даже говорить где была проблема(в кривых руках)

А RXC прерывание работает корректно и с Idle mode.
Ошибку исправил и все заработало.
Простите что ввел Вас в заблуждение.
Как говориться: "Прежде чем искать баги производителя, поищи свои".
Выражаю всем благодарность.
Тему можно закрыть.

GDI

Oct 9 2008, 13:11

Все же поборите свой стыд и расскажите в чем была проблема, чтоб другие не напоролись.

VladimirYU

Oct 10 2008, 05:19

Цитата(GDI @ Oct 9 2008, 17:11)

Все же поборите свой стыд и расскажите в чем была проблема, чтоб другие не напоролись.

+1.

Ruslan Konovalov

Oct 13 2008, 07:37

Ладно.
Было вот так:

Код

void main(void)
{
  MCUCR |= (0<<SM2)|(0<<SM1)|(0<<SM0); //Idle mode
  init_Usart();
  __enable_interrupt();
  UsartRxOn = 0;
  for(;;)
{
   MCUCR |= (1<<SE); //Разрешение спящего режима
   asm("sleep");  //Сон
   UsartRxOn = 1;//Разрешаем прием байтов
   Usart_read(); //Читаем строку
   UsartRxOn = 0; //Запрещаем прием байтов
   Usart_Send_Str(UsartBuff); //Возвращаем принятое
}
}

void Usart_read(void)
{
  while(!received);
}

//А вот само прерывание

#pragma vector = USART_RXC_vect
__interrupt void Recieve()
{
  unsigned char temp = 0;

  temp = UDR;
  if(UsartRxOn) //Если прием байтов разрешен
   {
  UsartBuff[UsartBuffPtr] = temp;
  UsartBuffPtr++;
  if(temp == 0x0D) received = 1; //0x0D - конец принимаемых данных.
  }
}

Вся проблема была в том, что процессор просыпается по прерыванию RXC, но на тот момент
UsartRxOn = 0. И по этому 1-й байт в UsartBuff не заносился.
Я просто поставил UsartRxOn = 1; перед функции засыпания и все исправилось.
Но по првильному вместо UsartRxOn лучше делать так:

Код

UCSRB |= (1 << RXCIE);//Разрешаем прерывание
UCSRB |= (1 << RXCIE);//Запрещаем прерывание

Вот такие кривые ручки.

VladimirYU

Oct 13 2008, 09:09

[/quote]
UCSRB |= (1 << RXCIE);//Разрешаем прерывание
UCSRB |= (1 << RXCIE);//Запрещаем прерывание
[/code]
Вот такие кривые ручки.

[/quote]

А найдите два отличия в Ваших строчках

Ruslan Konovalov

Oct 13 2008, 10:11

Цитата(VladimirYU @ Oct 13 2008, 12:09)

UCSRB |= (1 << RXCIE);//Разрешаем прерывание
UCSRB |= (1 << RXCIE);//Запрещаем прерывание
[/code]
Вот такие кривые ручки.

А найдите два отличия в Ваших строчках

CTRL+C и CTRL+V рулит.

Надо так:
UCSRB |= (1 << RXCIE); //Разрешаем прерывание
UCSRB &=~ (1 << RXCIE); //Запрещаем прерывание

А вот теперь думую все.

defunct

Oct 13 2008, 17:43

Цитата(Ruslan Konovalov @ Oct 7 2008, 06:33)

А что если соеденить ножку RX и INT0 и будить процессор по этому прирыванию ?
Может кто нибудь уже с этим сталкивался ?

Просыпается от прерывания UART'а и нормально вытаскиваются все данные. Ничего не теряется.
t2313, 90S2313, m8, m48/88/168.
Другие МК с UART'ами у меня не спят.

Цитата(VladimirYU @ Oct 9 2008, 08:11)

Idle mode enables the MCU to wake up from external triggered interrupts as well as
internal ones like the Timer Overflow and USART Transmit Complete interrupts.

Доки от Атмела, как русский язык по Черономырдину. Ни слова про RXC, является ли оно тем, что выделено, или речь только об интах?

external triggered interrupts as well as internal ones

Написано же - пробуждается как внешними так и внутренними источниками прерываний (т.е. любым прерыванием). Или для тех кто "в танке" надо перечислить каждый источник прерываня для каждого проца?

Цитата(Ruslan Konovalov @ Oct 13 2008, 10:37)

Вся проблема была в том, что процессор просыпается по прерыванию RXC, но на тот момент
UsartRxOn = 0. И по этому 1-й байт в UsartBuff не заносился.

Еще раз убеждаемся, что по чрезмерно сокращенным огрызкам кода (как в самом первом посте) реальную проблему найти просто невозможно.
Приводить нужно кусочек реального проекта по принципу "как есть", разве только имена переменных поменять для наглядности.