Прерывание по таймеру - поможите, после срабатывания прерывания возврат не на то место Опции

[dedded]

Ребят, в большом проекте попался кусок, над которым я уже туплю. Среда CVAVR 1.25. Нужно выдать на ногу, неважно какую, количество импульсов, с определенной частотой. После обработки прерывания по таймеру, где меняется сигнал на ноге, возвращается тупо не туда, где прерывание прервало работу проги.

#include <mega16.h>
#include <stdio.h>
#include <delay.h>


long int step_kol;
int step;


interrupt [TIM0_OVF] void timer0_ovf_isr(void)
{
// Reinitialize Timer 0 value
TCNT0=step // загрузили константу
step_kol++; //счетчик количества шагов
PORTC.0^=1; // дернули ногой

}

void main(void){

// уже не до красоты - переписал, чтобы понятно было порт С - выход

DDRC = 255;
PORTC = 0;

TIMSK=0x3;

#asm("sei")

while(1){


было
step=0x100; //константа для задания частоты
поправил, это просто описка
step=0x10;


TCCR0 = 0x00; //stop
TCNT0 = step; //set count
TCCR0 = 0x05; //start timer с предделителем на 1024


while (step_kol<1000){ //жду, когда в таймере пройдет 1000 циклов
#asm("wdr");
}
step_kol=0;
delay_ms(5000); сюда не доходит
}
}

кусок выдрал аккуратно вроде, точнее заново оформил
что же я не так делаю?

[GDI]

step все равно лучше сделать char или const char или вообще #define STEP 0x10 чтоб память не занимать.
инит таймера из главного цикла убрать - поставить ДО while(1)
delay_ms(5000) заменить на delay_ms(1000), но поставить 5 раз а между ними разместить сброс Сторожевого таймера
Перед делеями запретить прерывания от таймера - после делеев - разрешить.
Стек увеличить - может где то в другом месте он переполняется.

[dedded]

step все равно лучше сделать char или const char или вообще #define STEP 0x10 чтоб память не занимать.
инит таймера из главного цикла убрать - поставить ДО while(1)
delay_ms(5000) заменить на delay_ms(1000), но поставить 5 раз а между ними разместить сброс Сторожевого таймера
Перед делеями запретить прерывания от таймера - после делеев - разрешить.
Стек увеличить - может где то в другом месте он переполняется.

все исправил

вот так сие выглядит
#include <mega16.h>
#include <stdio.h>

long int step_kol;
char step;

// delay functions
#include <delay.h>

interrupt [TIM0_OVF] void timer0_ovf_isr(void)
{
TCCR0 = 0x00; //stop
TCNT0=step;
step_kol++;
PORTC.0^=1;
}

void main(void){
DDRC = 255;
PORTC = 0;
TIMSK=0x3;
#asm("sei")
step=0x10;

while(1){

TCCR0 = 0x00; //stop
TCNT0 = step; //set count
TCCR0 = 0x05; //start timer

while (step_kol<1000){
#asm("wdr");
}
step_kol=0;

delay_ms(1000);
#asm("wdr");
delay_ms(1000);
#asm("wdr");
delay_ms(1000);
#asm("wdr");
delay_ms(1000);
#asm("wdr");
delay_ms(1000);
#asm("wdr");

}

}
но вот так в отладчике
в прицепе 300 кил

Нифига не понимаю совсем, нашел старый проект, он на AT90s8535 - кусок подобный, все работает, чувствую, что просто туплю.

Эскизы прикрепленных изображений

 

[aesok]

Проанализируйте окошко "Call stack" и вот эту строку вашей программы:

TIMSK=0x3;

Анатолий.

[dedded]

Проанализируйте окошко "Call stack" и вот эту строку вашей программы:

TIMSK=0x3;

Анатолий.

TIMSK=0x1 решило все проблемы, всем спасибо за участие, особое - Анатолию, просто заработался уже, глаза мимо смотрели

[Сергей Борщ]

TIMSK=0x1 решило все проблемы, всем спасибо за участие, особое - Анатолию, просто заработался уже, глаза мимо смотрели

А если бы вы написали
TIMSK |= (1<<TOIE0);
или даже
TIMSK |= (0<<OCIE2)|(0<<TOIE2)|(0<<TICIE1)|(0<<OCIE1A)|(0<<OCIE1B)|(0<<TOIE1)|(1<<TOIE0);
То глазам было бы сильно проще.