проблема портирования программы с AtMega32 на AtMega 128 Опции

[Nekromant]

Доброго времени суток,
суть проблемы заключается ч том, что написанная программа для AtMega 128 работает без каких либо изъянов, но при попытке ее перенести на AtMega 32 возникла следующая проблема:
приведенная ниже конструкция на 32 меге просто не работает:
Вариант 1
if(tmp==0x00)
{
Closed=true;
}
else
{
Closed=false;
if(tmp==0x3F)
{
All=true;
}
else
{
All=false;
for( i = 1; i < 6; i++)
{
if(tmp & (1 << i))
SensorData[i]=false;
else
SensorData[i]=true;
}

}
}
Вариант 2
if(tmp==0x00)
{
Closed=true;
}

else
{
Closed=false;
if(tmp==0x3F)
{
All=true;
}
else
{
All=false;
tmp=(~tmp)&0x01;
if(tmp!=0x01)
{
Nurse1=true;
}
else
{
Nurse1=false;
}
tmp=tmp_data;
tmp=(~tmp)&0x02;
if(tmp!=0x02)
{
Nurse2=true;
}
else
{
Nurse2=false;
}
tmp=tmp_data;
tmp=(~tmp)&0x04;
if(tmp!=0x04)
{
Hall=true;
}
else
{
Hall=false;
}
tmp=tmp_data;
tmp=(~tmp)&0x08;
if(tmp!=0x08)
{
Bedroom=true;
}
else
{
Bedroom=false;
}
tmp=tmp_data;
tmp=(~tmp)&0x10;
if(tmp!=0x10)
{
Kitchen=true;
}
else
{
Kitchen=false;
}
tmp=tmp_data;
tmp=(~tmp)&0x20;
if(tmp!=0x20)
{
Door=true;
}
else
{
Door=false;
}
}
оба варианта ведут себя одинаково нормально на меге 128 и оба же варианта ведут себя одинаково плохо на меге 32

[MrYuran]

1. Используйте теги [code][/code] для кусков кода, иначе нечитабельно вообще
2. Вместо if/else/else/else лучше использовать switch/case по возможности
3. Вряд ли проблема в этом месте, код достаточно примитивный.
Опять же, не указали, что значит не работает и каким именно образом.

[Nekromant]

1)именно плохо в том, что на 32 меге изменяются только флаги nurse1 и nurse2 остальные как будто вообще не существуют
2) проверил все части программы которые берут эти флаги-их изменение ничего не выявило, т. е. как были активными только два первые флага так они и остались

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

2) проверил все части программы которые берут эти флаги-их изменение ничего не выявило, т. е. как были активными только два первые флага так они и остались

Телепатически предположу, что tmp изменяется в прерывании и объявлена без квалификатора volatile.

[MrYuran]

Телепатически предположу, что tmp изменяется в прерывании и объявлена без квалификатора volatile.

++
То же самое хотел сказать
Вообще, тяжело телепатировать на голодный желудок...

[Nekromant]

++
То же самое хотел сказать
Вообще, тяжело телепатировать на голодный желудок...

tmp не в прерывании изменяется.
tmp_data=Sens_Data;
tmp=tmp_data; вот откуда берется tmp.
поа функция один раз не не выполнится второй раз копирования не происходит)

[Dog Pawlowa]

tmp не в прерывании изменяется.
tmp_data=Sens_Data;
tmp=tmp_data; вот откуда берется tmp.
поа функция один раз не не выполнится второй раз копирования не происходит)

Дык это сигналы, связанные со внешним миром. Подключены они правильно? На те же порты?

А вообще-то ... Как лодку назовешь, так она и поплывет.

[Nekromant]

да, это сигналы, подключенные на порты, точнее 6 датчиков висящих на PORTC

Сообщение отредактировал Nekromant - Aug 6 2008, 08:48

[Stanislav_S]

да, это сигналы, подключенные на порты, точнее 6 датчиков висящих на PORTC

А фьюз JTAG как установлен?

[Nekromant]

фьюз я не трогал просто в функции маин в самом начале прописано
MCUCSR = (1 << JTD); //Disable JTAG
MCUCSR = (1 << JTD); //Disable JTAG
(по совету Haker Fox)