Erster Versuch in VHDL (Taster entprellen)- so in Ordnung?

Habe mir den Code nur kurz angesehen, aber das sieht mir reichlich 
kompliziert aus. Schau dir mal an wie die Entprellung auf einem 
Mikrocontroller gemacht wird:
http://www.mikrocontroller.net/articles/Entprellung#Komfortroutine_.28C_f.C3.BCr_AVR.29
Genauso kannst du es in VHDL umsetzen. Dem Timer-Interrupt entspricht 
ein Zähler der ein Clock-Enable-Signal erzeugt. Vorteil: du kannst 
mehrere Tasten gleichzeitig auswerten ohne dass du dazu mehrere Zähler 
brauchst.

@ Heiko Lechner (der_caveman)

>Dabei benutze ich schon die Aufsplittung in mehrere vhd Dateien und
>resultierend auch Komponenten.

Prinzipiell richtig, aber . . .

>Was dabei herauskam liegt im Anhang- es wäre sehr schön, wenn das mal
>ein paar erfahrene VHDLer kritisieren könnten.

Willst du das wirklich? ;-)

>Kurz gesagt tastet der "Entpreller" die Taste ab. War der vorherige

Schon mal OK.

>"überläuft", dann wurde oft genug der gleiche Zustand abgetastet, man
>kann davon ausgehen, dass der Pegel am Taster stabil bleibt, der Zustand
>geht über zu "Taste gedrückt".

Guter Ansatz. Aber sehr akademisch umgesetzt.

MFG
Falk

P.S. So gehts etwas kompakter

1

signal key_old, key_smp: std_logic;

2

signal cnt: std_logic_vector(7 downto 0);

3

constant max_cnt : integer:=200;

4

5

process(clk)

6

begin

7

  if rising_edge(clk) then

8

    key_smp <= key;       -- abtasten

9

    if key_smp /= key_old

10

      cnt <= cnt-1;       

11

    else

12

      cnt <= conv_std_logic_vector(max_cnt, 8);

13

    end if;

14

    if cnt=0 then

15

      key_old <= key_smp;

16

    end if;

17

  end if;

18

end process;

Falk Brunner wrote:

>>Was dabei herauskam liegt im Anhang- es wäre sehr schön, wenn das mal
>>ein paar erfahrene VHDLer kritisieren könnten.
>
> Willst du das wirklich? ;-)

Klar- wie soll ich sonst was lernen?
Ich will hier natürlich keinem irgendetwas aufzwingen.

> Guter Ansatz. Aber sehr akademisch umgesetzt.

Das kommt davon, wenn man nur Bücher ließt... ;)

> P.S. So gehts etwas kompakter
>
>
>

1

> signal key_old, key_smp: std_logic;

2

> signal cnt: std_logic_vector(7 downto 0);

3

> constant max_cnt : integer:=200;

4

> 

5

> process(clk)

6

> begin

7

>   if rising_edge(clk) then

8

>     key_smp <= key;       -- abtasten

9

>     if key_smp /= key_old

10

>       cnt <= cnt-1;

11

>     else

12

>       cnt <= conv_std_logic_vector(max_cnt, 8);

13

>     end if;

14

>     if cnt=0 then

15

>       key_old <= key_smp;

16

>     end if;

17

>   end if;

18

> end process;

19

>

... da verliert man irgendwie den Sinn zur Einfachheit...

Den C- Code muss ich mir mal angucken wenn ich etwas Ruhe habe- auf den 
ersten Blick ist mir da nichts ersichtlich :D

Danke erstmal an euch beide!

Den Link zum C-Code gibts in der Antwort von Andreas Schwarz

Ich habe mir nur "Reference design - Rotary encoder and simple LEDs on 
Spartan-3E Starter Kit (Revision C)" angeschaut, da finde ich auch keine 
Entprellung für den Drehgeber- Taster, aber in der Dokumentation dazu 
steht, dass das doch ne prima Lernaufgabe wäre ;)

@ Heiko Lechner (der_caveman)

>Spartan-3E Starter Kit (Revision C)" angeschaut, da finde ich auch keine
>Entprellung für den Drehgeber- Taster, aber in der Dokumentation dazu
>steht, dass das doch ne prima Lernaufgabe wäre ;)

Bei richtiger Auswertung eines Drehgebers ist die Entprellung 
inclusive.

MFG
Falk

Aber nicht bei dessen Drucktaster (rotary pushbutton).

Und was machst du wenn

a) du einfache Taster, keine Umschalter und keine zwei Ports je Taster 
verwenden willst?

oder

b) deine Taster nicht so "ideal" prellen wie in diesem Tutorial gezeigt, 
sondern Kontakt zwei schon anfaengt mit prellen, bevor Kontakt eins 
endgueltig aus ist?