[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH 01/13] pseries: Remove unused callbacks from sPAPR
From: |
Thomas Huth |
Subject: |
Re: [Qemu-ppc] [PATCH 01/13] pseries: Remove unused callbacks from sPAPR VIO bus state |
Date: |
Thu, 20 Oct 2016 08:55:45 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 |
On 20.10.2016 07:11, David Gibson wrote:
> The original QOMification of the spapr VIO devices in 3954d33 "spapr:
> convert to QEMU Object Model (v2)" moved some callbacks from the
> VIOsPAPRBus structure to the VIOsPAPRDeviceClass. Except, that it
> forgot to actually remove them from the VIOsPAPRBus structure (which
> still exists, though it doesn't fulfill quite the same function as it
> did pre-QOM).
>
> This patch removes those now unused callback fields.
>
> Signed-off-by: David Gibson <address@hidden>
> Reviewed-by: Alexey Kardashevskiy <address@hidden>
> ---
> include/hw/ppc/spapr_vio.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/include/hw/ppc/spapr_vio.h b/include/hw/ppc/spapr_vio.h
> index 40d0e5f..0b025fd 100644
> --- a/include/hw/ppc/spapr_vio.h
> +++ b/include/hw/ppc/spapr_vio.h
> @@ -76,8 +76,6 @@ struct VIOsPAPRDevice {
> struct VIOsPAPRBus {
> BusState bus;
> uint32_t next_reg;
> - int (*init)(VIOsPAPRDevice *dev);
> - int (*devnode)(VIOsPAPRDevice *dev, void *fdt, int node_off);
> };
>
> extern VIOsPAPRBus *spapr_vio_bus_init(void);
>
Reviewed-by: Thomas Huth <address@hidden>
- [Qemu-ppc] [PATCH 00/13] pseries: Consolidate guest device tree construction, David Gibson, 2016/10/20
- [Qemu-ppc] [PATCH 01/13] pseries: Remove unused callbacks from sPAPR VIO bus state, David Gibson, 2016/10/20
- Re: [Qemu-ppc] [PATCH 01/13] pseries: Remove unused callbacks from sPAPR VIO bus state,
Thomas Huth <=
- [Qemu-ppc] [PATCH 03/13] pseries: Remove rtas_addr and fdt_addr fields from machinestate, David Gibson, 2016/10/20
- [Qemu-ppc] [PATCH 02/13] pseries: Split device tree construction from device tree load, David Gibson, 2016/10/20
- [Qemu-ppc] [PATCH 06/13] pseries: Consolidate RTAS loading, David Gibson, 2016/10/20
- [Qemu-ppc] [PATCH 04/13] pseries: Make spapr_create_fdt_skel() get information from machine state, David Gibson, 2016/10/20
- [Qemu-ppc] [PATCH 05/13] pseries: Move adding of fdt reserve map entries, David Gibson, 2016/10/20
- [Qemu-ppc] [PATCH 10/13] spapr: Move /event-sources construction to spapr_build_fdt(), David Gibson, 2016/10/20