Skip to content

Skip unsupported _CRS resource descriptors - #316

Merged
IsaacWoods merged 2 commits into
rust-osdev:mainfrom
ArthurHeymans:UnsupportedCsr
Jul 25, 2026
Merged

Skip unsupported _CRS resource descriptors#316
IsaacWoods merged 2 commits into
rust-osdev:mainfrom
ArthurHeymans:UnsupportedCsr

Conversation

@ArthurHeymans

Copy link
Copy Markdown
Contributor

Skip unsupported _CRS resource descriptors instead of panicking. Callers can still consume the MMIO/IRQ descriptors they understand from mixed resource buffers containing GPIO or GenericSerialBus descriptors.

@martin-hughes

Copy link
Copy Markdown
Contributor

Looks like this PR might have accidentally picked up the contents of #317 during a force-push?

Probably not an issue if #317 requires no changes.

@ArthurHeymans

Copy link
Copy Markdown
Contributor Author

Looks like this PR might have accidentally picked up the contents of #317 during a force-push?

Probably not an issue if #317 requires no changes.

Hi, yes I needed both in a firmware project and forgot to change the base of this one. I'm looking forward to GitHub stacked pr feature https://github.github.com/gh-stack/ that would make this easier :-)

@martin-hughes

Copy link
Copy Markdown
Contributor

Ooh - I like the looks of that! So long as I can restrain myself from creating a giant queue of PRs in stack format...

In terms of 12d461e, I think this looks good. As with #317, thanks for adding a quick test. I guess ideally there'd be a matching small descriptor test, but personally I'd be happy to merge without it. (once #317 is in)

Allow DerefOf to read directly from BufferField objects. This matches AML
patterns emitted by firmware that creates named fields over buffers and then
uses DerefOf(field).
Skip unsupported _CRS resource descriptors instead of panicking. Callers can
still consume the MMIO/IRQ descriptors they understand from mixed resource
buffers containing GPIO or GenericSerialBus descriptors.
@ArthurHeymans

Copy link
Copy Markdown
Contributor Author

I added a test for unsupported small resource descriptor. Thanks for the suggestion.

@IsaacWoods

Copy link
Copy Markdown
Member

Nice, thanks for the PR! It should definitely be possible to rebase onto another base branch, but this is fine.

@IsaacWoods
IsaacWoods merged commit f6e5b14 into rust-osdev:main Jul 25, 2026
@ArthurHeymans
ArthurHeymans deleted the UnsupportedCsr branch July 25, 2026 15:54
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