Помощь - Поиск - Пользователи - Календарь
Полная версия этой страницы: код для приема комманд от ПК через СОМ порт и отправки данных обратно
Форум разработчиков электроники ELECTRONIX.ru > Сайт и форум > В помощь начинающему > MCS51, AVR, PIC, STM8, 8bit
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 не работает вообще.
Лед в обработчике прерывания мигает всегда, как положено.
Сабж?
Непомнящий Евгений
откройте для себя volatile
777777
volatile u08 COMMAND не пробовал сделать?

P.S. А заодно написать command маленькими буквами - в Си принято большими обозначать константы.
AlexTech
volatile не пришло в голову попробовать, работало же, потом отладку убрал и перестало.
вечером попробую, спасибо большое ) для поддержания чистоты стиля даже буквы на маленькие изменю )))

Может быть подскажете как лучше всего организовать обмен c ПК в случае когда приходящая команда не из одного байта, а например установка времени, как минимум 3 байта.
Понятно что буфер нужен.
Но как удобнее/правильнее его формировать из одиноких приходящих байтов и парсить потом, как отслеживать начало и конец команды.
Да и честно говоря хотелось бы ответ на ПК отправлять сразу после получения команды, а не потом в цикле.
Пробовал switch (command) засунуть прямо в обработчик SIG_UART_RECV, однако несмотря на то, что UDR освобождается, usart не желает ничего отправлять до выхода из прерывания.
У меня есть идеи, но какие то кривенькие честно говоря, я уверен что работать будут, но хотелось бы красиво и быстро )))
GDI
Откройте для себя Wake или Modbus.
AlexTech
Спасибо, попробую Wake прикрутить, Modbus вроде тяжеловат для моих целей
AlexTech
В общем попробовал Wake прикрутить...
Поскольку С версия написана под IAR пришлось немного помудрить, я gcc использую
Ну кое как он заработал, однако при нормальной оптимизации из обработчика прерывания SIG_UART1_RECV напрочь вываливается вот этот важный кусок кода:
Код
  if(data_byte == FEND)    
  {
    Pc_Rx_Pre = data_byte;      
    Pc_Rx_Crc = CRC_INIT;        
    Pc_Rx_Sta = WAIT_ADDR;        
    Do_Crc8(data_byte, &Pc_Rx_Crc);
    return;                    
  }

не только в железе, но и в AVR Studio, там даже брэкпоинт не поставить :/
может быть и еще что то отваливается, но пока не проверить, что это такое может быть? (((
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];

Если кому надо, то вот:
demiurg_spb
Цитата(AlexTech @ Feb 18 2011, 00:58) *
В файле откуда будем использовать - надо объявить переменные:
Немного неправильно с идеологической точки зрения.
Все потроха должны быть сокрыты в недрах модуля, а пользователю лишь предоставить минимально необходимый и достаточный интерфейс через заголовочный файл в котором перечисляются прототипы доступных юзеру функций и если требуется extern переменные и прочие енумы.
Просить пользователя объявлять кучу переменных, как Вы предлагаете - по меньшей мере некрасиво...
AlexTech
Это не мое решение, так было задумано в оригинале, я не взялся исправлять, меня и так устраивает
это собственно и есть extern переменные
demiurg_spb
Цитата(AlexTech @ Feb 18 2011, 14:48) *
Это не мое решение, так было задумано в оригинале, я не взялся исправлять, меня и так устраивает
Вольному воля.
Цитата
это собственно и есть extern переменные
А вот это не понял.
Посмотрел приложенный Вами исходник, понял.
AlexTech
Ога, потому что сначала смотрим код, потом комментируем
Это написано было не для профи, они и так разберутся, да и сами напишут, это для таких как я, кто использует чужие драйвера и либы,
в которых профи отлично расписывают как это работает, но совершенно не объясняют как этим пользоваться
Леонид Иванович
Есть новая реализация Wake, котрая лучше инкапсулирует данные.
http://electronix.ru/forum/index.php?showt...st&p=891230

Работа с буфером ведется с помощью функций

char Wake_GetByte(void); //читает байт из буфера приема
int Wake_GetWord(void); //читает слово из буфера приема
long Wake_GetDWord(void); //читает двойное слово из буфера приема

и

void Wake_AddByte(char cool.gif; //помещает байт в буфер передачи
void Wake_AddWord(int w); //помещает слово в буфер передачи
void Wake_AddDWord(long dw); //помещает двойное слово в буфер передачи

Конечно, оверхед, но ради красоты на него можно пойти.
IgorKossak
Автор, если бы Вы удосужились дать более вразумительное название своей теме, нежели приевшуюся фразу про "лыжи", то и желающих Вам помочь было бы больше.
Ivan Panov
Цитата(AlexTech @ Feb 17 2011, 07:55) *
Есть вот такой код(ниже только кусок) для приема комманд от ПК через СОМ порт и отправки данных обратно:
u08 COMMAND;
switch ( COMMAND )


Проблема скорее всего в u8 для оператора switch - переменная COMMAND должна быть UINT. Не знаю, какой Вы использовали компилятор, но древний ИАР2.28 в подобном случае хранил адрес перехода в переменой того же типа, что и тип Вашей COMMAND, и соответственно делал переход только в пределах 256 байт.
AlexTech
Цитата(IgorKossak @ Feb 26 2011, 21:08) *
Автор, если бы Вы удосужились дать более вразумительное название своей теме, нежели приевшуюся фразу про "лыжи", то и желающих Вам помочь было бы больше.

Да, простите, в следующий раз буду нормально темы называть.
Однако если бы вы прочитали всю тему, то поняли бы что проблема в общем то давно решена и тема закрыта и желающих помочь нашлось в кол-ве ))

u8 для оператора switch - нормально все работает, не знаю че там с ИАРом, но gcc все правильно обрабатывает
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.
Invision Power Board © 2001-2025 Invision Power Services, Inc.