Skip to content

Commit

Permalink
ecp_vlog: update INDD comment and remove cells_sim include
Browse files Browse the repository at this point in the history
  • Loading branch information
nneonneo committed Jun 25, 2021
1 parent 518065e commit 5ece3c9
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions tools/ecp_vlog.py
Original file line number Diff line number Diff line change
Expand Up @@ -452,9 +452,11 @@ def filter_node(node: Node) -> bool:
# 07:55 <daveshah> They are for the dedicated interconnect between IOLOGIC and PIO
return False
if node.pin_name == "INDD":
# I don't know what this pin is, but it often appears to be connected to $DI.
# Disabling it because sometimes it ends up in a multi-root configuration with PIO$O,
# which makes it (probably) redundant?
# INDD is the input after the delay block. This is currently redundant because
# the input source (PIO$O) will be exposed as an independent input, so the module's
# caller can simply hard-code an appropriate delay to the module input.
# If the I/O modules are ever implemented, it will be necessary to disambiguate
# PIO$O from INDD for the IOLOGIC$DI input to avoid a multi-root situation.
return False
return True

Expand Down Expand Up @@ -622,9 +624,9 @@ def print_definition(cls) -> None:

print(
f"""
/* Use the cells_sim library from yosys/techlibs/ecp5 */
`define NO_INCLUDES 1
`include "cells_sim.v"
/* This module requires the cells_sim library from yosys/techlibs/ecp5/cells.sim.v
for the TRELLIS_SLICE definition. Include that cell library before including this
file. */
module ECP5_SLICE(
input {", ".join(cls.input_pins)},
output {", ".join(cls.output_pins)}
Expand Down

0 comments on commit 5ece3c9

Please sign in to comment.