Make the kernel I/O output character device flushing configurable. The bsp_reset() function should reset the unit and do nothing else.
The arm/xilinx-zynq and arm/xilinx-zynqmp BSPs were the only ones doing an UART flush in bsp_reset(). The bsp_reset() function should reset the system and do nothing more. Doing additional things like flushing an UART device may not make sense for all applications. Some applications may not use the UART device, so it may not be initialized and powered off. Some applications may use it with an application-specific protocol which doesn't like the additional four '\r' during reset. Doing a UART flush takes some time and some applications may prefer a fast reset cycle. The bsp_reset() is the wrong place to do specific cleanup functions. Flushing the kernel I/O output character device is now optionally done in bsp_fatal_extension() depending on the new BSP option BSP_FLUSH_KERNEL_CHAR_OUTPUT. --- bsps/aarch64/xilinx-zynqmp/console/console.c | 3 ++- bsps/aarch64/xilinx-zynqmp/include/bsp.h | 2 -- bsps/arm/xilinx-zynq/console/debug-console.c | 10 +++++++++ bsps/arm/xilinx-zynq/start/bspreset.c | 4 ---- .../console/console-config.c | 3 ++- bsps/arm/xilinx-zynqmp-rpu/include/bsp.h | 2 -- bsps/arm/xilinx-zynqmp-rpu/start/bspreset.c | 3 --- .../xilinx-zynqmp/console/console-config.c | 2 +- bsps/arm/xilinx-zynqmp/include/bsp.h | 2 -- bsps/arm/xilinx-zynqmp/start/bspreset.c | 4 +--- bsps/include/bsp/bootcard.h | 5 +++++ bsps/shared/start/bspfatal-default.c | 4 ++++ spec/build/bsps/bspopts.yml | 2 ++ spec/build/bsps/optflushkerncharout.yml | 22 +++++++++++++++++++ 14 files changed, 49 insertions(+), 19 deletions(-) create mode 100644 spec/build/bsps/optflushkerncharout.yml diff --git a/bsps/aarch64/xilinx-zynqmp/console/console.c b/bsps/aarch64/xilinx-zynqmp/console/console.c index 9ce0b1da63..d1b2850952 100644 --- a/bsps/aarch64/xilinx-zynqmp/console/console.c +++ b/bsps/aarch64/xilinx-zynqmp/console/console.c @@ -41,6 +41,7 @@ #include <rtems/termiostypes.h> #include <bsp/aarch64-mmu.h> +#include <bsp/bootcard.h> #include <bsp/fdt.h> #include <bsp/irq.h> @@ -234,7 +235,7 @@ rtems_status_code console_initialize( return RTEMS_SUCCESSFUL; } -void zynqmp_debug_console_flush(void) +void bsp_flush_kernel_char_output(void) { zynq_uart_reset_tx_flush(&zynqmp_uart_instances[BSP_CONSOLE_MINOR]); } diff --git a/bsps/aarch64/xilinx-zynqmp/include/bsp.h b/bsps/aarch64/xilinx-zynqmp/include/bsp.h index 0ccca8b196..d36abde415 100644 --- a/bsps/aarch64/xilinx-zynqmp/include/bsp.h +++ b/bsps/aarch64/xilinx-zynqmp/include/bsp.h @@ -86,8 +86,6 @@ BSP_START_TEXT_SECTION void zynqmp_setup_mmu_and_cache(void); */ BSP_START_TEXT_SECTION void zynqmp_setup_secondary_cpu_mmu_and_cache( void ); -void zynqmp_debug_console_flush(void); - uint32_t zynqmp_clock_i2c0(void); uint32_t zynqmp_clock_i2c1(void); diff --git a/bsps/arm/xilinx-zynq/console/debug-console.c b/bsps/arm/xilinx-zynq/console/debug-console.c index d398ca7988..67fcbdf4a1 100644 --- a/bsps/arm/xilinx-zynq/console/debug-console.c +++ b/bsps/arm/xilinx-zynq/console/debug-console.c @@ -38,10 +38,20 @@ #include <rtems/sysinit.h> #include <bsp.h> +#include <bsp/bootcard.h> #include <dev/serial/zynq-uart.h> +#include <dev/serial/zynq-uart-regs.h> #include <bspopts.h> +void bsp_flush_kernel_char_output(void) +{ + rtems_termios_device_context *base = + &zynq_uart_instances[BSP_CONSOLE_MINOR].base; + + zynq_uart_reset_tx_flush(base); +} + static void zynq_debug_console_out(char c) { rtems_termios_device_context *base = diff --git a/bsps/arm/xilinx-zynq/start/bspreset.c b/bsps/arm/xilinx-zynq/start/bspreset.c index f8cc3b6999..fcdb1b8ded 100644 --- a/bsps/arm/xilinx-zynq/start/bspreset.c +++ b/bsps/arm/xilinx-zynq/start/bspreset.c @@ -33,17 +33,13 @@ * POSSIBILITY OF SUCH DAMAGE. */ -#include <bsp.h> #include <bsp/bootcard.h> -#include <dev/serial/zynq-uart.h> void bsp_reset(void) { volatile uint32_t *slcr_unlock = (volatile uint32_t *) 0xf8000008; volatile uint32_t *pss_rst_ctrl = (volatile uint32_t *) 0xf8000200; - zynq_uart_reset_tx_flush(&zynq_uart_instances[BSP_CONSOLE_MINOR]); - while (true) { *slcr_unlock = 0xdf0d; *pss_rst_ctrl = 0x1; diff --git a/bsps/arm/xilinx-zynqmp-rpu/console/console-config.c b/bsps/arm/xilinx-zynqmp-rpu/console/console-config.c index f52e008f2b..196b2dec58 100644 --- a/bsps/arm/xilinx-zynqmp-rpu/console/console-config.c +++ b/bsps/arm/xilinx-zynqmp-rpu/console/console-config.c @@ -35,6 +35,7 @@ #include <rtems/sysinit.h> #include <rtems/termiostypes.h> +#include <bsp/bootcard.h> #include <bsp/irq.h> #include <dev/serial/zynq-uart.h> @@ -79,7 +80,7 @@ rtems_status_code console_initialize( return RTEMS_SUCCESSFUL; } -void zynqmp_debug_console_flush(void) +void bsp_flush_kernel_char_output(void) { zynq_uart_reset_tx_flush(&zynqmp_uart_instances[BSP_CONSOLE_MINOR]); } diff --git a/bsps/arm/xilinx-zynqmp-rpu/include/bsp.h b/bsps/arm/xilinx-zynqmp-rpu/include/bsp.h index e386bd4b26..060147967c 100644 --- a/bsps/arm/xilinx-zynqmp-rpu/include/bsp.h +++ b/bsps/arm/xilinx-zynqmp-rpu/include/bsp.h @@ -83,8 +83,6 @@ extern "C" { */ BSP_START_TEXT_SECTION void zynqmp_setup_mpu_and_cache(void); -void zynqmp_debug_console_flush(void); - #ifdef __cplusplus } #endif /* __cplusplus */ diff --git a/bsps/arm/xilinx-zynqmp-rpu/start/bspreset.c b/bsps/arm/xilinx-zynqmp-rpu/start/bspreset.c index eecb4da838..a894f55f6e 100644 --- a/bsps/arm/xilinx-zynqmp-rpu/start/bspreset.c +++ b/bsps/arm/xilinx-zynqmp-rpu/start/bspreset.c @@ -30,13 +30,10 @@ * POSSIBILITY OF SUCH DAMAGE. */ -#include <bsp.h> #include <bsp/bootcard.h> void bsp_reset(void) { - zynqmp_debug_console_flush(); - while (true) { /* Wait */ } diff --git a/bsps/arm/xilinx-zynqmp/console/console-config.c b/bsps/arm/xilinx-zynqmp/console/console-config.c index eadd7f11a2..360d193ba2 100644 --- a/bsps/arm/xilinx-zynqmp/console/console-config.c +++ b/bsps/arm/xilinx-zynqmp/console/console-config.c @@ -81,7 +81,7 @@ rtems_status_code console_initialize( return RTEMS_SUCCESSFUL; } -void zynqmp_debug_console_flush(void) +void bsp_flush_kernel_char_output(void) { zynq_uart_reset_tx_flush(&zynqmp_uart_instances[BSP_CONSOLE_MINOR]); } diff --git a/bsps/arm/xilinx-zynqmp/include/bsp.h b/bsps/arm/xilinx-zynqmp/include/bsp.h index a08a5feee9..2785e4c2e0 100644 --- a/bsps/arm/xilinx-zynqmp/include/bsp.h +++ b/bsps/arm/xilinx-zynqmp/include/bsp.h @@ -79,8 +79,6 @@ extern "C" { */ BSP_START_TEXT_SECTION void zynqmp_setup_mmu_and_cache(void); -void zynqmp_debug_console_flush(void); - #ifdef __cplusplus } #endif /* __cplusplus */ diff --git a/bsps/arm/xilinx-zynqmp/start/bspreset.c b/bsps/arm/xilinx-zynqmp/start/bspreset.c index b43b19b05f..ed2f4da83a 100644 --- a/bsps/arm/xilinx-zynqmp/start/bspreset.c +++ b/bsps/arm/xilinx-zynqmp/start/bspreset.c @@ -30,12 +30,10 @@ * POSSIBILITY OF SUCH DAMAGE. */ -#include <bsp.h> +#include <bsp/bootcard.h> void bsp_reset(void) { - zynqmp_debug_console_flush(); - while (true) { /* Wait */ } diff --git a/bsps/include/bsp/bootcard.h b/bsps/include/bsp/bootcard.h index 5f339d65f8..dfdc3ae7e0 100644 --- a/bsps/include/bsp/bootcard.h +++ b/bsps/include/bsp/bootcard.h @@ -94,6 +94,11 @@ struct Per_CPU_Control; */ void bsp_start_on_secondary_processor(struct Per_CPU_Control *cpu_self); +/** + * @brief Flushes the kernel I/O output character device provided by the BSP. + */ +void bsp_flush_kernel_char_output(void); + /** @} */ #ifdef __cplusplus diff --git a/bsps/shared/start/bspfatal-default.c b/bsps/shared/start/bspfatal-default.c index 557a0960fa..de3f414407 100644 --- a/bsps/shared/start/bspfatal-default.c +++ b/bsps/shared/start/bspfatal-default.c @@ -184,6 +184,10 @@ void bsp_fatal_extension( printk("\n"); #endif + #if BSP_FLUSH_KERNEL_CHAR_OUTPUT + bsp_flush_kernel_char_output(); + #endif + /* * Check both conditions -- if you want to ask for reboot, then * you must have meant to reset the board. diff --git a/spec/build/bsps/bspopts.yml b/spec/build/bsps/bspopts.yml index 734292f421..b2d8ced4f6 100644 --- a/spec/build/bsps/bspopts.yml +++ b/spec/build/bsps/bspopts.yml @@ -25,6 +25,8 @@ links: uid: optincludes - role: build-dependency uid: optcflags +- role: build-dependency + uid: optflushkerncharout - role: build-dependency uid: optlinkflags - role: build-dependency diff --git a/spec/build/bsps/optflushkerncharout.yml b/spec/build/bsps/optflushkerncharout.yml new file mode 100644 index 0000000000..845184663a --- /dev/null +++ b/spec/build/bsps/optflushkerncharout.yml @@ -0,0 +1,22 @@ +SPDX-License-Identifier: CC-BY-SA-4.0 OR BSD-2-Clause +actions: +- get-integer: null +- define: null +build-type: option +copyrights: +- Copyright (C) 2024 embedded brains GmbH & Co. KG +default: +- enabled-by: + - bsps/arm/xilinx-zynq + - bsps/arm/xilinx-zynqmp + value: 1 +- enabled-by: true + value: 0 +description: | + If defined to a non-zero value, flush the kernel I/O output character device + provided by the BSP when the application exits. +enabled-by: true +format: '{}' +links: [] +name: BSP_FLUSH_KERNEL_CHAR_OUTPUT +type: build -- 2.35.3 _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel