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

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

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

откройте для себя volatile

volatile u08 COMMAND не пробовал сделать?

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

Сообщение отредактировал 777777 - Feb 17 2011, 06:19

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

Может быть подскажете как лучше всего организовать обмен c ПК в случае когда приходящая команда не из одного байта, а например установка времени, как минимум 3 байта.
Понятно что буфер нужен.
Но как удобнее/правильнее его формировать из одиноких приходящих байтов и парсить потом, как отслеживать начало и конец команды.
Да и честно говоря хотелось бы ответ на ПК отправлять сразу после получения команды, а не потом в цикле.
Пробовал switch (command) засунуть прямо в обработчик SIG_UART_RECV, однако несмотря на то, что UDR освобождается, usart не желает ничего отправлять до выхода из прерывания.
У меня есть идеи, но какие то кривенькие честно говоря, я уверен что работать будут, но хотелось бы красиво и быстро )))

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

Откройте для себя Wake или Modbus.

Спасибо, попробую Wake прикрутить, Modbus вроде тяжеловат для моих целей

В общем попробовал Wake прикрутить...
Поскольку С версия написана под IAR пришлось немного помудрить, я gcc использую
Ну кое как он заработал, однако при нормальной оптимизации из обработчика прерывания SIG_UART1_RECV напрочь вываливается вот этот важный кусок кода:

не только в железе, но и в AVR Studio, там даже брэкпоинт не поставить :/
может быть и еще что то отваливается, но пока не проверить, что это такое может быть? (((

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

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

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

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

Это не мое решение, так было задумано в оригинале, я не взялся исправлять, меня и так устраивает
это собственно и есть extern переменные

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

Вольному воля.
А вот это не понял.
Посмотрел приложенный Вами исходник, понял.

Ога, потому что сначала смотрим код, потом комментируем
Это написано было не для профи, они и так разберутся, да и сами напишут, это для таких как я, кто использует чужие драйвера и либы,
в которых профи отлично расписывают как это работает, но совершенно не объясняют как этим пользоваться

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

Есть новая реализация 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 ; //помещает байт в буфер передачи
void Wake_AddWord(int w); //помещает слово в буфер передачи
void Wake_AddDWord(long dw); //помещает двойное слово в буфер передачи

Конечно, оверхед, но ради красоты на него можно пойти.

Автор, если бы Вы удосужились дать более вразумительное название своей теме, нежели приевшуюся фразу про "лыжи", то и желающих Вам помочь было бы больше.

Проблема скорее всего в u8 для оператора switch - переменная COMMAND должна быть UINT. Не знаю, какой Вы использовали компилятор, но древний ИАР2.28 в подобном случае хранил адрес перехода в переменой того же типа, что и тип Вашей COMMAND, и соответственно делал переход только в пределах 256 байт.