Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DebugBridge fails to connect #1429

Closed
jacopoabramo opened this issue Aug 22, 2023 · 3 comments · Fixed by #1445
Closed

DebugBridge fails to connect #1429

jacopoabramo opened this issue Aug 22, 2023 · 3 comments · Fixed by #1445

Comments

@jacopoabramo
Copy link

Greetings,

I tried using the DebugBridge class together with the debug bridge IP in my design but could not make it work. I documented the problem in this post. Does the original author of this class have some idea on what am I doing wrong?

  • PYNQ version 3.0.1
  • board PYNQ-ZU
  • IP generated with Vivado 2023.1

Thanks in advance.

Best,

Jacopo

adhituua added a commit to adhituua/PYNQ that referenced this issue Sep 8, 2023
@fpjentzsch
Copy link
Contributor

Hi,

looks like a simple bug to me:
The DebugBridge constructor checks if the description is of type ReprDict (this line) while other drivers, such as the DMA, check for ReprDict or dict (here).

I was able to work around this using:

from pynq.lib.debugbridge import DebugBridge
from pynqutils.runtime.repr_dict import ReprDict

bridge_desc = ReprDict(ol.ip_dict["debug_bridge_0"]) # ol is the Overlay
bridge = DebugBridge(bridge_desc)
bridge.start_xvc_server(bufferLen=4096, serverAddress="0.0.0.0", serverPort=2542, reconnect=True, verbose=True)

@mariodruiz
Copy link
Collaborator

That's a good point @fpjentzsch. That check is not really necessary, although it can be fixed as you suggest.
Could you do a PR?

@fpjentzsch
Copy link
Contributor

That's a good point @fpjentzsch. That check is not really necessary, although it can be fixed as you suggest. Could you do a PR?

Sure: #1445

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants