Обработка нажатия кнопок Опции

[aBoomest]

Всем доброго дня.
Привожу нижеследующий код:

Код

  OnButtonLeft_Down:
  process(OnBtnLeft_DownEvent)
  begin
    if rising_edge(OnBtnLeft_DownEvent) then
        if (PulsePerPeriod_VAR > 0) then
              PulsePerPeriod_VAR := PulsePerPeriod_VAR - 1;           --<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
       end if;
     end if;
  end process OnButtonLeft_Down;

  OnButtonRight_Down:
  process(OnBtnRight_DownEvent)
  begin
    if rising_edge(OnBtnRight_DownEvent) then
        if (PulsePerPeriod_VAR < PulsePerPeriod_VAR_MAX) then
              PulsePerPeriod_VAR := PulsePerPeriod_VAR + 1;           --<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
       end if;
     end if;
  end process OnButtonRight_Down;

(PulsePerPeriod_VAR объявлена как shared variable)
Это обработка нажатия двух кнопок (вправо, влево)
Само нажатие работает нормально, по которому я формирую "событие-сигнал" нажатия кнопки (либо правой, либо левой - по названиям процессов все понятно)
при этом обращаю внимание на выделенные две строки. Если их закоментировать - все компилируется. Если оставить одну из них - тоже компилируется. А вот обе - сразу дает ошибку.
Я чую конечно что так писать нельзя. (Переменная PulsePerPeriod_VAR в обоих процессах одна и таже)
Но почему? Не могу понять и найти объяснение. Уже неделю копаю и в инете и в книгах Бибило.
1. Может кто-нибудь популярно объяснить почему так нельзя?
2. Как реализовать нажатие нескольких кнопок, когда по нажатию надо изменять значение одного и того же параметра (в данном случае увеличить или уменьшить значение переменной PulsePerPeriod_VAR)
3. Может кто посоветовать литературу такого плана - дается к.л. конструкция кода и далее какую структуру она порождает и описание это (примеры). Не готовые схемы счетчиков и т.п. а наоборот, с т.з. языка vhdl. Код -> структура -> описание.

Сообщение отредактировал aBoomest - Jun 18 2018, 18:18

[aBoomest]

Предлагаю все объединить в одном процессе. Заменить переменную на сигнал и таким образом описать суммирующие-вычитающий счетчик.

На счет того как засунуть в один процесс - ясно, это я тупанул жутко.

Не, чего-то и где-то я не допонимаю.
Запихал все в один процесс, как советовали выше. И, как я понимаю, ошибка в том, что нельзя в двух местах изменять переменную PulsePerPeriod_VAR. Как из ситуации-то выходить? Если по нажатию одной кнопки мне нужно увеличить значение на 1цу, по нажатию другой, соответственно уменьшить. Существует какое-то классическое/общепринятое решение в данном случае? МОжет в этот процесс надо как-то Clock какой-нибудь добавить.

Код

  OnButtonLeftRight_Down:
  process(OnBtnLeft_DownEvent,OnBtnRight_DownEvent) -- изменение длительности импульса
  begin
    if rising_edge(OnBtnLeft_DownEvent) then
       if (PulsePerPeriod_VAR > 0) then
           PulsePerPeriod_VAR := PulsePerPeriod_VAR - 1;
       end if;
    elsif rising_edge(OnBtnRight_DownEvent) then
       if (PulsePerPeriod_VAR < PulsePerPeriod_VAR_MAX) then
           PulsePerPeriod_VAR := PulsePerPeriod_VAR + 1;
       end if;
    end if;
  end process OnButtonLeftRight_Down;

[Flip-fl0p]

Первое и основное правило синхронных проектов - на тактовый вход триггера должен приходить только синхросигнал*. Т.е при вызове функции rising_edge\faling_edge использовать только синхросигнал.
Если нужно детектировать передний или задний фронт какого-либо сигнала, не являющегося тактовым - применяйте схемы, называемые детекторы фронта.
Вот кстати я недавно выкладывал свой модуль детектора фронта: https://electronix.ru/forum/index.php?showt...5&start=195

А вот кусок счетчика, который считает вверх, или вниз в зависимости от того, какие кнопки нажаты:

Код

  
   --================================================
    -- 0 разряд сигнала buttons соответствует кнопке 0
    -- 1 разряд сигнала buttons соответствует кнопке 1
    --================================================
    OnButtonLeftRight_Down : process(clk)
    begin
        if rising_edge(clk) then
            case buttons is
                when "10"   =>  PulsePerPeriod_VAR := PulsePerPeriod_VAR - 1;
                            
                when "01"   =>  PulsePerPeriod_VAR := PulsePerPeriod_VAR + 1;
                            
                when others => null;
           end case;
        end if;
    end process;

Но я бы так не делал Я бы для начала сделал обычный антидребезг и подключил его кнопкам. А реакцию схемы на нажатие кнопок обрабатывал бы простой FSM.
PS * Исключением из этого правила является специально сгенерированная на логике частота, например поделенная в 2 раза на триггере. Но даже в этом случае я предпочитаю применить PLL. Ибо частота сгенерированная на PLL имеет меньший джиттер, чем та-же частота сгенерированная на логике.