Skip to content

Conversation

@danmois
Copy link
Contributor

@danmois danmois commented Dec 4, 2025

PR Description

Add per-channel debugfs entries that provide comprehensive DPD (Digital Pre-Distortion) performance metrics and diagnostics.

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 compiled my changes, including the documentation
  • I have tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly
  • I have provided links for the relevant upstream lore

Copy link
Collaborator

@nunojsa nunojsa left a comment

Choose a reason for hiding this comment

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

Just one minor comment


if (ret) {
seq_printf(s, "Error: Failed to get DPD status for TX%u\n", chan);
return 0;
Copy link
Collaborator

Choose a reason for hiding this comment

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

The above is odd! Normal dev_err() plus proper return ret

mutex_lock(&phy->lock);
ret = adi_adrv9025_DpdStatusGet_v2(phy->madDevice, txChannel, &dpdStatus);
mutex_unlock(&phy->lock);

Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: could avoid this blank line

Fix memory leak for phy->agcConfig by using devm_kzalloc for
allocating memory instead of kzalloc.

Fixes: 43f0025 ("drivers: iio: adc: madura: adrv9025.c: AGC settings")
Signed-off-by: George Mois <george.mois@analog.com>
Add per-channel debugfs entries that provide comprehensive DPD
(Digital Pre-Distortion) performance metrics and diagnostics.

Signed-off-by: George Mois <george.mois@analog.com>
@danmois danmois force-pushed the staging/madura_dpd_adv_stat branch from d875137 to d879c98 Compare December 12, 2025 18:26
@danmois
Copy link
Contributor Author

danmois commented Dec 12, 2025

V2

  • remove blank line
  • display error message and return error code in case adi_adrv9025_DpdStatusGet_v2 fails

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