chipsalliance / f4pga

FOSS Flow For FPGA
https://f4pga.org
Apache License 2.0
358 stars 48 forks source link

Fixed CLI override function #663

Open madsrumlenordstrom opened 1 year ago

madsrumlenordstrom commented 1 year ago

This PR fixes a bug where you where not able to override top module from the CLI while using a flow file. For example like this: f4pga -vv build -f basys3.json -Dsources=[Blinky.sv] -Vtop=Blinky where basys3.json looks like this: { "default_part": "XC7A35TCPG236-1", "XC7A35TCPG236-1": { "default_target": "bitstream", "dependencies": { "build_dir": "build/basys3", "xdc": ["basys3.xdc"] } } } This resulted in an error where tool would complain that no top module if no top module is defined in basys3.json. The value from the top option was never passed to the flow configuration.