Прошу помощи Опции

[Maks448]

Уважаемые форумчане, объясните, пожалуйста, "чайнику" в чем прикол (или прокол?). Программка простая, как два пальца, по идее должна вывести в порт RS232 удвоенное количество байтов от числа, находящегося в счетчике (регистр Z). Реально же выводится 4608, т.е. четвертая часть. Где я допустил ошибку? Пробовал также на Mega8 - результат тот же.

.include "2333def.inc"
.def Tmp =r16
.cseg
.org 0
reset:
clr Tmp
out UBRRH,Tmp
ldi Tmp,$19
out UBRRL,Tmp ;Настройка скорости СОМ-порта

clr Tmp
out UCSRA,Tmp
ldi Tmp,$18
out UCSRB,Tmp ;Настройка UART

in Tmp,WDTCR
ori Tmp,$18
out WDTCR,Tmp
clr Tmp
out WDTCR,Tmp ;Отключение WDT

ldi Tmp,$20
out DDRC,Tmp ;Определение линии порта С как выход
cbi PortC,5 ;Включить светодиод
ldi ZL,Low(9216)
ldi ZH,high(9216) ;Загрузка счетчика байтов
Loop:
subi ZL,1
sbci ZH,0 ;Декремент счетчика
Wait1:
sbis UCSRA,UDRE ;Ожидание готовности UART
rjmp Wait1
out UDR,ZH ;Вывод старшего байта счетчика
Wait2:
sbis UCSRA,UDRE ;Ожидание готовности UART
rjmp Wait2
out UDR,ZL ;Вывод младшего байта
cpi ZL,0
ldi Tmp,0
cpc ZH,Tmp ;Проверка счетчика на 0
brne Loop ;Не 0 - возврат
sbi PortC,5 ;Иначе - выключить светодиод и
Cycle:
rjmp Cycle ;зациклить программу

[=GM=]

Уважаемые форумчане, объясните, пожалуйста, "чайнику" в чем прикол (или прокол?). Программка простая, как два пальца, по идее должна вывести в порт RS232 удвоенное количество байтов от числа, находящегося в счетчике (регистр Z). Реально же выводится 4608, т.е. четвертая часть. Где я допустил ошибку? Пробовал также на Mega8 - результат тот же.

1) У вас есть одна логическая неувязочка вот здесь

Код

    cpi    ZL,0
    ldi    tmp,0
    cpc    ZH,tmp      ;Проверка счетчика на 0
    brne   loop        ;Не 0 - возврат

Команда "cpi ZL,0" никогда не установит бит переноса. Что вы тогда проверяете командой "cpc ZH,tmp"?
Только наличие 0 в ZH, но ведь в младшем регистре ZL еще останется 0xFF значений. Как-то некозяво выходит. Т.е. должно передаваться 2*Z-0xFF байт, а не удвоенное значение.

2) Логичнее было бы поставить команду "brne loop" после команды "cpi ZL,0".

[rx3apf]

Команда "cpi ZL,0" никогда не установит бит переноса. Что вы тогда проверяете командой "cpc ZH,tmp"?
Только наличие 0 в ZH, но ведь в младшем регистре ZL еще останется 0xFF значений.

Тормозим ? Команда cpi zl,0 управляет не только C, но и Z, а команда cpc использует значение Z до команды. Тут все правильно, хотя не самое компактное решение для проверки на 0.

[=GM=]

Команда "cpi ZL,0" никогда не установит бит переноса. Что вы тогда проверяете командой "cpc ZH,tmp"?
Только наличие 0 в ZH, но ведь в младшем регистре ZL еще останется 0xFF значений.

Тормозим?

По-моему, это вы поспешили..и меня немного посмешили(:-)

Команда cpi zl,0 управляет не только C, но и Z, а команда cpc использует значение Z до команды. Тут все правильно, хотя не самое компактное решение для проверки на 0.

1) Команда "cpi zl,0" в программе автора топика просто-напросто не нужна, т.к. результаты проверки нигде не используются.

2) К тому же, команда "cpi zl,0" НИКОГДА не установит бит переноса, который автор пытается использовать далее в команде "cpc ZH,tmp".

3) "...а команда cpc использует значение Z до команды". Сами-то поняли, что сказали(:=)? Это НЕПРАВИЛЬНАЯ проверка на 0.

4) Вот вам простой пример. На начало цикла Loop пришли к моменту, когда zh=1, zl=0x00. Вычитаем 1, получаем zh=0, zl=0xFF, передаем два байта, с Су=0 доходим до команды "cpc ZH,tmp", проверяем zh на равенство 0, да, zh=0, значит не возвращаемся на метку Loop, завершаем цикл. ПОЗВОЛЬТЕ, НО zl=0xFF, т.е. ОСТАЛОСЬ ПЕРЕДАТЬ 255 БАЙТ. Однако, цикл ОШИБОЧНО завершен.

5) Для проверки двухбайтного счетчика на 0 должна быть примерно такая конструкция

Код

       cpi     zh,0      ;старший байт счетчика=0?
       brne    loop      ;нет
       cpi     zl,0      ;младший байт счетчика=0?
       brne    loop      ;нет

Хотя можно написать и попроще, скажем так

Код

       sbiw    zl,1      ;счетчик=0?
       brne    loop      ;нет,

совместив декремент и проверку на 0

[rx3apf]

По-моему, это вы поспешили..и меня немного посмешили(:-)

"Над кем смеетесь ? Над собой смеетесь !" © классик.

1) Команда "cpi zl,0" в программе автора топика просто-напросто не нужна, т.к. результаты проверки нигде не используются.

В самом деле ?

2) К тому же, команда "cpi zl,0" НИКОГДА не установит бит переноса, который автор пытается использовать далее в команде "cpc ZH,tmp".

А нам очень нужен этот бит переноса ? Для команды breq ?

3) "...а команда cpc использует значение Z до команды". Сами-то поняли, что сказали(:=)? Это НЕПРАВИЛЬНАЯ проверка на 0.

Это вполне правильная по сути, хоть и не самая рациональная, проверка на 0. В предложении, пожалуй, надо было добавить "использует предыдущее значение Z". Или просто процитирую даташит:

R7 •R6• R5• R4 •R3 •R2 •R1• R0 •Z
Previous value remains unchanged when the result is zero; cleared otherwise.
При копировании, правда, потерян символ надчеркивания (инверсия).

4) Вот вам простой пример. На начало цикла Loop пришли к моменту, когда zh=1, zl=0x00. Вычитаем 1, получаем zh=0, zl=0xFF, передаем два байта, с Су=0 доходим до команды "cpc ZH,tmp", проверяем zh на равенство 0, да, zh=0, значит не возвращаемся на метку Loop, завершаем цикл. ПОЗВОЛЬТЕ, НО zl=0xFF, т.е. ОСТАЛОСЬ ПЕРЕДАТЬ 255 БАЙТ. Однако, цикл ОШИБОЧНО завершен.

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

5) Для проверки двухбайтного счетчика на 0 должна быть примерно такая конструкция

Код

       cpi     zh,0    ;старший байт счетчика=0?
       brne    loop    ;нет
       cpi     zl,0    ;младший байт счетчика=0?
       brne    loop    ;нет

Крайне нерационально. Как надо - я написал в одном из первых писем в этой ветке.

Сообщение отредактировал rx3apf - Feb 13 2007, 19:23

[=GM=]

Был не прав. Признаю. Флаг Z в команде cpc формируется не так, как в команде cpi. Флаг С ни при чём.