asynchronous counter an Xilinx FPGA for a newbie

  • Thread starter Georges Konstantinidis
  • Start date
G

Georges Konstantinidis

Guest
Dear all
I'm deseprately trying to make an asynchronous counter to count the number
of inputs I have on a pin. I also want a reset input.
I copied the last version of my code at this e-mail .
The synthesis looks good but an error comes at the implementation design. I
don't kow to to do any more.
Thank you for fixing my bugs, Georges.


library ieee;
use ieee.std_logic_1164.all;

entity counter is
port(Load, Rst: in std_logic;
LED: out std_logic_vector(0 to 7)
);
end counter;

architecture behaviour of counter is
signal Qreg: std_logic_vector(0 to 7);

begin
process(Rst, Load)
begin
if Rst = '1' then -- Async reset
Qreg <= "00000000";
elsif Load='1' then
Qreg<=Qreg+1;
end if;

end process;

LED <= Qreg;

end behaviour;
 
Your synthesis tool does not synthesize:
process(Rst, Load)
begin
if Rst = '1' then -- Async reset
Qreg <= "00000000";
elsif Load='1' then
Qreg<=Qreg+1;
end if;
Click to expand...
But it synthesizes it with a different sensitivity list:
process(Rst, Load, QREG)

If you simulate (before synthesis) the process with this sensitive list
you will notice the loop if LOAD='1' and rst='0'.

I'm not sure if you want a latch, but a flipflop solves the problem.
Replace
elsif Load='1' then
with
Click to expand...
elsif rising_edge(Load) then

Egbert Molenkamp

"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> schreef in
bericht news:401ab556$0$777$ba620e4c@news.skynet.be...
Dear all
I'm deseprately trying to make an asynchronous counter to count the number
of inputs I have on a pin. I also want a reset input.
I copied the last version of my code at this e-mail .
The synthesis looks good but an error comes at the implementation design.
I
don't kow to to do any more.
Thank you for fixing my bugs, Georges.


library ieee;
use ieee.std_logic_1164.all;

entity counter is
port(Load, Rst: in std_logic;
LED: out std_logic_vector(0 to 7)
);
end counter;

architecture behaviour of counter is
signal Qreg: std_logic_vector(0 to 7);

begin
process(Rst, Load)
begin
if Rst = '1' then -- Async reset
Qreg <= "00000000";
elsif Load='1' then
Qreg<=Qreg+1;
end if;

end process;

LED <= Qreg;

end behaviour;
Click to expand...
 
"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in message news:<401ab556$0$777$ba620e4c@news.skynet.be>...
Dear all
I'm deseprately trying to make an asynchronous counter to count the number
of inputs I have on a pin. I also want a reset input.
I copied the last version of my code at this e-mail .
The synthesis looks good but an error comes at the implementation design. I
don't kow to to do any more.
Thank you for fixing my bugs, Georges.


library ieee;
use ieee.std_logic_1164.all;

entity counter is
port(Load, Rst: in std_logic;
LED: out std_logic_vector(0 to 7)
);
end counter;

architecture behaviour of counter is
signal Qreg: std_logic_vector(0 to 7);

begin
process(Rst, Load)
begin
if Rst = '1' then -- Async reset
Qreg <= "00000000";
elsif Load='1' then
Qreg<=Qreg+1;
end if;

end process;

LED <= Qreg;

end behaviour;
Click to expand...
Why do you want to do an async counter? No clock available?

What is the implementation error? (Actually, I know what it is -- but
I want to know what you think it is.)

Think about:

What happens if neither Rst nor Load are asserted?

--a
 
What do you mean by saying "asserted"?
"Andy Peters" <Bassman59a@yahoo.com> a écrit dans le message de
news:9a2c3a75.0401301451.70df14a6@posting.google.com...
"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in
message news:<401ab556$0$777$ba620e4c@news.skynet.be>...
Dear all
I'm deseprately trying to make an asynchronous counter to count the
number
of inputs I have on a pin. I also want a reset input.
I copied the last version of my code at this e-mail .
The synthesis looks good but an error comes at the implementation
design. I
don't kow to to do any more.
Thank you for fixing my bugs, Georges.


library ieee;
use ieee.std_logic_1164.all;

