CV-Bowen opened a new pull request, #21402: URL: https://github.com/apache/nuttx/pull/21402
## Summary Print a line before and after `rpmsg_port_register()` and `rpmsg_port_unregister()`, so a log shows whether a port came up or went down and where it stopped. `rpmsgvbs` moves from `rpmsg_port_uart.c` to `rpmsg_port.h` so every port transport can use it. ## Impact Log only, printed through `rpmsgerr` (needs `CONFIG_DEBUG_RPMSG_ERROR`). No API or Kconfig change. ## Testing Host: Ubuntu 22.04.5 LTS x86_64, QEMU version 10.2.1, sim: cc (Ubuntu 13.1.0-8ubuntu1~22.04) 13.1.0, qemu-armv8a: aarch64-none-elf-gcc (Arm GNU Toolchain 12.2.Rel1 (Build arm-12.24)) 12.2.1 20221205 qemu-armv8a:rpserver + qemu-armv8a:rpproxy (port uart link) with `CONFIG_DEBUG_RPMSG=y` and `CONFIG_DEBUG_RPMSG_ERROR=y` added on top (test-only): ``` [ 0.139636] [ 5] [ ERROR] [proxy] rpmsg_port_register: start register server2 [ 0.141835] [ 5] [ ERROR] [proxy] rpmsg_port_register: register server2 success [ 0.310938] [ 5] [ ERROR] [server] rpmsg_port_register: start register proxy2 [ 0.312046] [ 5] [ ERROR] [server] rpmsg_port_register: register proxy2 success ``` Baseline with the default configs: No regression in rpmsg: sim:rpserver/rpproxy and qemu-armv8a:rpserver/rpproxy boot and pass the fixed rpmsg cases (dump, ping, rpmsgfs; 15/15). qemu-armv8a was built with LIBC_ENVPATH and BINFMT_CHECKEXEC disabled, which current master needs for nxinit to start nsh there; unrelated to this change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
