1-Bit-Latch Warnung

[baumhous3 (Gast)]

Hallo zusammen,
ich habe ein Problem. Bin aktuell dabei in VHDL ein kleines Programm für 
die Uni zu schreiben, nun bin ich soweit fertig nur bekomme ich folgende 
Fehlermeldung:

WARNING:Xst:737 - Found 1-bit latch for signal <HB_bot>. Latches may be generated from incomplete case or if statements. We do not recommend the use of latches in FPGA/CPLD designs, as they may lead to timing problems.
WARNING:Xst:737 - Found 1-bit latch for signal <HB_bot1>. Latches may be generated from incomplete case or if statements. We do not recommend the use of latches in FPGA/CPLD designs, as they may lead to timing problems.

Ich denke der Fehler liegt in folgendem Programmteil. Nun weiß ich aber 
nicht wie ich das ganze anders schreiben kann. Hoffe ihr könnt mir da 
helfen.

process(clk,pwm_out,delayt,delayb)
begin
  -- Totzeit HB_top
  if rising_edge(clk) then
    if pwm_out = '1' AND (delayt < 199)         
    then delayt <= delayt + 1;
        delayb <= 0;
  elsif (delayt = 199) then HB_top <= pwm_out;
    end if;
  end if;
  
  -- Totzeit HB_bot
  if pwm_out = '0' AND (delayb < 199) 
    then delayb <= delayb + 1;
        delayt <= 0;
  elsif (delayb = 199) then HB_bot <= NOT pwm_out;
  end if;
end process;

Gruß,
baumhous3

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Man mischt nicht einen getakteten mit einem kombinatorischen Prozess:

process(clk,pwm_out,delayt,delayb)
begin
  -- Totzeit HB_top --- getaktet
  if rising_edge(clk) then
     ...
  end if;
  
  -- Totzeit HB_bot --- ohne Takt = kombinatorisch!
  if pwm_out = '0' AND (delayb < 199) 
     then delayb <= delayb + 1; --- hoppala, jetzt aber: eine kombinatorische Schleife!
     ...
  end if;
end process;

Zudem ist das Latch noch das kleinste Problem.
Viel schlimmer ist die versteckte kombinatorische Schleife:
http://www.lothar-miller.de/s9y/archives/42-Kombinatorische-Schleifen.html


Das hier ist das erste schlechte Zeichen:

process(clk,pwm_out,delayt,delayb)

Faustregeln:
In einem kombinatorischen Prozess steht kein "clk".
In einem getakteten Prozess sind keine anderen Signale ausser "clk"
(und schlimmstenfalls noch "rst").

[baumhous3 (Gast)]

Okay, aber wie könnte ich das ganze denn aufteilen ohne das ich diese 
hinbekomme? Hänge da aktuell auf dem Schlauch :(

[Lutz Laserblick (Gast)]

baumhous3 schrieb:
> Hänge da aktuell auf dem Schlauch :(

Nicht nur den Programmierstil ist sesaströs, auch im Deutschen greifts 
du auf zu "Stilblüten". Mang hängt nicht auf dem schlauch, man steht auf 
diesem!

SCNR

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Lutz Laserblick schrieb:
> sesaströs
Gefällt mir, dieses neue Wort...
> Mang
Mangomang!

baumhous3 schrieb:
> Okay, aber wie könnte ich das ganze denn aufteilen
Was?
> ohne das ich diese hinbekomme?
Welche?

Du willst die kombinatorische Schleife weghaben?
Dann mach einen Takt in den Zähler. Denn ein Zähler braucht 
Speicherglieder, um den aktuellen Zählerwert festzuhalten.
Und Speicherglieder sind in FPGAs immer Flipflops.
Und Fipflops brauchen einen Takt.

Probiers einfach mal so:

process(clk)
begin
  -- Totzeit HB_top
  if rising_edge(clk) then

    if pwm_out = '1' AND (delayt < 199) then 
        delayt <= delayt + 1;
        delayb <= 0;
    elsif (delayt = 199) then 
        HB_top <= pwm_out;
    end if;
--  end if;  --<<< mach das hier hier weg!!!
  
  -- Totzeit HB_bot
    if pwm_out = '0' AND (delayb < 199) then 
      delayb <= delayb + 1;
      delayt <= 0;
    elsif (delayb = 199) then 
      HB_bot <= NOT pwm_out;
    end if;

  end if; --<<< und dafür hier rein!

end process;

Also: nicht einfach nur irgendwelchen Code irgendwo herkopopieren, 
sondern auch kapieren...

[Bronco (Gast)]

Lothar Miller schrieb:
> herkopopieren
Gefällt mir, dieses neue Wort...

(Sorry, muß gerad auf ein Bitfile warten und hab nix besseres zu tun)

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Bronco schrieb:
>> herkopopieren
> Gefällt mir, dieses neue Wort...
Musste ich auch lange für üben...   ;-)

[Bronco (Gast)]

Könnte glatt von Helge Schneider sein:
Kopieren-po-pieren