Skip to content

feat: Dold RN5893 IMD Hardware Driver #1579

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.

Sign up for GitHub

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

Jump to bottom
Open
markxoe wants to merge 3 commits into
base: main
Choose a base branch
from
Open

feat: Dold RN5893 IMD Hardware Driver #1579

markxoe wants to merge 3 commits into from

Conversation

@markxoe
Copy link
Contributor

@markxoe markxoe commented Nov 27, 2025 •
edited
Loading

Describe your changes

  • add Dold RN5893 IMD Hardware Driver
  • add BringUp config for this driver

Issue ticket number and link

Original PR: #1395

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • I read the contribution documentation and made sure that my changes meet its requirements

@markxoe markxoe marked this pull request as draft November 27, 2025 08:55
barsnick
barsnick reviewed Nov 27, 2025
View reviewed changes
Copy link
Contributor

@barsnick barsnick left a comment

Choose a reason for hiding this comment

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

Just a quick initial read over the code.

@markxoe
feat: initial driver
fb46e72 
Signed-off-by: Mark Oude Elberink <[email protected]>
@markxoe
chore: implement review suggestions
9e6a3b2 
Signed-off-by: Mark Oude Elberink <[email protected]>
@markxoe
chore: update readme & always publish voltage
294e69b 
Signed-off-by: Mark Oude Elberink <[email protected]>
@markxoe markxoe force-pushed the feature/dold-rn5893-imd-driver branch from 04569ea to 294e69b Compare December 11, 2025 14:30
@markxoe markxoe marked this pull request as ready for review December 11, 2025 14:59
@markxoe markxoe requested a review from barsnick December 11, 2025 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Pietfried Pietfried Awaiting requested review from Pietfried Pietfried is a code owner

@hikinggrass hikinggrass Awaiting requested review from hikinggrass hikinggrass is a code owner

@corneliusclaussen corneliusclaussen Awaiting requested review from corneliusclaussen corneliusclaussen is a code owner

@cburandt cburandt Awaiting requested review from cburandt cburandt is a code owner

@SebaLukas SebaLukas Awaiting requested review from SebaLukas SebaLukas is a code owner

@mlitre mlitre Awaiting requested review from mlitre mlitre is a code owner

@florinmihut florinmihut Awaiting requested review from florinmihut florinmihut is a code owner

@djchhp djchhp Awaiting requested review from djchhp djchhp is a code owner

@barsnick barsnick Awaiting requested review from barsnick

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@markxoe @barsnick