и не будут, т.к. вы работаете по флагу ADIF, а его надо сбрасывать принудительно, если не используется соответствующий обработчик прерывания - вы этого не делаете, поэтому считываете из регистра ADCW одно и то же значение еще до завершения преобразования (кроме самого первого), которое просто не успевает обновиться. рекомендую ожидать конца преобразования по состоянию флага ADSC: while(ADCSRA & (1<<ADSC));edj90 писал(а):спасибо разобрался, вроде все проходит, но значения data1 и data2 не меняются.....Спойлер
это вам кто сказал? в даташите написано, что он сбрасывается либо вручную записью 1, либо аппаратно при входе в соответствующий обработчик прерываний. я не вижу обработчика прерываний в коде и не вижу, чтобы в бит ADIF где-либо записывалась единичка. строчка, на которую вы ссылаетесь, не меняет состояние ни одного бита в ADCSRA, кроме ADSC.Pink-Pank писал(а):ARV, флаг сбрасывается автоматом последующим запуском другого преобразования. Строчкой ADSRA |= (1<<ADSC);
студия - это avr-gcc, компилятор, которому можно доверять. а вопрос был про cvavr, и я не уверен, что он корректно отрабатывает volatile (да и вообще не уверен, volatile ли в нем объявление ADCSRA), так что превращение этой конструкции в sbi вполне допускаю со всеми вытекающими...Pink-Pank писал(а):sbi - это все-таки ассемблер. Хотя не исключаю возможности, что Си может ADCSRA |= (1<<ADCSC); преобразовать в sbi. Но студия у меня в read-modify-writy делала.
все таки CVAVR кладет все что надо и не надо
Истина где то рядом 
то есть? нужно указать для int размер памяти?Vanes писал(а):Даааа размер хекса не впечатляет....
Истина где то рядом 
это не то что нужно. нужно такое с этой меги где 2 потенценометра и 8 кнопок передаёт на другую мегу8, а она уже на исп. устройства. так вот потенцы преобразуют в число, в зависимости от величины которого изменяется длительность импульса передачи сигнала data1 и data2. а кнопки передают либо 0 либо1, 4 кнопки из которых работают как стент бай. надеюсь правильно объяснил.Vanes писал(а):как то такСпойлер
#include <mega8.h>
#include <delay.h>
void main(void)
{
PORTB=0xFF;
DDRB=0x00;
PORTC=0x00;
DDRC=0x00;
PORTD=0x00;
DDRD=0xFF;
while (1)
{
PORTD.0=!PINB.0; //первая кнопка
PORTD.1=!PINB.1; //вторая кнопка
PORTD.2=!PINB.2; //третья кнопка
PORTD.3=!PINB.3; //четвертая кнопка
if (PINB.4==0) //пятая кнопка
{
delay_ms(20);
PORTD.4=!PORTD.4;
};
if (PINB.5==0) //шестая кнопка
{
delay_ms(20);
PORTD.5=!PORTD.5;
};
if (PINB.6==0) //седьмая кнопка
{
delay_ms(20);
PORTD.6=!PORTD.6;
};
if (PINB.7==0) //восьмая кнопка
{
delay_ms(20);
PORTD.7=!PORTD.7;
};
}
}
Например, как-то так:edj90 писал(а):if (data3==0b00000000) как проверить не все нолики а только второй слева
Код: Выделить всё
if (~data3 & 0b00000010)Код: Выделить всё
#include <avr/io.h>
#define BTN_1 (1<<0) // Или 0b00000001
#define BTN_2 (1<<1) // Или 0b00000010
#define BTN_PIN PIND
#define BTN_PORT PORTD
#define BTN_DDR DDRD
unsigned char data3;
void main (void)
{
BTN_DDR &= ~(BTN_1 | BTN_2); // Устананавливаем выводы порта с кнопками в качестве входа
BTN_PORT = (BTN_1 | BTN_2); // Подвешиваем Pull-up резисторы на эти выводы
while (1) {
if ((BTN_PIN & BTN_1) == 0) { // проверяем состояние кнопки 1
if (data3 & 0b00000001) { // проверяем состояние вкл выкл
data3 &= ~0b00000001; // выключаем если вкл
} else {
data3 |= 0b00000001; // включаем если выкл
}
}
if ((BTN_PIN & BTN_2) == 0) { // здесь все тоже самое только вторая кнопка
if (data3 & 0b00000010) { // как проверить не все нолики а только второй слева
data3 &= ~0b00000010;
} else {
data3 |= 0b00000010;
}
}
}
}
Код: Выделить всё
#include <mega8.h>
long int data3;
void main (void)
{
PORTD=0b11111111;
DDRD= 0b00000000;
while (1)
{
if (PIND.0==0)\\ проверяем состояние кнопки 1
{
if (data3&0b00000001) \\проверяем состояние вкл выкл
{
data3|=0b00000001; \\ включаем если выкл
}
else
{
data3&=~0b00000001;\\ выключаем если вкл
};
}
if (PIND.1==0) \\здесь все тоже самое только вторая кнопка
{
if (data3&0b00000010) как проверить не все нолики а только второй слева
{
data3|=0b00000010;
}
else
{
data3&=~0b00000010;
};
}
}
}