Операция сдвига в CodeVisionAvr

[Admin2]

Не могу понять, верно написана часть кода, или нет. Это мой первый проект на CodeVisionAvr и на языке Си. Задача следующая: восемь раз считываю состояние первых трех пинов порта С. Состояние каждого из пинов является одним битом одной из трех переменных (Razr0, Razr1, Razr2). Т.е. нулевой бит порта С за восемь считываний должен сформировать байт данных в в переменную Razr0,Ю и т.д. Насколько я понимаю, без сдвиговых операций здесь не обойтись.

Написал, проверил на железе, не работает. Что в этом коде не так?

...
Counter++;
if (Counter<=8)
{
Razr0+=PINC.0;
Razr0=(Razr0 << 1); 
Razr1+=(PINC.1>>1);
Razr1=(Razr1 << 1); 
Razr2+=(PINC.2>>2);
Razr2=(Razr2 << 1);    
}
..

Может подскажете как в CodeVisionAvr выполнить трассировку? Вообще мне советуют перейти на WinAvr. Установил его, но там тоже не увидел ни отладки, ничего...

[ROD]

Razr0+=PINC.0;
Razr0=(Razr0 << 1);

В строке "Razr0+=PINC.0;" - Razr0 будет равен нулю, если PINC.0 был в низком уровне, и будет равен единице, если PINC.0 был высоком уровне.
Соответственно на каждом шаге цикла (у вас ведь там снаружи где-то цикл, да?) вы затираете переменную Razr0. И в конце она будет иметь вид либо 0000 0010, если на последнем проходе PINC.0 был в высоком уровне, либо вовсе будет равна нулю, если PINC.0 на последнем проходе был в низком уровне.

Попробуйте писать вот так:

buf=PINC.0;//Разумеется вы buf в начале программы объявите.
Razr0+=Razr0 || buf;
Razr0=(Razr0 << 1);

Или даже так:

Razr0+=Razr0 || PINC.0;
Razr0=(Razr0 << 1);

Все тоже самое касается Razr1, и Razr2.


Что касается трассировки, в первую очередь нужно:
1. Что бы у вас в фьюзах было зашито использование JTAG (обращаю внимание, что, например, у Atmega16 JTAG находится на порту C). На всякий случай обращу внимание, что фьюзы без ума трогать не надо.
2. Что бы в вашей отладочной плате (или готовой вашей схеме) JTAG был выведен и правильно подключен к компу.
3. Что бы ваша отладочная плата вообще поддерживалась CodeVisionAvr. В противном случае вообще забудьте об отладке. (именно по этому вас советуют WinAvr - он в этом плане более гибкий, вроде бы. Но я им не пользовался, так что не могу утверждать наверняка.)

[Admin2]

Да, у меня там цикл снаружи.

А эти два кода одинаково будут работать, если я считываю не нулевой, а первый или второй биты порта С? Насколько я понимаю, buf присвоит значение всего байта (а не второго бита). Поэтому его еще нужно будет дополнительно сдгигать вправо. Что будет, если PINC=00000010?

buf=PINC.1;
Razr1+=Razr1 || buf;
Razr1=(Razr1 << 1);

И:

Razr1+=Razr1 || PINC.1;
Razr1=(Razr1 << 1);

А как насчет чисто программной трассировки без отладочной платы?

[Admin2]

Сообщение от ROD

В строке "Razr0+=PINC.0;" - Razr0 будет равен нулю, если PINC.0 был в низком уровне, и будет равен единице, если PINC.0 был высоком уровне.
Соответственно на каждом шаге цикла (у вас ведь там снаружи где-то цикл, да?) вы затираете переменную Razr0. И в конце она будет иметь вид либо 0000 0010, если на последнем проходе PINC.0 был в высоком уровне, либо вовсе будет равна нулю, если PINC.0 на последнем проходе был в низком уровне.

Не свосем понятно почему я затираю. Я же плюсик поставил: Насколько я понимаю, к предыдущему значению Razr0 будет добавлено новое значение (PINC.0). Добавлено, а не просто присвоено. Разве не так? если Razr0=0b00001000 и PINC.0=0b00000001, после выполнения команды
Razr0+=PINC.0;
я ведь должен получить Razr0=0b00001001?

[ROD]

Плюсик спросни не увидел. К стати... сдвигать надо до считывания. Иначе мы левый бит "выкинем" при последнем проходе.

К стати, а что конкретно у вас на железе не работает, т.е. какой результат вы в итоге получаете?

[Admin2]

Все заработало, спасибо. Это я загнался...