search

OpenAMP / open-amp

The main OpenAMP library implementing RPMSG, Virtio, and Remoteproc for RTOS etc
https://www.openampproject.org/
Other
721 stars 291 forks source link

rpmsg_virtio: fix rpmsg_virtio_get_tx_payload_buffer() error #624

Closed CV-Bowen closed 1 month ago

CV-Bowen commented 1 month ago

If rpmsg_virtio_notify_wait() returns RPMSG_SUCCESS, we should not directly return NULL, but call rpmsg_virtio_get_tx_buffer to get the tx buffer again.

This is the new PR of https://github.com/OpenAMP/open-amp/pull/614, and you can find the detail discussions in https://github.com/OpenAMP/open-amp/pull/614

arnopo commented 1 month ago

@edmooring @tnmysh To be able to freeze the code for the release and start non-regression tests, I will merged this change without your review. I considered this fix simple and that you have reviewed it as a subpart of PR #614, which you already reviewed. Please tell me if this doesn't suit you, I will revert it.