ошибка в коде или компилятор?

[stells47]

Здравствуйте.

Неправильно работает простейший код,светодиод на PB4 должен гаснуть с задержкой,а загораться без задержки,от напряжение на PB2.

Но включается и выключается с одинаковой задержкой.Почему так?GccApplication1.c

ATTINY13

 

[Реклама]

Реклама: ООО ТД Промэлектроника, ИНН: 6659197470, Тел: 8 (800) 1000-321

[Lexter]

3 часа назад, stells47 сказал:

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

Потому что pwr_off за скобкой If.

[Реклама]

20% скидка на весь каталог электронных компонентов в ТМ Электроникс!

Акция "Лето ближе - цены ниже", успей сделать выгодные покупки!

Плюс весь апрель действует скидка 10% по промокоду APREL24 + 15% кэшбэк и бесплатная доставка!

Перейти на страницу акции

Реклама: ООО ТМ ЭЛЕКТРОНИКС, ИНН: 7806548420, info@tmelectronics.ru, +7(812)4094849

[stells47]

Спасибо.Сразу не заметил.Заработало,только наоборот=гаснет без задержки.

[Реклама]

Выбираем схему BMS для корректной работы литий-железофосфатных (LiFePO4) аккумуляторов

 Обязательным условием долгой и стабильной работы Li-FePO4-аккумуляторов, в том числе и производства EVE Energy, является применение специализированных BMS-микросхем. Литий-железофосфатные АКБ отличаются такими характеристиками, как высокая многократность циклов заряда-разряда, безопасность, возможность быстрой зарядки, устойчивость к буферному режиму работы и приемлемая стоимость. Но для этих АКБ, также как и для других, очень важен контроль процесса заряда и разряда, а специализированных микросхем для этого вида аккумуляторов не так много. Инженеры КОМПЭЛ подготовили список имеющихся микросхем и возможных решений от разных производителей. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[Павел Бачурин]

Может так?

#define F_CPU 4800000
#include <avr/io.h>
#include <util/delay.h>
#define PWR_ON PORTB|=(1<<4)
#define PWR_OFF PORTB&=(~(1<<4))
#define PAUSE 10000 // пауза на выключение в миллисекундах, максимум 32000
int delayOff = 0;

ADC_init()
{
	ADMUX=1;
	ADCSRA =134;
	ACSR=128;
}

int main(void)
{
	DDRB|=(1<<4);
	ADC_init();
	
    while(1)
    {
       ADCSRA|=(1<<ADSC);
	   while (! (ADCSRA & (1<<4))) 
	    if(ADCW > 450)
		{
			_delay_ms(1);
			if (delayOff < PAUSE) delayOff++;
			if (delayOff >= PAUSE)
			{
				PWR_OFF;
			}
		} else 
		{
			if (ADCW < 400) 
			{
				PWR_ON;
				delayOff = 0;
			}
		}
    }
}

 

Изменено 16 июня, 2019 пользователем Павел Бачурин
ошибка

[COKPOWEHEU]

13 часа назад, Павел Бачурин сказал:

ADCSRA =134;

ACSR=128;

while (! (ADCSRA & (1<<4)))

за такое - по рукам бить. Для кого придумали именованные биты

13 часа назад, Павел Бачурин сказал:

#define PAUSE 10000 // пауза на выключение в миллисекундах, максимум 32000

Почему именно 32000? На первый взгляд логичнее выглядит ограничение 65535 или 32767 или INT_MAX

[Павел Бачурин]

Да, часть ADC я не переписал, лень было в даташит лезть. Согласен, так писать нельзя