entity counter is
port(Load, Rst: in std_logic;
LED: out std_logic_vector(0 to 7)
);
end counter;

architecture behaviour of counter is
signal Qreg: std_logic_vector(0 to 7);

begin
process(Rst, Load)
begin
if Rst = '1' then -- Async reset
Qreg <= "00000000";
elsif Load='1' then
Qreg<=Qreg+1;
end if;

end process;

LED <= Qreg;

end behaviour;

Why do you want to do an async counter? No clock available?

What is the implementation error? (Actually, I know what it is -- but
I want to know what you think it is.)

Think about:

What happens if neither Rst nor Load are asserted?

--a
Click to expand...
 
Try this:

library ieee;
use ieee.std_logic_1164.all;

entity counter is
port( Load: in std_logic;
Rst: in std_logic;
LED: out std_logic_vector(0 to 7));

end counter;

architecture behaviour of counter is
begin

cnt : process(Rst, Load)

variable count : std_logic_vector(7 downto 0);

begin

if (Rst = '1' then) then
count := "00000000";
else if (Load'event and Load='1') then
count := count + 1;
end if;

LED <= count;

end process cnt;

end behaviour;

Now make sure that Load is 'locked' to a clock input on the FPGA, or
you'll probably get complaints of a logic line driving clock inputs.
Try using something like:

attribute LOC : string;
attribute LOC of signal Load : signal is "pin number";

Every time that a rising edge of Lock occurs, the variable count is
incremented. LED will be loaded with the value of count each time the
process 'runs' (I assume this is what you want?).


Georges Konstantinidis wrote:
What do you mean by saying "asserted"?
"Andy Peters" <Bassman59a@yahoo.com> a écrit dans le message de
news:9a2c3a75.0401301451.70df14a6@posting.google.com...

"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in

message news:<401ab556$0$777$ba620e4c@news.skynet.be>...

Dear all
I'm deseprately trying to make an asynchronous counter to count the

number

of inputs I have on a pin. I also want a reset input.
I copied the last version of my code at this e-mail .
The synthesis looks good but an error comes at the implementation

design. I

don't kow to to do any more.
Thank you for fixing my bugs, Georges.


library ieee;
use ieee.std_logic_1164.all;

entity counter is
port(Load, Rst: in std_logic;
LED: out std_logic_vector(0 to 7)
);
end counter;

architecture behaviour of counter is
signal Qreg: std_logic_vector(0 to 7);

begin
process(Rst, Load)
begin
if Rst = '1' then -- Async reset
Qreg <= "00000000";
elsif Load='1' then
Qreg<=Qreg+1;
end if;

end process;

LED <= Qreg;

end behaviour;

Why do you want to do an async counter? No clock available?

What is the implementation error? (Actually, I know what it is -- but
I want to know what you think it is.)

Think about:

What happens if neither Rst nor Load are asserted?

--a
Click to expand...
 
"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in message news:<401c1026$0$13244$ba620e4c@news.skynet.be>...

What do you mean by saying "asserted"?
Click to expand...
"Asserted" means "put into the active state."

It's a very common term. It's useful because it nicely ignores the
detail of whether the signal is active low or active high.

--a
 
Thanks for the info.
"Andy Peters" <Bassman59a@yahoo.com> a écrit dans le message de
news:9a2c3a75.0402020916.391b9ce6@posting.google.com...
"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in
message news:<401c1026$0$13244$ba620e4c@news.skynet.be>...

What do you mean by saying "asserted"?

"Asserted" means "put into the active state."

It's a very common term. It's useful because it nicely ignores the
detail of whether the signal is active low or active high.

--a
Click to expand...
 
Hello Andrew,
I was endeed using a "normal I/O port" which is synchronous or asynchronous
according to the datasheets. I should be able to use it.
Or I miss something.
I found also a template in ISE called "debounce circuit" which seems to
work.
In any case I will also try what you said
Thanks everyone for you cooperation, Georges.



"Andrew Greensted" <ajg112@ohm.york.ac.uk> a écrit dans le message de
news:bvlerm$crt$1@pump1.york.ac.uk...
Try this:

library ieee;
use ieee.std_logic_1164.all;

entity counter is
port( Load: in std_logic;
Rst: in std_logic;
LED: out std_logic_vector(0 to 7));

end counter;

architecture behaviour of counter is
begin

cnt : process(Rst, Load)

variable count : std_logic_vector(7 downto 0);

begin

if (Rst = '1' then) then
count := "00000000";
else if (Load'event and Load='1') then
count := count + 1;
end if;

LED <= count;

end process cnt;

end behaviour;

Now make sure that Load is 'locked' to a clock input on the FPGA, or
you'll probably get complaints of a logic line driving clock inputs.
Try using something like:

attribute LOC : string;
attribute LOC of signal Load : signal is "pin number";

Every time that a rising edge of Lock occurs, the variable count is
incremented. LED will be loaded with the value of count each time the
process 'runs' (I assume this is what you want?).


Georges Konstantinidis wrote:
What do you mean by saying "asserted"?
"Andy Peters" <Bassman59a@yahoo.com> a écrit dans le message de
news:9a2c3a75.0401301451.70df14a6@posting.google.com...

"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in

message news:<401ab556$0$777$ba620e4c@news.skynet.be>...

Dear all
I'm deseprately trying to make an asynchronous counter to count the

number

of inputs I have on a pin. I also want a reset input.
I copied the last version of my code at this e-mail .
The synthesis looks good but an error comes at the implementation

design. I

don't kow to to do any more.
Thank you for fixing my bugs, Georges.


library ieee;
use ieee.std_logic_1164.all;

entity counter is
port(Load, Rst: in std_logic;
LED: out std_logic_vector(0 to 7)
);
end counter;

architecture behaviour of counter is
signal Qreg: std_logic_vector(0 to 7);

begin
process(Rst, Load)
begin
if Rst = '1' then -- Async reset
Qreg <= "00000000";
elsif Load='1' then
Qreg<=Qreg+1;
end if;

end process;

LED <= Qreg;

end behaviour;

Why do you want to do an async counter? No clock available?

What is the implementation error? (Actually, I know what it is -- but
I want to know what you think it is.)

Think about:

What happens if neither Rst nor Load are asserted?

--a
Click to expand...
 
First of all , when you say that you are implementing a counter,
what are you going to count???????

assuming whatever you are going to count is A, then
use this code to increment the count
if A'event and A = '1' -- or A = '0' whatever be the case
count := count + 1;
end if;

or else you might endup counting something else or endup in a
infinite loop as u see to be now.




"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in message news:<401e8f16$0$322$ba620e4c@news.skynet.be>...
Thanks for the info.
"Andy Peters" <Bassman59a@yahoo.com> a écrit dans le message de
news:9a2c3a75.0402020916.391b9ce6@posting.google.com...
"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in
message news:<401c1026$0$13244$ba620e4c@news.skynet.be>...

What do you mean by saying "asserted"?

"Asserted" means "put into the active state."

It's a very common term. It's useful because it nicely ignores the
detail of whether the signal is active low or active high.

--a
Click to expand...
 
I want to count pulses coming from outside the FPGA (Spartan IIe on the
Burched board) and displays the valus on a 7 digits segment.
I tried already your code. the synthesis works, but not the implementation
design. I created an UCF file to assign the pin, maybe this file creates
problem.
I do not know at which moment of the process I should assign the pins.
Georges.



"Sajan" <s_sajan_s@yahoo.com> a écrit dans le message de
news:d244d444.0402030029.47146333@posting.google.com...
First of all , when you say that you are implementing a counter,
what are you going to count???????

assuming whatever you are going to count is A, then
use this code to increment the count
if A'event and A = '1' -- or A = '0' whatever be the case
count := count + 1;
end if;

or else you might endup counting something else or endup in a
infinite loop as u see to be now.




"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in
message news:<401e8f16$0$322$ba620e4c@news.skynet.be>...
Thanks for the info.
"Andy Peters" <Bassman59a@yahoo.com> a écrit dans le message de
news:9a2c3a75.0402020916.391b9ce6@posting.google.com...
"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in
message news:<401c1026$0$13244$ba620e4c@news.skynet.be>...

What do you mean by saying "asserted"?

"Asserted" means "put into the active state."

It's a very common term. It's useful because it nicely ignores the
detail of whether the signal is active low or active high.

--a
Click to expand...
 
I think this boils down to what is considered good design practice.
If you want to count the edges of a single input line, then it really
should be considered as a clock input, and therefore you should be using
a dedicated clock input pin.

If you use a standard IO pin, then ISE will grumble about using it to
drive clock inputs. If you're stuck with this setup, then you really
should be sampling this IO line. If you're expecting a fairly low input
frequency, and a long mark and space time (high and low time) then this
should be easy enough.

In a nutshell. If you where going to do this in good old fashioned 74
series, you'd plug the signal you want to count into the counter clock
input. For similar reasons you should connect the same signal into the
FPGA clock input.

Andy

P.S. Chances are someone else knows better. I'm quite willing to be
proved wrong!!

Georges Konstantinidis wrote:
Hello Andrew,
I was endeed using a "normal I/O port" which is synchronous or asynchronous
according to the datasheets. I should be able to use it.
Or I miss something.
I found also a template in ISE called "debounce circuit" which seems to
work.
In any case I will also try what you said
Thanks everyone for you cooperation, Georges.



"Andrew Greensted" <ajg112@ohm.york.ac.uk> a écrit dans le message de
news:bvlerm$crt$1@pump1.york.ac.uk...

Try this:

library ieee;
use ieee.std_logic_1164.all;

entity counter is
port( Load: in std_logic;
Rst: in std_logic;
LED: out std_logic_vector(0 to 7));

end counter;

architecture behaviour of counter is
begin

cnt : process(Rst, Load)

variable count : std_logic_vector(7 downto 0);

begin

if (Rst = '1' then) then
count := "00000000";
else if (Load'event and Load='1') then
count := count + 1;
end if;

LED <= count;

end process cnt;

end behaviour;

Now make sure that Load is 'locked' to a clock input on the FPGA, or
you'll probably get complaints of a logic line driving clock inputs.
Try using something like:

attribute LOC : string;
attribute LOC of signal Load : signal is "pin number";

Every time that a rising edge of Lock occurs, the variable count is
incremented. LED will be loaded with the value of count each time the
process 'runs' (I assume this is what you want?).


Georges Konstantinidis wrote:

What do you mean by saying "asserted"?
"Andy Peters" <Bassman59a@yahoo.com> a écrit dans le message de
news:9a2c3a75.0401301451.70df14a6@posting.google.com...


"Georges Konstantinidis" <georges_konstantinidis@hotmail.com> wrote in

message news:<401ab556$0$777$ba620e4c@news.skynet.be>...


Dear all
I'm deseprately trying to make an asynchronous counter to count the

number


of inputs I have on a pin. I also want a reset input.
I copied the last version of my code at this e-mail .
The synthesis looks good but an error comes at the implementation

design. I


don't kow to to do any more.
Thank you for fixing my bugs, Georges.


library ieee;
use ieee.std_logic_1164.all;

entity counter is
port(Load, Rst: in std_logic;
LED: out std_logic_vector(0 to 7)
);
end counter;

architecture behaviour of counter is
signal Qreg: std_logic_vector(0 to 7);

begin
process(Rst, Load)
begin
if Rst = '1' then -- Async reset
Qreg <= "00000000";
elsif Load='1' then
Qreg<=Qreg+1;
end if;

end process;

LED <= Qreg;

end behaviour;

Why do you want to do an async counter? No clock available?

What is the implementation error? (Actually, I know what it is -- but
I want to know what you think it is.)

Think about:

What happens if neither Rst nor Load are asserted?

--a
Click to expand...
 
Georges Konstantinidis wrote:

Dear all
I'm deseprately trying to make an asynchronous counter to count the number
of inputs I have on a pin. I also want a reset input.
I copied the last version of my code at this e-mail .
The synthesis looks good but an error comes at the implementation design. I
don't kow to to do any more.
Thank you for fixing my bugs, Georges.
Click to expand...
(snip of VHDL code)

If you write it in Verilog I might be able to tell you.

I don't think it looks like an asynchronous (ripple) counter,
though. It might be that you can't make ripple counters
on the system your synthesis is targeting.

-- glen
 
You must log in or register to reply here.

Welcome to EDABoard.com

Sponsor

Back
Top