search

tukl-msd / msdlib.loopy

the Loopy hardware-in-the-loop framework
3 stars 4 forks source link

Only lower-case instances in ISE14.6 #4

Closed cdeschryver closed 10 years ago

cdeschryver commented 10 years ago

Xilinx decided to allow only lower-case identifiers for hardware instances either since 14.5 or 14.6. Cores injected by the driver, like muxes and queues, are upper-case. Furthermore, the names of cores can be checked before starting the backend.

Thomas Fischer (8/7/2013 11:22 AM): While changing the core names is basically no problem, the name checking of user-defined cores needs some thought. The naming is probably depending on the workflow (i.e. vivado might allow upper-case names, some third-party workflow might allow them to or have other restrictions. Consequently, a namespace check should be integrated in the workflow backend, not in the frontend.

cdeschryver commented 10 years ago

check if bug is still there in latest ISE release

AndrewSftD commented 10 years ago

Update: bug seems to still be there (at least in the ISE versions loopy is using), but:

Me and Mathias have fixed the loopy generics, basically loopy now works, and as long as only lower case instance names are used in the provided vhdl files, there should be no problem any more.

cdeschryver commented 10 years ago

marked as wontfix + closed