Re: [PATCH v1 0/4] semihosting read console support

[Keith Packard]

Alex Bennée <address@hidden> writes:

> Hi,
>
> This series introduces a new "blocking" console read function for
> semihosting and plumbs it in to the ARM semihosting code. The main bit
> of work is Keith's patch (with a few tweaks by me). The other
> preparatory patches make sure the PC is updated after semihosting
> succeeds and a little bit of clean-up.
>
> The following patches need review
>    01 - target arm remove unused EXCP_SEMIHOST leg
>    02 - target arm only update pc after semihosting compl
>    04 - tests tcg add a dumb as bricks semihosting consol

I've reviewed all four of these patches to the best of my (limited)
ability. I've also tested this with picolibc's semihosting support for
cortex-m3

Reviewed-by: Keith Packard <address@hidden>
Tested-by: Keith Packard <address@hidden>

Thanks much for the rework, definitely beyond my understanding of QEMU
internals.

-- 
-keith

signature.asc
Description: PGP signature