Add test coverage for ipbus ram slaves

ipbus / ipbus-firmware

Firmware that implements a reliable high-performance control link for particle physics electronics, based on the IPbus protocol

https://ipbus.web.cern.ch

Other

41 stars 32 forks source link

Add test coverage for ipbus ram slaves #81

Closed alessandrothea closed 5 years ago

alessandrothea commented 5 years ago

The existing automatic tests don't provide appropriate coverage of the ipbus ram slave code. Dedicated tests have to be added, covering

VHDL Syntax
Interface with ipbus
interface with the payload clock logic

The latter will require adding a dedicated pattern generator.

alessandrothea commented 5 years ago

Note: Both dpram36 and spdpram72 were found to be inferred as distributed RAMs while expected to be implemented as block rams. The code has been slightly re-arranged according to the Xilinx prescriptions. They are both inferred as block RAMs, now.