busy-Flag richtig setzen

Ralf P. Gast 07.02.2013 08:56 #3033779

Lesenswert?

•

Hallo zusammen,

ich habe einen Prozess, der während seiner Aktivität ein busy-Flag 
ausgeben soll.

Dabei wird ein start-Signal angelegt, welches aber nur für einen Takt 
high ist.
Daraufhin soll ein counter gestartet werden und zwar solange, bis der 
Counter einen bestimmten Wert erreicht hat. Während dieser Zeit soll das 
busy-Flag gesetzt bleiben und erst beim Erreichen des Counters wieder 
auf Null gesetzt werden und alles geht erst mit einem start-Signal 
wieder los.

Dazu habe ich folgendes probiert:

busy_flag : PROCESS(clk, res)
  IF res = '1' THEN
     counter <= (others => '0');
     busy <= '0';
  ELSIF rising_edge(CLK) THEN
     IF start = '1' THEN                  -- für einen Takt start = 1
        busy <= '1';                      -- busy länger als einen Takt setzen
     IF busy <= '1' THEN                  -- sobald busy = 1
        counter <= counter + 1;           -- counter hochzählen
        IF and_reduce(counter) = '1' THEN -- wenn counter am Maximum
           rdy <= '1';                    -- ready flag auf 1 setzen
           busy <= '0';                   -- busy flag zurücksetzen
        ELSE                              -- solange counter noch zählt
           rdy <= '0';                    -- ready flag ist 0
           busy <= '1';                   -- busy bleibt auf 1
        END IF;
     END IF;
  END IF;
END PROCESS;


Leider geht das busy-Signal schon mit der ersten Flanke nach dem Reset 
auf 1, damit fängt der Counter an zu zählen und alles geht schief.

Erkennt ihr meine Beschreibung in dem VHDL Code wieder, oder habe ich 
einen groben Schnitzer gemacht.

Man könnte die Sache auch einfach mit einer FSM umgehen (ein state auf 
start warten, zweiter state busy=1 und counter zählen und dann wieder 
zurück), aber das wollte ich bewusst vermeiden.

Hat jemand einen Hinweis?
Liegt es an der sensitivity list? Wenn ja, habe ich es noch nicht ganz 
verstanden, was da rein muss... meines Erachtens nur Signale, die 
außerhalb des Prozesses getrieben werden, im Prozess aber abgefragt.

Vielen Dank!
Ralf

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

Cihan K. (lazoboy61) 07.02.2013 09:16 #3033803

Lesenswert?

•

▲
▼

Wie wird das Start gesetzt? Ist res und start synchron zur Clock? Wenn 
nicht könntest du Probleme damit haben. Abgesehen davon wie hast du 
start und busy initialisiert?

Cihan

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ralf P. Gast 07.02.2013 09:29 #3033824

Lesenswert?

•

▲
▼

Hallo,

Cihan Kalayci schrieb:
> Wie wird das Start gesetzt? Ist res und start synchron zur Clock?
start ist synchron, res nicht, wird aber im Topfile einsynchronisiert.
In diesem Block sit also alles synchron :)

> Abgesehen davon wie hast du start und busy initialisiert?
Diese beiden Signale werden doch mit dem Reset mit einem eindeutigen 
Wert belegt. Also sollte doch nach dem Reset alles glatt gehen?

Vielen Dank!
Ralf

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ralf P. Gast 07.02.2013 09:34 #3033827

Lesenswert?

•

▲
▼

Aber selbst wenn ich alle signale mit

  signal start     : std_logic := '0';
  signal rdy       : std_logic := '0';
  signal busy      : std_logic := '0';

initialisiere, geht es leider immer noch nicht.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Dussel Gast 07.02.2013 09:36 #3033830

Lesenswert?

•

▲
▼

In C tritt so ein Fehler häufig auf:
IF busy <= '1' THEN
Überleg mal ;-)

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Cihan K. (lazoboy61) 07.02.2013 09:50 #3033853

Lesenswert?

•

▲
▼

Dussel schrieb:
> In C tritt so ein Fehler häufig auf:
> IF busy <= '1' THEN
> Überleg mal ;-)

