ipbus/ipbus-firmware

Missing vhd file in ipbus_transport_axi dependency file

Closed this issue · 4 comments

Hi,

The dep file components/ipbus_transport_axi/firmware/cfg/ipbus_transport_axi.dep requires the test-bench file ipbus_transport_ram_if_simtb.vhd. It does not, however, require the ipbus_example VHDL file, which is used by that test-bench. (Or, as alternative, split up the synth and sim parts of the IPBus transport business, but that is maybe uglier.)

Cheers,
Jeroen

Hi,

Good point! Since most people using ipbus_transport_axi will not need the sim testbench, I'd prefer instead splitting out src ipbus_transport_ram_if_simtb.vhd into a new dep file, ipbus_transport_axi_simtb.dep, and including ipbus_example.vhd from that dep file.

Cheers,
Tom

Crap ;-) Let me go an cancel my pull request then, and make a new one.

Jeroen

Ok, let's try this one then: #138.

Ok. This was handled in pull request #138. All happy now.