Skip to content

Increase MIPI hw reset sleep from 3 to 8 seconds#14865

Merged
Nir-Az merged 1 commit into
realsenseai:developmentfrom
Nir-Az:increase-mipi-hw-reset-sleep-v2
Mar 25, 2026
Merged

Increase MIPI hw reset sleep from 3 to 8 seconds#14865
Nir-Az merged 1 commit into
realsenseai:developmentfrom
Nir-Az:increase-mipi-hw-reset-sleep-v2

Conversation

@Nir-Az

@Nir-Az Nir-Az commented Mar 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Increase the Python-side sleep after MIPI hardware reset from 3 to 8 seconds
  • MIPI devices (e.g. D457) don't physically disconnect/reconnect — the C++ layer simulates a 3s reconnect in d400-mipi-device.cpp, but the Python wait was too short for reliable re-enumeration on some machines
  • We add this as a WA for a MIPI driver 1.0.2.23 reggresion

Test plan

  • Run hw-reset tests on a GMSL/MIPI machine to verify devices come back online reliably

Copilot AI review requested due to automatic review settings March 25, 2026 13:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR increases the post–hardware-reset wait in the Python test utilities to improve reliability of re-enumeration for non-removable (e.g., MIPI) devices where the device may not physically disconnect/reconnect.

Changes:

  • Increase the fixed sleep after HW reset for non-removable devices from 3s to 8s.
  • Keep existing reset flow: trigger hardware_reset(), then wait for devices to be considered online again.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread unit-tests/py/rspy/devices.py
Comment thread unit-tests/py/rspy/devices.py
@Nir-Az Nir-Az requested a review from sareluzi March 25, 2026 13:23
@Nir-Az Nir-Az merged commit 705f62d into realsenseai:development Mar 25, 2026
30 of 31 checks passed
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 this pull request may close these issues.

3 participants