код для приема комманд от ПК через СОМ порт и отправки данных обратно Опции

[AlexTech]

Есть вот такой код(ниже только кусок) для приема комманд от ПК через СОМ порт и отправки данных обратно:

Код

u08 COMMAND;

UART_INTERRUPT_HANDLER(SIG_UART_RECV)
{
  COMMAND = inb(UDR);
  blink(); // мигаем ледом
}

int main(void)
{
  sei();
  // тут убраны разные инициализации

  while(1)
  {
    //
    // разная фигня
    //

    if ( COMMAND != 0x00 )
    {
      switch ( COMMAND )
      {
        case GET_TIME:
          uartSendDT(); //отправляем на ПК время с ds1307
          break;
        case GET_TEMP:
          uartSendTemp(); //отправляем на ПК температуру с ds1820
          break;
      }//end switch
      COMMAND = 0x00;
    }
    _delay_ms(5);
  }//end loop

}// end main

Так вот, если вместо "разная фигня" есть какой то код, то все работает,
если же там пусто, то при опции оптимизатора -О1 работает от случая к случаю, при -О2,О3,Оs не работает вообще.
Лед в обработчике прерывания мигает всегда, как положено.
Сабж?

Сообщение отредактировал AlexTech - Feb 17 2011, 07:08

[AlexTech]

В общем полностью переписал в gcc нотации используя куски оригинального кода, заменил все char на uint8_t, добавил поддержку 2х usartов для 128 меги,
добавил volatile где явно не хватало, изменил функцию инициализации, все заработало.
В файле откуда будем использовать - надо объявить переменные:

Код

volatile uint8_t Command;

volatile uint8_t Rx_Cmd,
      Rx_Cnt,
      Rx_Data[FRAME];

volatile uint8_t Tx_Cmd,
      Tx_Cnt,
      Tx_Data[FRAME];

Если кому надо, то вот:

Сообщение отредактировал AlexTech - Feb 18 2011, 12:51

Прикрепленные файлы

 wake.rar ( 3.42 килобайт ) Кол-во скачиваний: 41

 

[demiurg_spb]

В файле откуда будем использовать - надо объявить переменные:

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