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

I2CError on some configurations Urukul #2271

Open
thomasfire opened this issue Nov 1, 2023 · 4 comments
Open

I2CError on some configurations Urukul #2271

thomasfire opened this issue Nov 1, 2023 · 4 comments
Assignees

Comments

@thomasfire
Copy link
Contributor

Bug Report

One-Line Summary

I2CError on some configurations' Urukul

Issue Details

Steps to Reproduce

  1. Build release-7 with following configuation:
{
  "target": "kasli",
  "min_artiq_version": "7.0",
  "variant": "test",
  "hw_rev": "v2.0",
  "base": "satellite",
  "peripherals": [
    {
      "type": "urukul",
      "dds": "ad9910",
      "ports": [0, 1],
      "synchronization": true,
      "clk_sel": 2
    },
    {
      "type": "phaser",
      "ports": [2]
    },
    {
      "type": "grabber",
      "ports": [3, 4]
    },
    {
      "type": "zotino",
      "ports": [5]
    }
  ]
}
  1. Test Urukuls with artiq_sinara_tester

Expected Behavior

Tests pass as usual.

Actual (undesired) Behavior

During calibration of inter-device synchronization:

*** Testing Urukul DDSes.
  File "/nix/store/8bzpg145r6vbqssfyva0hw8zcwpv3hiw-python3.9-artiq-7.8185.cc81464/bin/.artiq_sinara_tester-wrapped", line 9, in <module>
    sys.exit(main())
  File "/nix/store/8bzpg145r6vbqssfyva0hw8zcwpv3hiw-python3.9-artiq-7.8185.cc81464/lib/python3.9/site-packages/artiq/frontend/artiq_sinara_tester.py", line 788, in main
    experiment.run(tests)
  File "/nix/store/8bzpg145r6vbqssfyva0hw8zcwpv3hiw-python3.9-artiq-7.8185.cc81464/lib/python3.9/site-packages/artiq/frontend/artiq_sinara_tester.py", line 739, in run
    getattr(self, f"test_{name}")()
  File "/nix/store/8bzpg145r6vbqssfyva0hw8zcwpv3hiw-python3.9-artiq-7.8185.cc81464/lib/python3.9/site-packages/artiq/frontend/artiq_sinara_tester.py", line 299, in test_urukuls
    sync_delay_seed, io_update_delay = self.calibrate_urukul(channel_dev)
  File "/nix/store/8bzpg145r6vbqssfyva0hw8zcwpv3hiw-python3.9-artiq-7.8185.cc81464/lib/python3.9/site-packages/artiq/language/core.py", line 54, in run_on_core
    return getattr(self, arg).run(run_on_core, ((self,) + k_args), k_kwargs)
  File "/nix/store/8bzpg145r6vbqssfyva0hw8zcwpv3hiw-python3.9-artiq-7.8185.cc81464/lib/python3.9/site-packages/artiq/coredevice/core.py", line 140, in run
    self._run_compiled(kernel_library, embedding_map, symbolizer, demangler)
  File "/nix/store/8bzpg145r6vbqssfyva0hw8zcwpv3hiw-python3.9-artiq-7.8185.cc81464/lib/python3.9/site-packages/artiq/coredevice/core.py", line 130, in _run_compiled
    self.comm.serve(embedding_map, symbolizer, demangler)
  File "/nix/store/8bzpg145r6vbqssfyva0hw8zcwpv3hiw-python3.9-artiq-7.8185.cc81464/lib/python3.9/site-packages/artiq/coredevice/comm_kernel.py", line 716, in serve
    self._serve_exception(embedding_map, symbolizer, demangler)
  File "/nix/store/8bzpg145r6vbqssfyva0hw8zcwpv3hiw-python3.9-artiq-7.8185.cc81464/lib/python3.9/site-packages/artiq/coredevice/comm_kernel.py", line 698, in _serve_exception
    raise python_exn
artiq.coredevice.exceptions.I2CError: failed to ack bus address

If you move Urukul ports to 8,9 it will work fine.

Your System (omit irrelevant parts)

  • Operating System: NixOS
  • ARTIQ version: 7
@Spaqin
Copy link
Collaborator

Spaqin commented Nov 1, 2023

Any messages about timing not being met in the build log?

@thomasfire
Copy link
Contributor Author

Any messages about timing not being met in the build log?

None I could find
build.log

@bodokaiser
Copy link

@thomasfire did you solve the problem?

@bodokaiser
Copy link

bodokaiser commented Apr 29, 2024

@Spaqin According to this forum post, the problem arises as there is aliasing for thee urukulX_eeproms when generating the device_db.py.

I believe #2149 was created to document the bug. We can close one of the issues.

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

No branches or pull requests

3 participants