Проблема с динамической индикацией

[ARTEM277]

Здравствуйте форумчане !
Нужна помощь начинающему в СИ.За основу был взят код из интернета.
Задача довольно простая .Нужно вывести число 1234 на четырёхразрядный семисегментный индикатор.Контроллер используется ATmega 8535. Проблемка следующая ... Если выводить число 1111 то высвечивается все четко.Но стоит в переменную display1 записать 1234 то всё смешивается.Собрал в протеусе ,все работает нормально а в реальной плате никак.Бродя по интернету я понял ,что это классическое построение программы.Где же ошибка ?



#include ‹avr/io.h›
#include ‹avr/interrupt.h›
#include ‹util/delay.h›
//------------------0-----1-----2-----3-----4-----5-----6-----7-----8------9--
char SEGMENT[ ] = {0x3F, 0x06, 0x5B, 0x4F, 0x66, 0x6D, 0x7D, 0x07, 0x7F, 0x6F};
volatile unsigned char segcounter = 8;
volatile int display1 = 0;
// Обработчик прерывания по переполнению таймера2
ISR (TIMER2_OVF_vect)
{
PORTD = 0xFF; // Гасим все разряды
PORTB = (8 ‹‹ segcounter) ; Двигаем 1 начиная с PB3

switch (segcounter)
{
case 0:
PORTD = ~(SEGMENT[display1 % 10000 / 1000]); // раскладываем числа
break;
case 1:
PORTD = ~(SEGMENT[display1 % 1000 / 100]);
break;
case 2:
PORTD = ~(SEGMENT[display1 % 100 / 10]);
break;
case 3:
PORTD = ~(SEGMENT[display1 % 10]);
break;
}
if ((segcounter++) › 2) segcounter = 0 ;



}
// Главная функция
int main (void)
{
DDRB |= (1 ‹‹ PB6)|(1 ‹‹ PB5)|(1 ‹‹ PB4)|(1 ‹‹ PB3); //настраиваем на выход порт B бегущая еденица выбирает индикатор
PORTB = 0x00;

DDRD |= (1 ‹‹ PD6)|(1 ‹‹ PD5)|(1 ‹‹ PD4)|(1 ‹‹ PD3)|(1 ‹‹ PD2)|(1‹‹PD1)|(1 ‹‹ PD0); //настройка порта для вывода кода числа

PORTD = 0x00;

TIMSK |= (1 ‹‹ TOIE2); // Разрешение прерывания по таймеру2
TCCR2 |= (1 ‹‹ CS22)|(0 ‹‹ CS21)|(0 ‹‹ CS20); // Предделитель на 8
sei(); // Глобально разрешаем прерывания
while(1)
{

display1=1234;

}
}

[GMP]

А какая частота обновления у вас? Попробуйте ее уменьшить.

[pictele]

Включите гашение по всем портам.

[Wiza_]

volatile unsigned char segcounter = 8;

PORTB = (8 ‹‹ segcounter) ; Двигаем 1 начиная с PB3

Вы наверно хотели сказать: volatile unsigned char segcounter = 0;
Иначе не совсем понятно куда будет вывод PORTB = (8 ‹‹ 8 )

if ((segcounter++) › 2) segcounter = 0

Не уверен, но возможно 2 надо поенять на 3

А вообще, судя по заполненности массива цифр - вам нужен бегущий ноль вместо бегущей еденицы. Отсюда и каша

[Wiza_]

PORTB = (8 ‹‹ segcounter) ; Двигаем 1 начиная с PB3

Поменяйте на: PORTB = 120 - (8 ‹‹ segcounter) ; Двигаем 0 начиная с PB3

PORTD = 0xFF; // Гасим все разряды

Этим вы, скорее, все разряды зажигаете. Но тут не принципиально

[ARTEM277]

Вы наверно хотели сказать: volatile unsigned char segcounter = 0;
Иначе не совсем понятно куда будет вывод PORTB = (8 ‹‹ 8 )

Не уверен, но возможно 2 надо поенять на 3

А вообще, судя по заполненности массива цифр - вам нужен бегущий ноль вместо бегущей еденицы. Отсюда и каша

Благодарю всех за ответы .
Wiza Скорее всего вы правы по поводу бегущего ноля. После того как шел с работы крутил в голове и тоже пришёл к выводу что надо ноль двигать.Пока ,что некогда попробовать .... как проэксперементирую опишусь мгновенно...
Wiza а как вы из кода поняли ,что ноль нужен ?

[Wiza_]

Wiza а как вы из кода поняли ,что ноль нужен

Не из кода. Из симптомов. Хотя если бы привели схему как конкретно из трёх возможных способов вы подключаете дисплей к меге - ответ можно было бы дать точнее. Между мегой и дисплеем микросхемы/транзисторы есть ? Если есть - то где и какие ?

[baiderin]

И как включены: инвертором или повторителем.

[ARTEM277]

Не из кода. Из симптомов. Хотя если бы привели схему как конкретно из трёх возможных способов вы подключаете дисплей к меге - ответ можно было бы дать точнее. Между мегой и дисплеем микросхемы/транзисторы есть ? Если есть - то где и какие ?

Конечно есть транзисторы p-n-p.И получается ,что они инверторы.Поэтому надо гонять ноль по порту а не еденицу.Сейчас пока другой работой нагружен.Обязательно поправлю и отпишусь.
Но почему же протеус отображает всё верно ? Вот в чем вопрос...

[ARTEM277]

И как включены: инвертором или повторителем.

Получается ,что инвертором...