[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 02/26] hw/ppc/sam460ex: Add missing 'hw/pci/pci.h' header
From: |
Alistair Francis |
Subject: |
Re: [PATCH 02/26] hw/ppc/sam460ex: Add missing 'hw/pci/pci.h' header |
Date: |
Mon, 6 Jul 2020 09:50:09 -0700 |
On Sat, Jul 4, 2020 at 7:50 AM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> This file uses pci_create_simple() and PCI_DEVFN() which are both
> declared in "hw/pci/pci.h". This include is indirectly included
> by an USB header. As we want to reduce the USB header inclusions
> later, include the PCI header now, to avoid later:
>
> hw/ppc/sam460ex.c:397:5: error: implicit declaration of function
> ‘pci_create_simple’; did you mean ‘sysbus_create_simple’?
> [-Werror=implicit-function-declaration]
> 397 | pci_create_simple(pci_bus, PCI_DEVFN(6, 0), "sm501");
> | ^~~~~~~~~~~~~~~~~
> | sysbus_create_simple
> hw/ppc/sam460ex.c:397:5: error: nested extern declaration of
> ‘pci_create_simple’ [-Werror=nested-externs]
> hw/ppc/sam460ex.c:397:32: error: implicit declaration of function
> ‘PCI_DEVFN’ [-Werror=implicit-function-declaration]
> 397 | pci_create_simple(pci_bus, PCI_DEVFN(6, 0), "sm501");
> | ^~~~~~~~~
> hw/ppc/sam460ex.c:397:32: error: nested extern declaration of ‘PCI_DEVFN’
> [-Werror=nested-externs]
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Alistair
> ---
> hw/ppc/sam460ex.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/hw/ppc/sam460ex.c b/hw/ppc/sam460ex.c
> index 1a106a68de..fae970b142 100644
> --- a/hw/ppc/sam460ex.c
> +++ b/hw/ppc/sam460ex.c
> @@ -38,6 +38,7 @@
> #include "hw/usb/hcd-ehci.h"
> #include "hw/ppc/fdt.h"
> #include "hw/qdev-properties.h"
> +#include "hw/pci/pci.h"
>
> #include <libfdt.h>
>
> --
> 2.21.3
>
>
- [PATCH 00/26] hw/usb: Give it love, reduce 'hw/usb.h' inclusion out of hw/usb/, Philippe Mathieu-Daudé, 2020/07/04
- [PATCH 01/26] hw/arm/sbsa-ref: Remove unused 'hw/usb.h' header, Philippe Mathieu-Daudé, 2020/07/04
- [PATCH 02/26] hw/ppc/sam460ex: Add missing 'hw/pci/pci.h' header, Philippe Mathieu-Daudé, 2020/07/04
- Re: [PATCH 02/26] hw/ppc/sam460ex: Add missing 'hw/pci/pci.h' header,
Alistair Francis <=
- [PATCH 03/26] hw/usb: Remove unused VM_USB_HUB_SIZE definition, Philippe Mathieu-Daudé, 2020/07/04
- [PATCH 04/26] hw/usb: Reduce 'exec/memory.h' inclusion, Philippe Mathieu-Daudé, 2020/07/04
- [PATCH 05/26] hw/usb/desc: Add missing header, Philippe Mathieu-Daudé, 2020/07/04
- [PATCH 06/26] hw/usb/hcd-dwc2: Remove unnecessary includes, Philippe Mathieu-Daudé, 2020/07/04
- [PATCH 07/26] hw/usb/hcd-dwc2: Restrict some headers to source, Philippe Mathieu-Daudé, 2020/07/04