Skip to content

iio: adc: adrv9002: fix frequency hopping table configuration#2967

Merged
nunojsa merged 1 commit intomainfrom
staging/xlnx/fix-adrv9002-fh
Oct 6, 2025
Merged

iio: adc: adrv9002: fix frequency hopping table configuration#2967
nunojsa merged 1 commit intomainfrom
staging/xlnx/fix-adrv9002-fh

Conversation

@nunojsa
Copy link
Copy Markdown
Collaborator

@nunojsa nunojsa commented Oct 2, 2025

PR Description

Fix multiple issues in the frequency hopping bin table write function:

  • Correct typo "bellow" to "below" in comment
  • Fix off-by-one error in entry bounds check (should be >= not >)
  • Use correct rx20_if variable for rx2OffsetFrequencyHz instead of rx10_if
  • Use correct rx2_gain variable for rx2GainIndex instead of rx1_gain

These fixes ensure proper configuration of RX2 channel parameters and prevent potential buffer overflow when the entry index equals max_sz.

PR Type

  • Bug fix (a change that fixes an issue)
  • New feature (a change that adds new functionality)
  • Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

  • I have conducted a self-review of my own code changes
  • I have tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly (if there is the case)

Fix multiple issues in the frequency hopping bin table write function:
- Correct typo "bellow" to "below" in comment
- Fix off-by-one error in entry bounds check (should be >= not >)
- Use correct rx20_if variable for rx2OffsetFrequencyHz instead of rx10_if
- Use correct rx2_gain variable for rx2GainIndex instead of rx1_gain

These fixes ensure proper configuration of RX2 channel parameters and
prevent potential buffer overflow when the entry index equals max_sz.

Signed-off-by: Nuno Sá <nuno.sa@analog.com>
@nunojsa nunojsa merged commit f7a682b into main Oct 6, 2025
28 checks passed
@nunojsa nunojsa deleted the staging/xlnx/fix-adrv9002-fh branch October 6, 2025 09:39
github-actions Bot pushed a commit that referenced this pull request Oct 6, 2025
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