[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 00/15] hw: Use QOM alias properties and few QOM/QDev clean
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2 00/15] hw: Use QOM alias properties and few QOM/QDev cleanups |
Date: |
Mon, 25 Sep 2023 13:05:35 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Philippe Mathieu-Daudé <philmd@linaro.org> writes:
> On 3/2/23 15:55, Philippe Mathieu-Daudé wrote:
>> Since v1:
>> - Addressed Markus & Zoltan review comments
>> - Introduce/use qdev_unrealize_and_unref()
>> These patches are extracted from a QOM/QDev refactor series,
>> so they are preliminary cleanups noticed while working on it:
>> - Use correct type when calling qdev_prop_set_xxx()
>> - Unify some qdev properties in MIPS models
>> - Replace intermediate properties by link properties
>> - Remove DEFINE_PROP_DMAADDR() macro which is used one time
>> - Use qdev_realize_and_unref() instead of open-coding it
>> Philippe Mathieu-Daudé (15):
>> hw/pci/pcie_sriov: Replace fprintf(error_pretty) ->
>> warn_reportf_err()
>> hw/qdev: Introduce qdev_unrealize_and_unref()
>> linux-user/syscall: Do not open-code qdev_unrealize_and_unref()
>> hw/pci/pcie_sriov: Do not open-code qdev_unrealize_and_unref()
>> hw/i386/sgx: Do not open-code qdev_realize_and_unref()
>> hw/ppc/sam460ex: Correctly set MAL properties
>> hw/arm/nrf51: Alias 'flash-size' QOM property in SoC object
>> hw/arm/fsl-imx: Alias 'phy-num' QOM property in SoC object
>> hw/usb/hcd-ohci: Include missing 'sysbus.h' header
>> hw/display/sm501: Embed OHCI QOM child in chipset
>> hw/display/sm501: Alias 'dma-offset' QOM property in chipset object
>> hw/display/sm501: Unify common QOM properties
>> hw/qdev: Remove DEFINE_PROP_DMAADDR() and 'hw/qdev-dma.h'
>> hw/mips: Declare all length properties as unsigned
>> hw/mips/itu: Pass SAAR using QOM link property
>
> Ping for review?
Needs a rebase. I reviewed some all the same. If you want more from me
before you respin, let me know.