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

AD400x: fix sign for even-numbered devices #51

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

threexc
Copy link
Contributor

@threexc threexc commented Feb 5, 2025

AD400x: fix sign for even-numbered devices

The channel name property in Base.m has to be removed, since it varies
between differential and pseudo-differential devices.

Also change where the SampleRate property and the setters are defined - now there's a version in each device class.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation by running /docs/source/reference_api/gen_sysobj_doc.m script locally, if applies.

@threexc threexc changed the title Tgamblin/ad400x sign AD400x: fix sign for even-numbered devices Feb 5, 2025
@threexc threexc force-pushed the tgamblin/ad400x_sign branch 2 times, most recently from bb133b5 to c86aac5 Compare February 5, 2025 21:11
The channel name property in Base.m has to be removed, since it varies
between differential and pseudo-differential devices.

The SampleRate functionality has been simplified and moved into each
individual class, as attempts to make it work with varying channel names
as a per-channel property while the value and getter/setter methods
themselves were defined in AD400x/Base.m didn't seem to work.

Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
Run gen_sysobj_doc.m and commit the changes.

Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
@threexc threexc force-pushed the tgamblin/ad400x_sign branch from c86aac5 to 0bb41e8 Compare February 5, 2025 22:04
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.

1 participant