Re: [PATCH 1/2] exec/ioport: Factor portio_list_register_flush_coalesced() out

[Richard Henderson]

On 2/7/23 13:46, Philippe Mathieu-Daudé wrote:
> We always follow the same pattern when registering
> coalesced portio:
>
>    - portio_list_init()
>    - portio_list_set_flush_coalesced()
>    - portio_list_add()
>
> Factor these 3 operations in a single helper named
> portio_list_register_flush_coalesced().
>
> Drop portio_list_set_flush_coalesced() which is now
> inlined.
>
> Signed-off-by: Philippe Mathieu-Daudé<philmd@linaro.org>
> ---
>   hw/display/qxl.c      |  7 +++----
>   hw/display/vga.c      |  5 ++---
>   include/exec/ioport.h |  5 ++++-
>   softmmu/ioport.c      | 27 ++++++++++++++++++++++-----
>   4 files changed, 31 insertions(+), 13 deletions(-)

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

r~