Skip to content

Siglent driver update for latest SDS firmwares#1023

Merged
azonenberg merged 10 commits intongscopeclient:masterfrom
fredzo:siglent-driver-fix
Dec 9, 2025
Merged

Siglent driver update for latest SDS firmwares#1023
azonenberg merged 10 commits intongscopeclient:masterfrom
fredzo:siglent-driver-fix

Conversation

@fredzo
Copy link
Copy Markdown
Contributor

@fredzo fredzo commented Dec 3, 2025

Hi Andrew,

This PR is an update of Siglent driver for latest SDS firmwares (aka E11 protocol).
It's been tested on my SDS 2000X HD after updating to latest 1.2.3.1 firmware (from 11/4/2025).
This firmware version comes with a few changed:

  • Firmware version now has patch uboot number: identification logic has been adapted for it
  • The waveform size reported in 12bit (HD) mode has been fixed by Siglent
  • The pagination issue with digital channes have been fixed by siglent
  • A new quirkiness has been introduced: MAXP? value (max page size) is not always honored by the scope that sometimes uses smaller pages. Pagination logic in the driver has been updated to dynamically adapt page size after first page when this occurs.

This PR also fixes other issues reported here : #1016

  • Fixes SIGSEGV when pagination fails
  • Better handling of protocol errors
  • Fixe for setting sample rate value

Other fixes:

  • Fixes digital acquisition when no analog channel is active
  • Fixes data reading for large waveform when transport is buffering

Please let me know if you have any comment.

Best,

Frederic.

Frederic Borry and others added 10 commits November 25, 2025 00:29
Updated SDS2000X HD driver for 1.2.3.1 firmware version (no longer needs size workaround + new version string pattern with uboot patch version number).
Added sanity check on return waveform data size to prevent buffer overflows when the scope reports an inconsistent size.
Fixed concurrent access when changing sample rate while acquisition is running.
Fixed sample depth modification for E11 protocol.
@azonenberg
Copy link
Copy Markdown
Collaborator

@fredzo is this ready to merge? It looks like it doesn't fix all the #1016 problems yet? I want to get it in before v0.1.1 regardless.

@azonenberg azonenberg added this to the v0.1.1 milestone Dec 9, 2025
@fredzo
Copy link
Copy Markdown
Contributor Author

fredzo commented Dec 9, 2025

Well up to you @azonenberg , but I think you can merge it as it is : it already fixes a good part of the problems reported in #1016 (at least the ones that I was able to reproduce on my SDS2000X HD) + it adds support for the latest SDS2000X HD firmware.
I'll create another PR if needed after Adreas report.

@azonenberg
Copy link
Copy Markdown
Collaborator

OK so ready to merge but we'll keep #1016 open until confirmed separately? Got it.

@azonenberg azonenberg merged commit 390070a into ngscopeclient:master Dec 9, 2025
@fredzo fredzo deleted the siglent-driver-fix branch December 9, 2025 20:18
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.

2 participants