Skip to content

Proper errors for device tree#81

Merged
Kamilosok merged 4 commits into
Operacja-System:mainfrom
Kamilosok:dt_errors
Mar 20, 2026
Merged

Proper errors for device tree#81
Kamilosok merged 4 commits into
Operacja-System:mainfrom
Kamilosok:dt_errors

Conversation

@Kamilosok

Copy link
Copy Markdown
Collaborator

This PR will concern updating the device tree interface to the error handling mechanism added in [#58] (Why wasn't this split into a different PR?)

@Kamilosok Kamilosok added the refactor Changes to the code structure that don't affect behavior label Mar 19, 2026
Separated public vs external docs
@Kamilosok Kamilosok added documentation Improvements or additions to documentation and removed enhancement labels Mar 19, 2026
@Kamilosok

Copy link
Copy Markdown
Collaborator Author

Note that I didn't touch any uses of the dt library outside the scope of my work, so that will need to be changed

@Kamilosok Kamilosok marked this pull request as ready for review March 20, 2026 19:59
@Kamilosok Kamilosok force-pushed the dt_errors branch 2 times, most recently from c928333 to 94c745f Compare March 20, 2026 20:08
Documentation updated to reflect the new erroring
Comment thread include/dt/dt.h Outdated
Comment thread include/dt/dt.h Outdated
Comment thread include/dt/dt.h Outdated
Comment thread include/dt/dt.h Outdated
Comment thread src/lib/dt/dt_access.c Outdated
Comment thread src/lib/dt/dt_common.h Outdated

@Msaters Msaters left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm

Comment thread include/dt/dt.h Outdated
@Kamilosok Kamilosok merged commit 3a829fd into Operacja-System:main Mar 20, 2026
9 checks passed
@Kamilosok Kamilosok deleted the dt_errors branch March 25, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation refactor Changes to the code structure that don't affect behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants