Am 18. September 2024 00:51:32 UTC schrieb d...@treblig.org:
>From: "Dr. David Alan Gilbert" <d...@treblig.org>
>
>mch_mcfg_base has been unused since it was added by
>  6f1426ab0f ("ich9: APIs for pc guest info")
>back in 2013.

Indeed.

>
>Remove it.
>
>Signed-off-by: Dr. David Alan Gilbert <d...@treblig.org>

Reviewed-by: Bernhard Beschow <shen...@gmail.com>

I'm curious: How do you detect such unused code?

>---
> hw/pci-host/q35.c         | 10 ----------
> include/hw/pci-host/q35.h |  2 --
> 2 files changed, 12 deletions(-)
>
>diff --git a/hw/pci-host/q35.c b/hw/pci-host/q35.c
>index d5a657a02a..f3e713318e 100644
>--- a/hw/pci-host/q35.c
>+++ b/hw/pci-host/q35.c
>@@ -662,16 +662,6 @@ static void mch_realize(PCIDevice *d, Error **errp)
>                                    OBJECT(&mch->smram));
> }
> 
>-uint64_t mch_mcfg_base(void)
>-{
>-    bool ambiguous;
>-    Object *o = object_resolve_path_type("", TYPE_MCH_PCI_DEVICE, &ambiguous);
>-    if (!o) {
>-        return 0;
>-    }
>-    return MCH_HOST_BRIDGE_PCIEXBAR_DEFAULT;
>-}
>-
> static Property mch_props[] = {
>     DEFINE_PROP_UINT16("extended-tseg-mbytes", MCHPCIState, ext_tseg_mbytes,
>                        16),
>diff --git a/include/hw/pci-host/q35.h b/include/hw/pci-host/q35.h
>index 22fadfa3ed..ddafc3f2e3 100644
>--- a/include/hw/pci-host/q35.h
>+++ b/include/hw/pci-host/q35.h
>@@ -181,8 +181,6 @@ struct Q35PCIHost {
> #define MCH_PCIE_DEV                           1
> #define MCH_PCIE_FUNC                          0
> 
>-uint64_t mch_mcfg_base(void);
>-
> /*
>  * Arbitrary but unique BNF number for IOAPIC device.
>  *

Reply via email to