Uio update 20260827 - #119
Open
bentheredonethat wants to merge 3 commits into
Open
bentheredonethat wants to merge 3 commits into
bentheredonethat wants to merge 3 commits into
Conversation
tnmysh
previously approved these changes
Sep 18, 2026
Collaborator
|
The changes look fine. Could you please add how host side device-tree looks like in some README? Thanks, |
tnmysh
self-requested a review
September 18, 2026 17:14
tnmysh
dismissed
their stale review
September 18, 2026 17:15
Posted comment after the ack. So removing the ack.
Open the Linux irq_shmem_demo host path on the libmetal UIO bus using stable logical names exposed through /sys/class/uio/uioX/name. Read the host IPI remote kick bit from the backing device-tree node through libmetal,uio-ipi-bitmask, and derive descriptor and payload sizes from the opened IO regions so the host demo no longer depends on generated device names or carveout size macros. Signed-off-by: Ben Levinsky <ben.levinsky@amd.com>
Remove amd_linux_userspace host-side config header generation now that the Linux demo opens fixed logical UIO devices and derives the remaining runtime values directly from libmetal IO regions and the backing device tree. Drop config.h.in and the configure-time CMake path, and update the Linux host README to document the required UIO logical names, the libmetal,ipi-remote-mask property, and the direct run flow. Signed-off-by: Ben Levinsky <ben.levinsky@amd.com>
bentheredonethat
force-pushed
the
uio-update-20260827
branch
from
September 21, 2026 15:12
a64e3e6 to
c875243
Compare
Add a host-side device-tree example to the Linux userspace README so reviewers can see how the shared-memory carveouts, UIO logical names, timer node, and IPI bitmask are wired on a real platform. Also fix the documented IPI property name to match what the host code reads at runtime. Signed-off-by: Ben Levinsky <ben.levinsky@amd.com>
tnmysh
approved these changes
Sep 21, 2026
tnmysh
approved these changes
Sep 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
pivot Libmetal Linux demo to use new UIO device discovery