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

Philippe Mathieu-Daudé posted 4 patches 2 years, 8 months ago
Test checkpatch failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210819163422.2863447-1-philmd@redhat.com
Maintainers: Laurent Vivier <laurent@vivier.eu>, Michael Tokarev <mjt@tls.msk.ru>
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(-)
[PATCH v2 0/4] hw/dma: Always expect 'dma' link property to be set to simplify
Posted by Philippe Mathieu-Daudé 2 years, 8 months ago
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(-)

-- 
2.31.1


Re: [PATCH v2 0/4] hw/dma: Always expect 'dma' link property to be set to simplify
Posted by Peter Maydell 2 years, 8 months ago
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