chili-chips-ba / openeye-CamSI

A truly opensource camera serial interface. No frills. No backdoors that compromise security. Outstanding signal integrity. Hi-rez video pipeline with remote connectivity. For Sony, Series7 & open FPGA makers on limited budget. Augments openXC7 CI/CD, challenging its timing-savvy. Promotes the lesser-known EU boards.
https://nlnet.nl/project/TISG
BSD 3-Clause "New" or "Revised" License
24 stars 5 forks source link

openXC7 - slave_oserdese has disconnected OQ/OFB output ports #31

Closed kerimbavcic closed 12 hours ago

kerimbavcic commented 2 days ago

NOTE: when using Vivado, the issue is not present!

When using openXC7 (place and route step) we get an error in OSERDESE2 module, slave_oserdese to be specific, where it says OQ and OFB ports are not connected. The exact message we get is:

ERROR: OSERDESE2 'u_hdmi_top.u_hdmi_backend._tdms_sdat[0].u_oser_dat.slave_oserdese' has disconnected OQ/OFB output ports
2 warnings, 1 error
make: *** [Makefile:53: top.fasm] Error 25

Here is the code snippet: image

openXC7 installed version (https://github.com/openXC7/toolchain-installer):

We noticed that openXC7 had some issues with OSERDESE2 before in the past which were fixed as part of this bugfix.

chili-chips-ba commented 2 days ago
hansfbaier commented 1 day ago

@chili-chips-ba Can I have a test design, that highlights your issue?

chili-chips-ba commented 1 day ago

... RTL is already checked in. @kerimbavcic and @harunkovacevic are in the process of checking in the openXC7 part of the setup:

They'll ping you shortly...

harunkovacevic commented 1 day ago

@hansfbaier You can find all necessary files here:

AdamLee7 commented 1 day ago

i have fix this error, I'll submit the code in a moment

AdamLee7 commented 1 day ago

https://github.com/openXC7/nextpnr-xilinx/pull/53 I fixed the bug in the pack code, but due to other missing logic in packing and routing, and incomplete prjxray-db information, your netlist may still not work properly.

chili-chips-ba commented 1 day ago

@AdamLee7 what is your recommendation for our next step?

hansfbaier commented 1 day ago

@chili-chips-ba If your tests succeeds I will merge. The patch looks good to me so far.

AdamLee7 commented 1 day ago

@AdamLee7 what is your recommendation for our next step?

  • Is it to simply pull in and try your patch
  • then keep reporting issues as we come across them?!

yes, just do it, if there is any problem, you can commit an issue

harunkovacevic commented 12 hours ago

We have cloned nextpnr-xilinx repo, adapted _xilinx/pack_ioxc7.cc file, compiled and installed a new nextpnr version. Then we ran a build and "disconnected ports OSERDESE2 issue" was fixed but we've gotten a new error explained in this issue.