mbox series

[0/4] hw/misc/a9scu: Verify CPU count is valid and simplify a bit

Message ID 20200901144100.116742-1-f4bug@amsat.org
Headers show
Series hw/misc/a9scu: Verify CPU count is valid and simplify a bit | expand

Message

Philippe Mathieu-Daudé Sept. 1, 2020, 2:40 p.m. UTC 
Trivial patches:
- verify the A9 CPU count is in range,
- simplify using MemoryRegionOps valid/impl,
- log unimplemented registers.

Philippe Mathieu-Daudé (4):
  hw/misc/a9scu: Do not allow invalid CPU count
  hw/misc/a9scu: Simplify setting MemoryRegionOps::valid fields
  hw/misc/a9scu: Simplify setting MemoryRegionOps::impl fields
  hw/misc/a9scu: Report unimplemented accesses with qemu_log_mask(UNIMP)

 hw/misc/a9scu.c | 59 +++++++++++++++++++++++--------------------------
 1 file changed, 28 insertions(+), 31 deletions(-)

Comments

Peter Maydell Sept. 7, 2020, 3:38 p.m. UTC | #1 
On Tue, 1 Sep 2020 at 15:41, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> Trivial patches:
> - verify the A9 CPU count is in range,
> - simplify using MemoryRegionOps valid/impl,
> - log unimplemented registers.
>
> Philippe Mathieu-Daudé (4):
>   hw/misc/a9scu: Do not allow invalid CPU count
>   hw/misc/a9scu: Simplify setting MemoryRegionOps::valid fields
>   hw/misc/a9scu: Simplify setting MemoryRegionOps::impl fields
>   hw/misc/a9scu: Report unimplemented accesses with qemu_log_mask(UNIMP)
>
>  hw/misc/a9scu.c | 59 +++++++++++++++++++++++--------------------------
>  1 file changed, 28 insertions(+), 31 deletions(-)



Applied to target-arm.next, thanks.

-- PMM