mbox series

[v2,0/4] hw/dma: Always expect 'dma' link property to be set to simplify

Message ID 20210819163422.2863447-1-philmd@redhat.com (mailing list archive)
Headers show
Series hw/dma: Always expect 'dma' link property to be set to simplify | expand

Message

Philippe Mathieu-Daudé Aug. 19, 2021, 4:34 p.m. UTC 
Fix a qdev realization ordering, then simplify a pair of DMA
devices by always passing a MemoryRegion property to the device.
Doing so we can move the AddressSpace field to the device struct,
removing need for heap allocation.

Philippe Mathieu-Daudé (4):
  hw/arm/xlnx-zynqmp: Realize qspi controller *after* qspi_dma
  hw/dma/xlnx_csu_dma: Run trivial checks early in realize()
  hw/dma/xlnx_csu_dma: Always expect 'dma' link property to be set
  hw/dma/xlnx-zdma Always expect 'dma' link property to be set

 include/hw/dma/xlnx-zdma.h    |  2 +-
 include/hw/dma/xlnx_csu_dma.h |  2 +-
 hw/arm/xlnx-versal.c          |  2 ++
 hw/arm/xlnx-zynqmp.c          | 54 +++++++++++++++++++++--------------
 hw/dma/xlnx-zdma.c            | 24 ++++++++--------
 hw/dma/xlnx_csu_dma.c         | 31 ++++++++++----------
 6 files changed, 63 insertions(+), 52 deletions(-)

Comments

Peter Maydell Aug. 26, 2021, 1:19 p.m. UTC | #1 
On Thu, 19 Aug 2021 at 17:34, Philippe Mathieu-Daudé <philmd@redhat.com> wrote:
>
> Fix a qdev realization ordering, then simplify a pair of DMA
> devices by always passing a MemoryRegion property to the device.
> Doing so we can move the AddressSpace field to the device struct,
> removing need for heap allocation.



Applied to target-arm.next, thanks.

-- PMM