Re: [PATCH v2] bsps: Add BSP_FLUSH_KERNEL_CHAR_OUTPUT BSP option

2024-03-21 Thread Chris Johns
On 21/3/2024 5:39 pm, Sebastian Huber wrote:> On 21.03.24 00:28, Chris Johns wrote: >> On 21/3/2024 2:11 am, Sebastian Huber wrote: >>> On 19.03.24 18:44, Chris Johns wrote: On 20/3/2024 2:03 am, Sebastian Huber wrote: > On 19.03.24 14:50, Kinsey Moore wrote: >> The xilinx-zynqmp-rpu

Re: [PATCH v2] bsps: Add BSP_FLUSH_KERNEL_CHAR_OUTPUT BSP option

2024-03-21 Thread Sebastian Huber
On 21.03.24 00:28, Chris Johns wrote: On 21/3/2024 2:11 am, Sebastian Huber wrote: On 19.03.24 18:44, Chris Johns wrote: On 20/3/2024 2:03 am, Sebastian Huber wrote: On 19.03.24 14:50, Kinsey Moore wrote: The xilinx-zynqmp-rpu bsp_reset() is modified, but not included in the spec file for

Re: [PATCH v2] bsps: Add BSP_FLUSH_KERNEL_CHAR_OUTPUT BSP option

2024-03-20 Thread Chris Johns
On 21/3/2024 2:11 am, Sebastian Huber wrote: > On 19.03.24 18:44, Chris Johns wrote: >> On 20/3/2024 2:03 am, Sebastian Huber wrote: >>> On 19.03.24 14:50, Kinsey Moore wrote: The xilinx-zynqmp-rpu bsp_reset() is modified, but not included in the spec file for the new option. Its family

Re: [PATCH v2] bsps: Add BSP_FLUSH_KERNEL_CHAR_OUTPUT BSP option

2024-03-20 Thread Sebastian Huber
On 19.03.24 18:44, Chris Johns wrote: On 20/3/2024 2:03 am, Sebastian Huber wrote: On 19.03.24 14:50, Kinsey Moore wrote: The xilinx-zynqmp-rpu bsp_reset() is modified, but not included in the spec file for the new option. Its family differs from the arm/xilinx-zynqmp BSP family with a -rpu

Re: [PATCH v2] bsps: Add BSP_FLUSH_KERNEL_CHAR_OUTPUT BSP option

2024-03-19 Thread Chris Johns
On 19/3/2024 7:39 pm, Sebastian Huber wrote: > Make the kernel I/O output character device flushing configurable. The > bsp_reset() function should reset the unit and do nothing else. This changes existing behaviour. RTEMS is poor at cleanly handling the console output on reset. Working with the

Re: [PATCH v2] bsps: Add BSP_FLUSH_KERNEL_CHAR_OUTPUT BSP option

2024-03-19 Thread Chris Johns
On 20/3/2024 2:03 am, Sebastian Huber wrote: > On 19.03.24 14:50, Kinsey Moore wrote: >> The xilinx-zynqmp-rpu bsp_reset() is modified, but not included in the spec >> file for the new option. Its family differs from the arm/xilinx-zynqmp BSP >> family with a -rpu suffix. > > Yes, but this BSP is

Re: [PATCH v2] bsps: Add BSP_FLUSH_KERNEL_CHAR_OUTPUT BSP option

2024-03-19 Thread Sebastian Huber
On 19.03.24 14:50, Kinsey Moore wrote: The xilinx-zynqmp-rpu bsp_reset() is modified, but not included in the spec file for the new option. Its family differs from the arm/xilinx-zynqmp BSP family with a -rpu suffix. Yes, but this BSP is quite new. I would prefer to let it not flush anything

Re: [PATCH v2] bsps: Add BSP_FLUSH_KERNEL_CHAR_OUTPUT BSP option

2024-03-19 Thread Kinsey Moore
The xilinx-zynqmp-rpu bsp_reset() is modified, but not included in the spec file for the new option. Its family differs from the arm/xilinx-zynqmp BSP family with a -rpu suffix. I'd be fine with this being enabled for the AArch64 BSPs as well, but I imagine that's better as a separate patch.

[PATCH v2] bsps: Add BSP_FLUSH_KERNEL_CHAR_OUTPUT BSP option

2024-03-19 Thread Sebastian Huber
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