Jump to content

chiggs

Members
  • Content Count

    9
  • Joined

  • Last visited

  • Days Won

    1
  1. Hi Richard, Could you kindly explain how one could participate in the IP-XACT committee? Is it possible for non accellera members to contribute? Thanks, Chris
  2. Further comments on the code in distrib/src/dpi: 1. The splitting of vendor implementations for uvm_hdl into separate files is a step backwards. There is now no default implementation, and no attempt at all to keep a common code-base or feature set. Since PLI is a long established standard, there shouldn't be much need for vendor specific tweaks, and those that are should be minimal enough to be handled with #ifdefs in the single file. There's now much duplication, which is bad. Long term the goal should be to minimise the vendor specific hacks and converge on one universal imple
  3. I have a register map along the following lines: -----------------------+-----------------+--------------------------------- Byte offset | Register Name | Description -----------------------+-----------------+--------------------------------- 0x00 | width | Width (in bits) of structure 0x04 | height | Number of rows 0x08 | offset | Offset of the structures ********** Variable sized gap ********** $offset | Item 1 | First structure $offset + 4 | I
  4. According to this paper it should be possible to wrap SystemC using SWIG: http://glibersat.linux62.org/~glibersat/CSD13_final.pdf
  5. There are some issues with the following file: code/distrib/src/dpi/uvm_dpi.cc It includes a .cc file inside an extern "C" { block which is misleading It includes everything into a single file which is not generally considered best practice for software development since it will pollute the namespace It breaks a build systems ability to correctly determine dependencies I recommend that this file should be removed from the distribution since it is superfluous and problematic for the reasons described above. The only possible advantage of this mechanism would be a slight speed increase due to
×
×
  • Create New...