mbox series

[v6,0/4] tests/qtest: Update tests using PL011 UART

Message ID 20250102105822.43532-1-philmd@linaro.org
Headers show
Series tests/qtest: Update tests using PL011 UART | expand

Message

Philippe Mathieu-Daudé Jan. 2, 2025, 10:58 a.m. UTC 
This series split a patch [*] which was previously posted
too compact in multiple trivial steps. The last patch
initializes the PL011 Control register, enabling the
UART and its transmit channel, before using it to send
the 'T' character in the boot-serial-test.

[*] https://lore.kernel.org/qemu-devel/20240719181041.49545-10-philmd@linaro.org/

Supersedes: <20240719181041.49545-10-philmd@linaro.org>

Philippe Mathieu-Daudé (4):
  tests/qtest/boot-serial-test: Improve ASM comments of PL011 tests
  tests/qtest/boot-serial-test: Reduce for() loop in PL011 tests
  tests/qtest/boot-serial-test: Reorder pair of instructions in PL011
    test
  tests/qtest/boot-serial-test: Initialize PL011 Control register

 tests/qtest/boot-serial-test.c | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

Comments

Richard Henderson Jan. 2, 2025, 3:02 p.m. UTC | #1 
On 1/2/25 02:58, Philippe Mathieu-Daudé wrote:
> This series split a patch [*] which was previously posted
> too compact in multiple trivial steps. The last patch
> initializes the PL011 Control register, enabling the
> UART and its transmit channel, before using it to send
> the 'T' character in the boot-serial-test.
> 
> [*] https://lore.kernel.org/qemu-devel/20240719181041.49545-10-philmd@linaro.org/
> 
> Supersedes: <20240719181041.49545-10-philmd@linaro.org>
> 
> Philippe Mathieu-Daudé (4):
>    tests/qtest/boot-serial-test: Improve ASM comments of PL011 tests
>    tests/qtest/boot-serial-test: Reduce for() loop in PL011 tests
>    tests/qtest/boot-serial-test: Reorder pair of instructions in PL011
>      test
>    tests/qtest/boot-serial-test: Initialize PL011 Control register
> 
>   tests/qtest/boot-serial-test.c | 23 ++++++++++++++---------
>   1 file changed, 14 insertions(+), 9 deletions(-)
> 

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~