Ist auf der ersten Hinsicht nicht zu sehen.

Ich hatte mal den gleichen Fehler gemacht und das Signal wurde die ganze 
Zeit in der Synthese wegoptimiert. Nach langem suchen hatte ich dann 
selber entdeckt, dass ich das Signal mit <= abgefragt habe. Dummer C&P 
Fehler.

Cihan

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ralf P. Gast 07.02.2013 10:01 #3033868

Lesenswert?

•

▲
▼

AAAAHHHHH !!!!!
Dussel schrieb:
> In C tritt so ein Fehler häufig auf:
> IF busy <= '1' THEN
> Überleg mal ;-)
Sowas müsste der Compiler doch aber bemerken und meckern?
Krasse Sache... auf was man alles achten muss.

Bei solchen Sachen hat man nach 1h Fehlersuche keine Chance mehr. Da ist 
man einfach blind...

Vielen Dank!

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Cihan K. (lazoboy61) 07.02.2013 10:29 #3033921

Lesenswert?

•

▲
▼

Ralf P. schrieb:
> AAAAHHHHH !!!!!
> Dussel schrieb:
>> In C tritt so ein Fehler häufig auf:
>> IF busy <= '1' THEN
>> Überleg mal ;-)
> Sowas müsste der Compiler doch aber bemerken und meckern?
> Krasse Sache... auf was man alles achten muss.
>
> Bei solchen Sachen hat man nach 1h Fehlersuche keine Chance mehr. Da ist
> man einfach blind...
>
> Vielen Dank!

Bei mir kam eine Warnung dazu. Müsste bei dir auch drin sein eigentlich.
Naja, hauptsache Fehler gefunden, Danke hier an Dussel.

Cihan

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lothar M. (lkmiller) Moderator (Firma: Titel) 07.02.2013 12:58 #3034140

Lesenswert?

•

▲
▼

Ralf P. schrieb:
>>>> IF busy <= '1' THEN
> Sowas müsste der Compiler doch aber bemerken und meckern?
Nein, sieh dir mal die Definition eines std_logic an:

PACKAGE std_logic_1164 IS
    -------------------------------------------------------------------    
    -- logic state system  (unresolved)
    -------------------------------------------------------------------    
    TYPE std_ulogic IS ( 'U',  -- Uninitialized
                         'X',  -- Forcing  Unknown
                         '0',  -- Forcing  0
                         '1',  -- Forcing  1
                         'Z',  -- High Impedance   
                         'W',  -- Weak     Unknown
                         '-'   -- Don't care

Kleiner als '0' ist hier also noch 'X' und 'U'. Und deshalb ist diese 
Abfrage prinzipiell korrekt, und kann in der Simulation durchaus Sinn 
machen...

> Krasse Sache... auf was man alles achten muss.
Tja, das durchaus...  ;-)

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Dussel Gast 07.02.2013 14:54 #3034430

Lesenswert?

•

▲
▼

Lothar Miller schrieb:
> Und deshalb ist diese
> Abfrage prinzipiell korrekt,
Heißt das, dass das ein kleiner-gleich ist? Ich bin jetzt davon 
ausgegangen, dass das eine (ungewollte) Zuweisung wie das bekannte 
if(a=b) in C ist.
Dann war das ja nur ein Zufallsfund. :-)

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Lothar M. (lkmiller) Moderator (Firma: Titel) 07.02.2013 16:21 #3034644

Lesenswert?

•

▲
▼

Dussel schrieb:
> Heißt das, dass das ein kleiner-gleich ist?
Ja.

> Ich bin jetzt davon ausgegangen, dass das eine (ungewollte) Zuweisung
> wie das bekannte if(a=b) in C ist.
In VHDL kann an dieser Stelle nichts zugewiesen werden.

> Dann war das ja nur ein Zufallsfund. :-)
Tja, nochmal Dussel gehabt. Nomen est Omen...  ;-)

Antwort Zitieren Bearbeiten Löschen Beitrag melden

busy-Flag richtig setzen

Antwort schreiben