Кто Atmel на с програмировать умеет?

[maxx™]

Есть вот такой кусок кода (опрос клавиатуры):
unsigned char LookKey()
{
unsigned char i,j;

for (i=0;i<4;i++)
{
if (i == 0) KEYPORT = (unsigned char)~ROW1;
if (i == 1) KEYPORT = (unsigned char)~ROW2;
if (i == 2) KEYPORT = (unsigned char)~ROW3;
if (i == 3) KEYPORT = (unsigned char)~ROW4;

for (j=0;j<255;j++); // Delay

if (i == 3){iDebug = (KEYPIN & (COL1+COL2+COL3+COL4)); }

if ( (KEYPIN & (COL1+COL2+COL3+COL4)) != (COL1+COL2+COL3+COL4))
{
i = i * 4;
if ((KEYPIN & COL1) != COL1) i = i + 1;
if ((KEYPIN & COL2) != COL2) i = i + 2;
if ((KEYPIN & COL3) != COL3) i = i + 3;
if ((KEYPIN & COL4) != COL4) i = i + 4;
return Table[i-1];
}
}
return 0;
}

Если закоментировать строчку if (i == 3){iDebug = (KEYPIN & (COL1+COL2+COL3+COL4)); } то не работает, с ней работает. Переменная iDebug нигде больше не используется. Не работает - это не опрашивается клавиатура при нажатии клавиш на линии ROW4. Остальные клавиши работают при любом раскладе. В чем проблема, есть идеи?

[Newton]

Может Delay короткий? обычно для такой ерунды что-то типа sleep используется.

[Hose]

Забавно.
Я бы, в порядке эксперимента,
сперва сделал j volatile, а потом бы еще дважды цикл прокрутил, для верности(если тактовая частота не настолько низка).

[CrazySquirrel]

Может _delay_ms поставить?
Какой МК?

[maxx™]

Может _delay_ms поставить?
Какой МК?

at90s8515 8 МГц. Оно-то конешно хорошо, но весь прикол в том что остальные 9 кнопок опрашиваются нормально. Но увеличить цикл или сделать delay попробую.

[{Totoro}]

Видно помогла задержка

[maxx™]

Видно помогла задержка

К сожалению не помогла. Пока так и оставил.

[{Totoro}]

Стряяяннно.... а если убрать if (i == 3) и оставить iDebug = (KEYPIN & (COL1+COL2+COL3+COL4));?

или можно попробовать убрать всю строку if (i == 3){iDebug = (KEYPIN & (COL1+COL2+COL3+COL4)); } и поменять местами строки
if (i == 3) KEYPORT = (unsigned char)~ROW4;
и
if (i == 0) KEYPORT = (unsigned char)~ROW1;
...
если не будет реакции теперь на первую строку кнопок - должно быть дело всё-таки в задержке

[maxx™]

Стряяяннно.... а если убрать if (i == 3) и оставить iDebug = (KEYPIN & (COL1+COL2+COL3+COL4));?

А смысл? Тогда клавиатура точно опрашиваться не будет, кто тогда значение в порт правильное пропишет?

[{Totoro}]

А смысл? Тогда клавиатура точно опрашиваться не будет, кто тогда значение в порт правильное пропишет?

С какого перепугу? ты в порт пишешь KEYPORT = ... а опрашиваешь KEYPIN
Эта строка выполняется только тогда, когда у тебя в самом большом цикле i==3

[{Totoro}]

если не используешь _delay_ms твой пустой цикл компилятор похерит если включена оптимизация кода. Лист смотрел?
Нарисуй на всякий for (j=0;j<255;j++) {_NOP;}; // Delay

[maxx™]

если не используешь _delay_ms твой пустой цикл компилятор похерит если включена оптимизация кода. Лист смотрел?
Нарисуй на всякий for (j=0;j<255;j++) {_NOP;}; // Delay

Я даже просто delay_ms делал. Это или какой-то глюк компилятора или еще что-то. Впрочем клавиатура это третье дело, работает пока и ладно. Вот когда все заработает тогда может и вернуть к этому вопросу.

[Hose]

К сожалению не помогла. Пока так и оставил.

Прикольно!
С интересом посмотрел бы ассемблерный листинг данного чуда.
Может оптимизатор. Еще, для подобного случая, есть крайне маловероятный, почти сказочный вариант - что-то с порчей данных в ОЗУ. В последнем случае, данный эффект достигается за счет сдвига адресов(стека), при неиспользовании переменной iDebug.

[maxx™]

Прикольно!
С интересом посмотрел бы ассемблерный листинг данного чуда.
Может оптимизатор. Еще, для подобного случая, есть крайне маловероятный, почти сказочный вариант - что-то с порчей данных в ОЗУ. В последнем случае, данный эффект достигается за счет сдвига адресов(стека), при неиспользовании переменной iDebug.

Ну компилятор в этом очевидно и виноват. Потому как в оригинале это было написано на IAR, а я переделываю под WinAVR.