Do I have a race condition for clk33_div?

[Tim Doyle]

I am looking to determine if the following VHDL process block creates
a race condition for clk33_div, given that it is a signal that is
being updated seemingly at same time it is being evaluated if I
understand VHDL constructs.

Any thoughts or guidelines would be greatly appreciated.

Regards,

Tim Doyle

Here is the code:


CLOCK_DIV: process (clk33, resetn)
begin
if resetn = '0' then
clk33_div <= (others =>'0');
clock_out <= '0';
elsif clk33'event and clk33 = '1' then
if clk33_div = "11000" then
clk33_div <= (others => '0');
clock_out <= not(clock_out);
else
clk33_div <= clk33_div + '1';
clock_out <= clock_out;
end if;
end if;
end process;

 

[Mike Treseler]

On Jun 12, 8:29 am, Tim Doyle <tim9doy...@gmail.com> wrote:
I am looking to determine if the following VHDL process block creates
a race condition for clk33_div, given that it is a signal that is
being updated seemingly at same time it is being evaluated

As long as I use clk33 as the only clock and
clk_div as an enable input for other logic
on the same clock, there will be no timing problems.
Think of the right side of an assignment as
gates on the D side of the flop

Barry wrote:

In VHDL, a signal is not assigned its new value until you reach the
"end process" statement

Which happens once every clock tick in Tim's example.
The code looks functional to me.
clock_out <= clock_out; is not necessary.

-- Mike Treseler

 

[Barry]

On Jun 12, 8:29 am, Tim Doyle <tim9doy...@gmail.com> wrote:

I am looking to determine if the following VHDL process block creates
a race condition for clk33_div, given that it is a signal that is
being updated seemingly at same time it is being evaluated if I
understand VHDL constructs.

Any thoughts or guidelines would be greatly appreciated.

Regards,

Tim Doyle

Here is the code:

CLOCK_DIV: process (clk33, resetn)
        begin
                if resetn = '0' then
                        clk33_div <= (others =>'0');
                        clock_out <= '0';
                elsif clk33'event and clk33 = '1' then
                        if clk33_div = "11000" then
                                clk33_div <= (others => '0');
                                clock_out <= not(clock_out);
                        else
                                clk33_div <= clk33_div + '1';
                                clock_out <= clock_out;
                        end if;
                end if;
        end process;

Hi Tim,

In VHDL, a signal is not assigned its new value until you reach the
"end process" statement (or a wait statement). This is different from
the behavior of a variable in a process.

Barry