Skills Verilog Design Flow
Design, implement, and verify Verilog/SystemVerilog modules with spec-driven development, self-checking testbenches, and automated simulation workflows. Supports Synopsys VCS, Cadence Xrun, Icarus Verilog simulators, and slang static syntax checker. Use when the user needs to write Verilog modules, design digital circuits, create counters/FSMs/interfaces, simulate and verify designs, or analyze VCD waveforms.
git clone https://github.com/openclaw/skills
T=$(mktemp -d) && git clone --depth=1 https://github.com/openclaw/skills "$T" && mkdir -p ~/.claude/skills && cp -r "$T/skills/billchen1020/verilog-design" ~/.claude/skills/openclaw-skills-verilog-design-flow && rm -rf "$T"
T=$(mktemp -d) && git clone --depth=1 https://github.com/openclaw/skills "$T" && mkdir -p ~/.openclaw/skills && cp -r "$T/skills/billchen1020/verilog-design" ~/.openclaw/skills/openclaw-skills-verilog-design-flow && rm -rf "$T"
skills/billchen1020/verilog-design/SKILL.mdCore Rules
Phase 1: Understand Requirements
- Ask clarifying questions if the design spec is incomplete
- Identify: clock/reset strategy, interface signals, functionality, timing constraints
- Confirm the target: synthesis (FPGA/ASIC) or simulation only
Phase 2: Write Design Spec
- Create a markdown spec document with:
- Module name and purpose
- Port list (direction, width, description)
- Functional description
- Timing diagram (if applicable)
- Test scenarios checklist
- Store spec in memory or as a file for reference
Phase 3: Implement Verilog
- One-always-one-signal coding style: Each signal should be assigned in exactly one always block
- Separate sequential (posedge clk) and combinational (@*) logic
- Declare intermediate signals for complex logic
- Avoid mixing blocking (=) and non-blocking (<=) assignments in the same always block
- Follow synthesizable coding guidelines:
- Use
for sequential logicalways @(posedge clk) - Use
orassign
for combinational logicalways @(*) - Avoid latches (ensure all branches assign in combinational blocks)
- Explicit reset strategy (sync/async)
- Use
- Include header comments with author, date, and revision (see Version Tracking below)
- Use descriptive signal names, avoid single-letter variables
Phase 3b: Static Syntax Check with Slang
Before simulation, run static syntax checking using slang:
# Check Verilog/SystemVerilog syntax slang <module_name>.v # Or for SystemVerilog files slang <module_name>.sv
What slang checks:
- Syntax errors and parsing issues
- Type mismatches
- Undefined references
- Port connection errors
- SystemVerilog compliance
If slang reports errors:
- Fix all syntax errors before proceeding to simulation
- Pay attention to warnings about potential issues
- Re-run slang until "Build succeeded: 0 errors"
Phase 3c: Design Review Checklist
Before simulation, verify:
- Slang syntax check passes (0 errors)
- All sequential signals have explicit reset values
- No combinational logic loops (synthesis will error)
- No unintentional latches (all
/if
branches assign in combinational blocks)case - State machines have
case branchdefault - Clock domain crossing signals are properly synchronized
- Vector widths match between assignment source and destination
- Array indices are within declared bounds
- No blocking assignments (
) in sequential always blocks= - No non-blocking assignments (
) in combinational always blocks<= -
directive present in all source filestimescale
Phase 4: Write Testbench
- Create self-checking testbench using:
- Clock generator (typical:
for 10ns period)always #5 clk = ~clk; - Reset stimulus
- Input stimulus generation
- Expected output generation/comparison
or$display()
for pass/fail reporting$monitor()
after all tests complete$finish()
- Clock generator (typical:
- Save as
<module_name>_tb.v
Phase 5: Simulate with EDA Tools
The skill automatically detects and uses available simulators in priority order:
- Synopsys VCS (if
command available)vcs - Cadence Xrun (if
command available)xrun - Icarus Verilog (fallback)
Simulator Detection Logic
# Priority order: VCS → Xrun → Icarus which vcs && use_vcs which xrun && use_xrun fallback to iverilog
Synopsys VCS (Verilog)
vcs -full64 -debug_acc+all -l sim.log -R <module_name>.v <module_name>_tb.v
Synopsys VCS (SystemVerilog)
vcs -full64 -debug_acc+all -sverilog -l sim.log -R <module_name>.sv <module_name>_tb.sv
Cadence Xrun (Verilog)
xrun -64bit -access rwc -l sim.log <module_name>.v <module_name>_tb.v
Cadence Xrun (SystemVerilog)
xrun -64bit -access rwc -sv -l sim.log -R <module_name>.sv <module_name>_tb.sv
Icarus Verilog (Fallback)
iverilog -o <module_name>.vvp <module_name>.v <module_name>_tb.v vvp <module_name>.vvp
VCD Waveform Output
⚠️ Important: Always use VCD format for waveform dumping to ensure compatibility:
initial begin $dumpfile("<module_name>.vcd"); $dumpvars(0, <module_name>_tb); end
- VCS and Xrun support VCD via
/$dumpfile()$dumpvars() - FSDB format (for Verdi) is NOT supported by the VCD analysis scripts
- Keep testbench VCD-compatible for cross-simulator portability
Phase 6: Debug & Iterate
- If assertions fail or outputs incorrect:
- Review waveforms with
ORgtkwave - Use Python VCD analysis script:
python3 <skill_dir>/scripts/check_vcd.py <module>.vcd - See references/vcd-analysis.md for detailed API
- Review waveforms with
- Fix RTL bugs, update testbench if needed
- Re-simulate until all tests pass
- Update spec with any design changes
Testbench Template
`timescale 1ns/1ps module <module>_tb; // Parameters parameter CLK_PERIOD = 10; // Signals reg clk; reg rst_n; // ... add inputs/outputs // Instantiate DUT <module> dut ( .clk(clk), .rst_n(rst_n), // ... ports ); // Clock generation initial begin clk = 0; forever #(CLK_PERIOD/2) clk = ~clk; end // VCD dump initial begin $dumpfile("<module>.vcd"); $dumpvars(0, <module>_tb); end // Test stimulus initial begin // Initialize rst_n = 0; // ... init inputs // Release reset #(CLK_PERIOD * 2); rst_n = 1; // Apply test vectors // ... stimulus code // Check results // ... self-checking assertions #(CLK_PERIOD * 10); $finish(); end // Monitor initial begin $monitor("Time=%0t: signals=...", $time); end endmodule
VCD Analysis
For automated waveform checking, use Python VCD parsing. Reference: references/vcd-analysis.md
Data Storage
- Design specs: Store in
memory/verilog_specs/<module_name>_spec.md - Verilog files: Create in workspace as
<module_name>.v - Testbenches: Create as
<module_name>_tb.v - Simulation outputs: Generate
(Icarus),.vvp
, andsim.log
files.vcd
Simulator Auto-Detection Script
Use the provided helper script to automatically select and run the best available simulator:
# The script checks for VCS → Xrun → Icarus in order bash <skill_dir>/scripts/simulate.sh <module_name>
Example workflow:
# 1. Detect simulator and run bash scripts/simulate.sh counter # 2. Check simulation log cat sim.log # 3. Analyze VCD waveforms python3 scripts/check_vcd.py counter.vcd
External Tools
| Tool | Command | Purpose |
|---|---|---|
| Synopsys VCS | | Compile & Simulate Verilog |
| Synopsys VCS (SV) | | Compile & Simulate SystemVerilog |
| Cadence Xrun | | Compile & Simulate Verilog |
| Cadence Xrun (SV) | | Compile & Simulate SystemVerilog |
| Icarus Verilog | | Compile Verilog (fallback) |
| VVP | | Run simulation |
| GTKWave | | View waveforms (optional) |
Common Pitfalls
| Issue | Fix |
|---|---|
| Multiple drivers | Ensure one-always-one-signal: each signal assigned in exactly one always block |
| Latch inference | Ensure all / branches assign in combinational always |
| Missing reset | Include explicit reset in sequential always blocks |
| Race conditions | Use non-blocking in sequential logic only |
| Simulation mismatch | Check and delays |
| VCD not generated | Ensure called before |
Debugging Guide
Simulation Hangs / Freezes
| Symptom | Cause | Solution |
|---|---|---|
| No output, simulation stuck | Combinational loop | Check for circular logic in combinational always blocks |
| Infinite loop warning | Zero-delay feedback | Add delay elements or check async feedback paths |
| Division by zero | Runtime calculation error | Check divisor is never zero |
| Array out of bounds | Invalid index | Verify index range before array access |
Output Shows 'X' (Unknown)
| Symptom | Cause | Solution |
|---|---|---|
| Specific signal is X | Uninitialized register | Add explicit reset value |
| Wide bus partially X | Mixed width assignment | Check vector width consistency |
| After reset release | Reset deassertion timing | Ensure reset held long enough |
| Random X propagation | X propagation from input | Trace back to source of X |
Timing Issues
| Symptom | Cause | Solution |
|---|---|---|
| Output one cycle late | Blocking vs non-blocking | Use in sequential always blocks |
| Glitches on output | Combinational logic hazard | Add register stage or use synchronous output |
| Setup/hold violations (ASIC) | Clock/data skew | Check synthesis timing reports |
Synthesis Errors
| Error | Cause | Solution |
|---|---|---|
| "Not synthesizable" | Unsupported Verilog construct | Replace with synthesizable equivalent |
| "Multiple drivers" | Signal assigned in multiple always | Merge logic or use intermediate signals |
| "Latch inferred" | Incomplete if/case in combinational | Add default assignment or use |
| "Undriven signal" | Output declared but not assigned | Connect to logic or tie to constant |
Version Tracking
File Header Template
Every Verilog file should include:
/** * Module: <module_name> * Description: <brief description> * Author: <name> * Date: <YYYY-MM-DD> * Version: <major>.<minor>.<patch> * * Changelog: * v1.0.0 - <date> - Initial release * v1.1.0 - <date> - <description of changes> * v2.0.0 - <date> - <breaking changes> * * Parameters: * - PARAM1: <description> (default: <value>) * - PARAM2: <description> (default: <value>) * * Ports: * - clk: <description> * - rst_n: <description> * ... */
Version Numbering
- Major: Breaking changes (interface change, removed features)
- Minor: New features, backward compatible
- Patch: Bug fixes, no functional change
Git Workflow (Recommended)
# Before starting new feature git checkout -b feature/new-functionality # After completing and testing git add <files> git commit -m "feat: add <feature> to <module>" git checkout main git merge feature/new-functionality