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.
> *