> 5. linux: according the openamp spec [3] it should now send the > ns_announcement to the co-processor (slave) > 6. co-processor: should receive the ns announcement and binds now the > endpoint > 7. co-processor: can now send messages to linux > > This is basically what I'm expecting. > > > Do you think this is a bug or is the dynamic endpoint binding not > handled? This line is there since ever [4] ... As you mentioned this is a legacy limitation that should be addressed. It is not related to this work. Thanks, Arnaud > > Any other thoughts about this? > >> This is a feature that is missing in the rpmsg virtio, and perhaps in >> rpmsg protocol >> itself (a ns annoucement ack message or something similar). >> >> >> A fix for the remoteproc virtio is available here: >> https://github.com/arnopo/meta-st-stm32mp-oss/commit/3e57fe73bd19c9bb835ac5a118e50727758b0b96 >> >> >> Don't hesitate to give me feedback on the fix, if you test it. > > I added it to my branch and till now I don't see any side effects >> >> I plan to propose the fix after this series. >> >> Thanks, >> Arnaud >> >>> >>> Cheers, >>> Philipp >>> > > Cheers, > Philipp > > [1]: > https://github.com/STMicroelectronics/STM32CubeMP1/blob/master/Projects/STM32MP157C-DK2/Applications/OpenAMP/OpenAMP_raw/Src/openamp.c#L242 > > > [2]: > https://elixir.bootlin.com/linux/v5.17-rc5/source/drivers/rpmsg/virtio_rpmsg_bus.c#L425 > > > [3]: > https://github.com/OpenAMP/open-amp/blob/main/docs/img/coprocessor-rpmsg-ns.png > > > [4]: > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=bcabbccabffe7326f046f25737ba1084f463c65c > [5] https://github.com/OpenAMP/open-amp/pull/160
