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

adi/ad908[1|4]_mc.py: Fix label parsing in _map_to_dict() #490

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

mhennerich
Copy link
Contributor

Description

In recent kernels the IIO core adds labels for all channels with extended names. This now causes undesired behavior. Before splitting, check if the split string is part of the string. This will also catch the 'buffer_only' case.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How has this been tested?

On HW using Linux 6.1

  • Test A
  • Test B

Test Configuration:

  • Hardware:
  • OS:

Documentation

If this is a new feature or example please mention or link any documentation. All new hardware interface classes require documentation.

In recent kernels the IIO core adds labels for all channels
with extended names. This now causes undesired behavior.
Before splitting, check if the split string is part of the string.
This will also catch the 'buffer_only' case.

Signed-off-by: Michael Hennerich <[email protected]>
@mhennerich mhennerich requested a review from tfcollins October 25, 2023 08:17
@github-actions
Copy link

github-actions bot commented Oct 25, 2023

Unit Test Results

1 249 tests  ±0      247 ✔️ ±0   5m 6s ⏱️ +6s
       1 suites ±0   1 002 💤 ±0 
       1 files   ±0          0 ±0 

Results for commit 3aec7b7. ± Comparison against base commit 1ee2658.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

Generated documentation for this PR is available at Link

Copy link
Collaborator

@tfcollins tfcollins left a comment

Choose a reason for hiding this comment

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

Change is fine. Think we can get the updated context files?

@tfcollins tfcollins changed the base branch from master to main October 26, 2023 17:12
@tfcollins tfcollins merged commit 7c3d411 into main Oct 26, 2023
@tfcollins tfcollins deleted the fix-map_to_dict-ad9081-ad9084 branch October 26, 2023 17:12
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