diff mbox series

[02/14] hw/intc/i8259: Fix Kconfig dependency on ISA bus

Message ID 20191231183216.6781-3-philmd@redhat.com
State New
Headers show
Series hw: Fix various --without-default-devices issues | expand

Commit Message

Philippe Mathieu-Daudé Dec. 31, 2019, 6:32 p.m. UTC
The 8259 Interrupt Controller sits on a ISA bus.
Add the missing dependency to fix:

    LINK    x86_64-softmmu/qemu-system-x86_64
  /usr/bin/ld: ../hw/intc/i8259_common.o: in function `pic_common_realize':
  hw/intc/i8259_common.c:84: undefined reference to `isa_register_ioport'
  /usr/bin/ld: hw/intc/i8259_common.c:86: undefined reference to `isa_register_ioport'
  /usr/bin/ld: ../hw/intc/i8259_common.o: in function `i8259_init_chip':
  hw/intc/i8259_common.c:97: undefined reference to `isa_create'
  collect2: error: ld returned 1 exit status
  make[1]: *** [Makefile:206: qemu-system-x86_64] Error 1

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
Cc: "Michael S. Tsirkin" <mst@redhat.com> (supporter:PC Chipset)
---
 hw/intc/Kconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Thomas Huth Jan. 7, 2020, 12:28 p.m. UTC | #1
On 31/12/2019 19.32, Philippe Mathieu-Daudé wrote:
> The 8259 Interrupt Controller sits on a ISA bus.
> Add the missing dependency to fix:
> 
>     LINK    x86_64-softmmu/qemu-system-x86_64
>   /usr/bin/ld: ../hw/intc/i8259_common.o: in function `pic_common_realize':
>   hw/intc/i8259_common.c:84: undefined reference to `isa_register_ioport'
>   /usr/bin/ld: hw/intc/i8259_common.c:86: undefined reference to `isa_register_ioport'
>   /usr/bin/ld: ../hw/intc/i8259_common.o: in function `i8259_init_chip':
>   hw/intc/i8259_common.c:97: undefined reference to `isa_create'
>   collect2: error: ld returned 1 exit status
>   make[1]: *** [Makefile:206: qemu-system-x86_64] Error 1
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> Cc: "Michael S. Tsirkin" <mst@redhat.com> (supporter:PC Chipset)
> ---
>  hw/intc/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/hw/intc/Kconfig b/hw/intc/Kconfig
> index 10a680b53a..a189d6fedd 100644
> --- a/hw/intc/Kconfig
> +++ b/hw/intc/Kconfig
> @@ -3,6 +3,7 @@ config HEATHROW_PIC
>  
>  config I8259
>      bool
> +    select ISA_BUS
>  
>  config PL190
>      bool
> 

The code uses "parent = TYPE_ISA_DEVICE", so this seems to be right.

Reviewed-by: Thomas Huth <thuth@redhat.com>
diff mbox series

Patch

diff --git a/hw/intc/Kconfig b/hw/intc/Kconfig
index 10a680b53a..a189d6fedd 100644
--- a/hw/intc/Kconfig
+++ b/hw/intc/Kconfig
@@ -3,6 +3,7 @@  config HEATHROW_PIC
 
 config I8259
     bool
+    select ISA_BUS
 
 config PL190
     bool