AVR studio в вопросах и ответах

[An-SU]

Код: Выделить всё

#define F_CPU 8000000UL   //obyavleniye chastoty CPU mk
#include <avr/io.h>   //podklucheniye biblioteki mk
#include <avr/interrupt.h>   //podklucheniye biblioteki zaderjek
#include <util/delay.h>   //podklucheniye zaderjek
#include <stdio.h>

unsigned int Sch;

static void ti_in(void)
{
   TCCR1A|=0;   //регистр управления таймером/счетчиком
   TCNT1|=0;   //Регистр таймера/счетчика
   OCR1A=Sch;   //значение сравнения, до 16 бит
   TIMSK|=(1<<OCIE1A);  
   TCCR1B|=(1<<CS12)|(1<<WGM12);   //установка делителя, режима работы таймера
}

ISR(TIMER1_COMPA_vect)   //функция, выполняемая автоматически, при совпадении числел - счетчика и установленного значения
{
   PORTC=~PORTC;
}

static void TI_Write(unsigned int Sch)
{
 unsigned int sreg = SREG;
 cli();
 OCR1A=Sch;
SREG=sreg;
}

int main(void)
{
   cli();
   DDRC=0xFF;
   Sch=100; //в реале период / 2
   ti_in();
   sei();

    while (1)
    {
      if (!(PINE &= (1<<PE1)))
       TI_Write(300);

      if (!(PINE &= (1<<PE0)))
       TI_Write(1000);
    }
}

[ARV]

static void TI_Write(unsigned int Sch)
{
 unsigned int sreg = SREG;
 cli();
 OCR1A=Sch;
SREG=sreg;
}

[An-SU]

ARV, ну удаление особой роли в данном случае не сыграет, повторюсь, я просто ориентировался на:
https://docplayer.ru/59393693-Mikroshem ... dl-to.html 124 страница сверху

[ARV]

у вас творятся чудеса, поэтому все лишнее - безжалостно отсекаем.
у вас кнопки на порту Е - я так понимаю, на общий? где настройка порта Е?

[prinv]

Как-то больно мудро опрашиваете клавиатуру
Попробуйте

Код: Выделить всё

if (!(PINE &= (1<<PE1)))

заменить на

Код: Выделить всё

if(!(PINE & (1<<PE1)))

"PINE &=" что вы этим хотите добиться?

[ARV]

кстати, да - глаз замылился уже не заметил &=

[prinv]

Как реализована подтяжка кнопок? Внешними резисторами?
Если нет, то её нужно включить

Код: Выделить всё

PORTE = (1<<PE0) | (1<<PE1);

[An-SU]

Исправил все указанные вами ошибки, но по прежнему изменяется лишь во время нажатия

[prinv]

Покажите получившийся код
Я кажется нашёл косяк.
У вас и глобальная переменная, и параметр функции имеют одно имя - Sch

[ARV]

Я кажется нашёл косяк.
У вас и глобальная переменная, и параметр функции имеют одно имя - Sch

по идее, это не косяк: по правилу экранирования локальная переменная (параметр) экранирует глобальную, т.е. внутри функции видна именно переменная-параметр.

[prinv]

Не косяк, если это не влияет на работу

[ARV]

ну так не должно влиять...

[prinv]

Ну видишь какие чудеса происходят - лучше исключить непонятки

[An-SU]

Проверил - не влияет эта переменная.

Код: Выделить всё

#define F_CPU 8000000UL   //obyavleniye chastoty CPU mk
#include <avr/io.h>   //podklucheniye biblioteki mk
#include <avr/interrupt.h>   //podklucheniye biblioteki zaderjek
#include <util/delay.h>   //podklucheniye zaderjek
#include <stdio.h>

static void ti_in(void)
{
   TCCR1A|=0;   //регистр управления таймером/счетчиком
   TCNT1|=0;   //Регистр таймера/счетчика
   OCR1A=Sch;   //значение сравнения, до 16 бит
   TIMSK|=(1<<OCIE1A); 
   TCCR1B|=(1<<CS12)|(1<<WGM12);   //установка делителя, режима работы таймера
}

ISR(TIMER1_COMPA_vect)   //функция, выполняемая автоматически, при совпадении числел - счетчика и установленного значения
{
   PORTC=~PORTC;
}

static void TI_Write(unsigned int Sch)
{
 //unsigned int sreg = SREG;
 //cli();
 OCR1A=Sch;
//SREG=sreg;
}

int main(void)
{
   cli();
   DDRC=0xFF;
   PORTE = (1<<PE0) | (1<<PE1);
   unsigned int Sch=100; //в реале период / 2
   ti_in();
   sei();

    while (1)
    {
      if (!(PINE & (1<<PE1)))
       TI_Write(300);

      if (!(PINE & (1<<PE0)))
       TI_Write(1000);
    }
}

[prinv]

А если вот так переписать?

Код: Выделить всё

void TI_Write(unsigned int Sch1)
{
 OCR1A=Sch1;
}

[An-SU]

Пробовал, безрезультатно

[ARV]

а давайте-ка уточним, что вы вкладываете в понятие "не работает" программа? вы как ОЖИДАЕТЕ увидеть её работу?
период мигания у вас НЕ МЕНЯЕТСЯ, так как у вас не режим СТС, а просто счет таймера. при каком бы значении не происходило сравнение OCR1A с TCNT, период между этими событиями только в момент нажатия кнопки будет иным, что легко не заметить, а все прочее время - тем же самым, как определено предделителем таймера

[prinv]

[uquote="ARV",url="/forum/viewtopic.php?p=3475881#p3475881"]не режим СТС[/uquote]
Почему не CTC? Только WGM12 установлен.
Я бы только поменял всё на безусловное присваивание

Код: Выделить всё

void ti_in(void)
{
   TCCR1B = 0; // И ПЕРВОЙ СТРОЧКОЙ ЭТО БЫ ДОБАВИЛ
   TCCR1A=0;   //регистр управления таймером/счетчиком
   TCNT1=0;   //Регистр таймера/счетчика
   OCR1A=Sch;   //значение сравнения, до 16 бит
   TIMSK=(1<<OCIE1A);
   TCCR1B=(1<<CS12)|(1<<WGM12);   //установка делителя, режима работы таймера
}

[An-SU]

Да, как раз в СТС именно и сравнивается счетчик с OCR.

А под "не работает" имею в виду то, что после нажатия на кнопку частота возвращается к старому значению, в то время как (почему я ранее и сделал Sch глобальным) по идее должно обновиться и не изменяться уже

[ARV]

Только WGM12 установлен

проглядел