hw: Few more QOM/QDev cleanups

[PATCH 0/7] hw: Few more QOM/QDev cleanups

Philippe Mathieu-Daudé posted 7 patches 7 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20231017140150.44995-1-philmd@linaro.org

Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Raphael Norwitz <raphael.norwitz@nutanix.com>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, Gerd Hoffmann <kraxel@redhat.com>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Alistair Francis <alistair@alistair23.me>, Peter Maydell <peter.maydell@linaro.org>, Jason Wang <jasowang@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Fam Zheng <fam@euphon.net>

hw/block/vhost-user-blk.c |  4 ++--
hw/display/virtio-gpu.c   |  2 +-
hw/dma/xilinx_axidma.c    |  6 ++----
hw/dma/xlnx-zdma.c        |  7 ++-----
hw/dma/xlnx_csu_dma.c     | 13 ++++---------
hw/net/cadence_gem.c      |  7 ++-----
hw/scsi/virtio-scsi.c     |  2 +-
hw/usb/hcd-xhci-sysbus.c  |  4 ----
hw/usb/hcd-xhci.c         |  2 ++
hw/virtio/virtio-pmem.c   |  5 +----
10 files changed, 17 insertions(+), 35 deletions(-)

Expand all Fold all

[PATCH 0/7] hw: Few more QOM/QDev cleanups

Posted by Philippe Mathieu-Daudé 7 months ago

- Remove a pointless check,
- Use QOM cast macros,
- Declare QDev links statically using DEFINE_PROP_LINK()

Philippe Mathieu-Daudé (7):
  hw/virtio/virtio-pmem: Replace impossible check by assertion
  hw/block/vhost-user-blk: Use DEVICE() / VIRTIO_DEVICE() macros
  hw/display/virtio-gpu: Use VIRTIO_DEVICE() macro
  hw/scsi/virtio-scsi: Use VIRTIO_SCSI_COMMON() macro
  hw/dma: Declare link using static DEFINE_PROP_LINK() macro
  hw/net: Declare link using static DEFINE_PROP_LINK() macro
  hw/usb: Declare link using static DEFINE_PROP_LINK() macro

 hw/block/vhost-user-blk.c |  4 ++--
 hw/display/virtio-gpu.c   |  2 +-
 hw/dma/xilinx_axidma.c    |  6 ++----
 hw/dma/xlnx-zdma.c        |  7 ++-----
 hw/dma/xlnx_csu_dma.c     | 13 ++++---------
 hw/net/cadence_gem.c      |  7 ++-----
 hw/scsi/virtio-scsi.c     |  2 +-
 hw/usb/hcd-xhci-sysbus.c  |  4 ----
 hw/usb/hcd-xhci.c         |  2 ++
 hw/virtio/virtio-pmem.c   |  5 +----
 10 files changed, 17 insertions(+), 35 deletions(-)

-- 
2.41.0

Re: [PATCH 0/7] hw: Few more QOM/QDev cleanups

Posted by Michael S. Tsirkin 7 months ago

On Tue, Oct 17, 2023 at 04:01:43PM +0200, Philippe Mathieu-Daudé wrote:
> - Remove a pointless check,
> - Use QOM cast macros,
> - Declare QDev links statically using DEFINE_PROP_LINK()

virtio things
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>



> Philippe Mathieu-Daudé (7):
>   hw/virtio/virtio-pmem: Replace impossible check by assertion
>   hw/block/vhost-user-blk: Use DEVICE() / VIRTIO_DEVICE() macros
>   hw/display/virtio-gpu: Use VIRTIO_DEVICE() macro
>   hw/scsi/virtio-scsi: Use VIRTIO_SCSI_COMMON() macro
>   hw/dma: Declare link using static DEFINE_PROP_LINK() macro
>   hw/net: Declare link using static DEFINE_PROP_LINK() macro
>   hw/usb: Declare link using static DEFINE_PROP_LINK() macro
> 
>  hw/block/vhost-user-blk.c |  4 ++--
>  hw/display/virtio-gpu.c   |  2 +-
>  hw/dma/xilinx_axidma.c    |  6 ++----
>  hw/dma/xlnx-zdma.c        |  7 ++-----
>  hw/dma/xlnx_csu_dma.c     | 13 ++++---------
>  hw/net/cadence_gem.c      |  7 ++-----
>  hw/scsi/virtio-scsi.c     |  2 +-
>  hw/usb/hcd-xhci-sysbus.c  |  4 ----
>  hw/usb/hcd-xhci.c         |  2 ++
>  hw/virtio/virtio-pmem.c   |  5 +----
>  10 files changed, 17 insertions(+), 35 deletions(-)
> 
> -- 
> 2.41.0

Re: [PATCH 0/7] hw: Few more QOM/QDev cleanups

Posted by Philippe Mathieu-Daudé 7 months ago

On 17/10/23 16:01, Philippe Mathieu-Daudé wrote:

> Philippe Mathieu-Daudé (7):
>    hw/virtio/virtio-pmem: Replace impossible check by assertion
>    hw/block/vhost-user-blk: Use DEVICE() / VIRTIO_DEVICE() macros
>    hw/display/virtio-gpu: Use VIRTIO_DEVICE() macro
>    hw/scsi/virtio-scsi: Use VIRTIO_SCSI_COMMON() macro
>    hw/dma: Declare link using static DEFINE_PROP_LINK() macro
>    hw/net: Declare link using static DEFINE_PROP_LINK() macro

Patches 1-6 queued to hw-misc.