или подключить к земле, - используя параметр INPUT_PULLUP?
Это конечно полная ерунда написана. От непонимания.или может все незадействованные ноги ардуино посадить на минус, чтобы не ловили помехи какие-нибудь
Может. Попробуйте для начала подключить коротким проводом его. Чтобы исключить остальные проблемы.3 метра подключен к ардуине, может из за этого виснуть?
Код: Выделить всё
ISR(TIMER1_OVF_vect){
static uint8_t entry;
static uint8_t pos;
static pos_t *scr = screen;
regs_t regs;
// считаем входы
if(++entry >= IND_RPT){
entry = 0;
// каждые 2 мс
two_ms++;
// смена разряда
scr++;
if(++pos >= POS_CNT){
pos = 0;
scr = screen;
}
regs.word = _BV(11+pos);
}
// готовим один из двух символов для вывода
regs.word &= 0xf800;
regs.word |= digs[scr->symbol[entry >= scr->bright]];
// выдача в сдвиговые регистры через SPI
SPDR = regs.bytes[1];
while(bit_is_clear(SPSR, SPIF));
SPDR = regs.bytes[0];
while(bit_is_clear(SPSR, SPIF));
// строб для защелкивания данных
PORTB |= LOAD_PIN;
PORTB &= ~LOAD_PIN;
}
то есть когда от прерывания к прерыванию я сохраняю состояние структуры, т.е. значение битов, управляющих разрядами индикатора, это не только не делает нерабочий код рабочим, но и полностью рушит индикацию!
Код: Выделить всё
typedef union{
uint8_t bytes[2]; // 2 байта для выдачи в регистры
uint16_t word;
} regs_t;
да, но что в таком случае может происходить? только то, что основной цикл станет выполняться очень медленно, а динамическая индикация продолжится, хоть и с некоторыми артефактами вроде разной яркости разрядов.WiseLord писал(а):Но не может ли быть так, что доступ к "статической" структуре получается с точки зрения ассемблера более длинным, что в итоге является той каплей, что прерывание не успевает выполниться за нужное время?
а вы посмотрите внимательно:jcxz писал(а):Что тут непонятного?
SPI работает на частоте вдвое меньше тактовой, т.е. передача 1 байта требует 16 тактов - пока исполнится код, организующий ожидание, пройдет половина этого интервала.jcxz писал(а):Ожидание завершения передачи по SPI в while() внутри ISR - это конечно верх быдлокодинга
Код: Выделить всё
regs.word &= 0xf800;
regs.word |= digs[scr->symbol[entry >= scr->bright]];использование static заставляет компилятор сделать загрузку 2 байт в структуру при входе в функцию и сохранение этих 2 байт при выходе, только и всего, никакой другой оптимизации он не делает. не должен делать, во всяком случае.technik-1017 писал(а):Использование static заставляет компилятор оптимизировать данный код (скорее всего он выкидывает часть)
как минимум в том, что в обычном случае добавление static к любой локальной переменной никаких побочных эффектов, кроме увеличения статически выделенной памяти, давать не должно, т.е. что со static, что без, поведение программы должно быть неизменным. в моём случае это не так.jcxz писал(а):А что значит "со static не работает"? В чём это выражается "не работает"?
невероятный вариант: локальная переменная потому и локальная, что более нигде не используется, хоть со static, хоть без. а всякие "опасные" действия (с указателями) у меня отсутствуют в коде.jcxz писал(а):Например - когда делаете static, то в другом месте другой код, у вас эту область памяти портит. Потому и не работает. Как вариант.
для локальной static-переенной это присваивание делается один раз, по идее вы не должны были спрашивать это, это ведь азы Си. для первого входа в обработчик надо, чтобы указатель указывал на начало массива, а в ходе обработки (см. код) этот указатель сдвигается.jcxz писал(а):Например - что такое static pos_t *scr = screen; и зачем ему (static-переменной scr) на каждом входе в функцию присваивается (переменная/константа/etc.(?)) screen? Какой тут смысл делать статик если по использованию тут нужна обычная автоматическая переменная?
ну, это, наверное, сделать можно, хотя фактически получится, что состояние индикаторов будет отставать от того, что делает обработчикjcxz писал(а):ожидания делать не после записей, а перед. А строб перенести в начало ISR. Это сразу уменьшает ожидание внутри ISR на эти самые 16 тактов.
но вряд ли это критично... 16 тактов, я имею ввиду.
выбирайте, я не запрещаю.jcxz писал(а):я бы выбрал другой МК
так о том и речь: когда эта переменная БЕЗ static, т.е. содержит мусор на входе в функцию, все работает, а когда я делаю её static, т.е. она на первом входе уже инициализирована нулями - работать перестаёт...technik-1017 писал(а):переменная regs вроде должна быть (задумано) как static ну и также не инициализирована
Код: Выделить всё
static pos_t *scr = screen;как минимум в том[/uquote]Вы не поняли вопрос. Перечитайте ещё раз.jcxz писал(а):А что значит "со static не работает"? В чём это выражается "не работает"?
И что что "не используется"? Она в памяти присутствует? Рядом с ней другие переменные (static или глобальные) присутствуют? А раз присутствуют, значит любая неправильная операция с памятью может её порушить.ARV писал(а):невероятный вариант: локальная переменная потому и локальная, что более нигде не используется, хоть со static, хоть без.jcxz писал(а):Например - когда делаете static, то в другом месте другой код, у вас эту область памяти портит. Потому и не работает. Как вариант.
Ну да, а ещё вы пишете абсолютно безглючный код, который никогда не глючит.ARV писал(а):а всякие "опасные" действия (с указателями) у меня отсутствуют в коде.
Зачем тогда вопрос задали раз у вас ничего не глючит?А вам по идее с этого места надо взять учебник по си и начать изучать азы. Ибо смотрим в листинг (IAR):ARV писал(а):для локальной static-переенной это присваивание делается один раз, по идее вы не должны были спрашивать это, это ведь азы Си.
Код: Выделить всё
inline void CpuUseTimeTick()
{
_Z14CpuUseTimeTickv: (+1)
...
//static u16 t = cpuUseCalc;
0x4821 LDR.N R0,??CpuUseTimeTick_0+0x8
0x78C0 LDRB R0,[R0, #+3]
0x8020 STRH R0,[R4, #+0]
...
Или всё-таки вы их не знаете? Код: Выделить всё
static pos_t *scr = screen;