Skip to content

Enhance rpmsg buf config#684

Open
tnmysh wants to merge 1 commit into
OpenAMP:mainfrom
tnmysh:enhance_rpmsg_buf_config
Open

Enhance rpmsg buf config#684
tnmysh wants to merge 1 commit into
OpenAMP:mainfrom
tnmysh:enhance_rpmsg_buf_config

Conversation

@tnmysh
Copy link
Copy Markdown
Collaborator

@tnmysh tnmysh commented Apr 29, 2026

@tnmysh
Copy link
Copy Markdown
Collaborator Author

tnmysh commented Apr 29, 2026

@arnopo

we can review this one after the release.

@tnmysh tnmysh force-pushed the enhance_rpmsg_buf_config branch 2 times, most recently from 7310369 to 3474c42 Compare May 29, 2026 20:12
@tnmysh tnmysh marked this pull request as ready for review May 29, 2026 20:13
@tnmysh tnmysh requested a review from wmamills May 29, 2026 20:13
@tnmysh tnmysh force-pushed the enhance_rpmsg_buf_config branch from 3474c42 to 6ddcf11 Compare May 30, 2026 02:40
Introduce new feature bit that allows rpmsg buffer size via virtio
device config space. If the feature is available then, driver will set
single rpmsg buffer size from virtio device config space in the resource
table.

New virtio feature allows to configure rpmsg single buffer size via
virtio device config space. Hence, rx buffer size can be provided by
vdev device config space as well. If BUFSZ feature bit is set, then use
rx buffer size from the virtio config space in the resource table.

Signed-off-by: Tanmay Shah <tanmay.shah@amd.com>
@tnmysh tnmysh force-pushed the enhance_rpmsg_buf_config branch from 6ddcf11 to 7c0b658 Compare May 30, 2026 02:47
Comment thread lib/rpmsg/rpmsg_virtio.c
return status;

if (features & (1 << VIRTIO_RPMSG_F_BUFSZ))
rvdev->config = *config;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Here you should use virtio_read_config to retrieve the config from the transport layer
Did you have a look to @xiaoxiang781216 work in #155 ?
Seems to me the good approach

@arnopo arnopo requested a review from edmooring June 2, 2026 09:06
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.

2 participants