Why is this an issue?

Active-low enables should be preferred over active-high enables.

How to fix it

Code examples

Noncompliant code example

process (clk, rst) is
begin
  if rst = '1' then
    o1 <= '0';
  elsif rising_edge(clk) then
    if enable = '1' then -- Noncompliant: Active-high enable
      o1 <= i1;
    end if;
  end if;
end process;

Compliant solution

process (clk, rst) is
begin
  if rst = '1' then
    o1 <= '0';
  elsif rising_edge(clk) then
    if enable = '0' then -- Compliant: Active-low enable
      o1 <= i1;
    end if;
  end if;
end process;

Resources

Related rules

• Active-high enables should be preferred over active-low enables