Skip to content

drivers/abp2: faulty saul init for multiple abp2 devices#21852

Merged
crasbe merged 1 commit intoRIOT-OS:masterfrom
LeonardHerbst:apb2_fix
Nov 6, 2025
Merged

drivers/abp2: faulty saul init for multiple abp2 devices#21852
crasbe merged 1 commit intoRIOT-OS:masterfrom
LeonardHerbst:apb2_fix

Conversation

@LeonardHerbst
Copy link
Copy Markdown
Contributor

Contribution description

abp2 device register two SAUL entries each.
For all but the first abp2 device the wrong SAUL info is added to the registry.

@github-actions github-actions bot added Area: drivers Area: Device drivers Area: SAUL Area: Sensor/Actuator Uber Layer labels Nov 6, 2025
Copy link
Copy Markdown
Contributor

@mguetschow mguetschow left a comment

Choose a reason for hiding this comment

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

Nice one, looks straightforward enough, thanks for spotting and fixing!

@mguetschow mguetschow added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Nov 6, 2025
@riot-ci
Copy link
Copy Markdown

riot-ci commented Nov 6, 2025

Murdock results

✔️ PASSED

3672dbc drivers/abp2: faulty saul init for multiple abp2 devices

Success Failures Total Runtime
10915 0 10916 09m:38s

Artifacts

@benpicco benpicco added the Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) label Nov 6, 2025
@crasbe crasbe added this pull request to the merge queue Nov 6, 2025
Merged via the queue into RIOT-OS:master with commit fddf362 Nov 6, 2025
28 checks passed
@LeonardHerbst LeonardHerbst deleted the apb2_fix branch December 1, 2025 09:46
@leandrolanzieri leandrolanzieri added this to the Release 2026.01 milestone Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: drivers Area: Device drivers Area: SAUL Area: Sensor/Actuator Uber Layer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants