Skip to content

Commit

Permalink
display: include dependencies explicitly
Browse files Browse the repository at this point in the history
acpi-vga-stub.c pulls in vga_int.h
However that currently pulls in ui/console.h which
breaks e.g. on systems without pixman.
It's better to remove ui/console.h from vga_int.h
and directly include it where it's used.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Message-Id: <20221109222112.74519-1-mst@redhat.com>
Tested-by: Laurent Vivier <lvivier@redhat.com>
Reported-by: Miroslav Rezanina <mrezanin@redhat.com>
Reported-by: Frederic Bezies <fredbezies@gmail.com>
Reported-by: Laurent Vivier <lvivier@redhat.com>
Fixes: cfead31 ("AcpiDevAmlIf interface to build VGA device descs")
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
  • Loading branch information
mstsirkin committed Nov 10, 2022
1 parent 53a3b83 commit 28cf396
Show file tree
Hide file tree
Showing 10 changed files with 10 additions and 1 deletion.
1 change: 1 addition & 0 deletions hw/display/ati_2d.c
Expand Up @@ -12,6 +12,7 @@
#include "ati_regs.h"
#include "qemu/log.h"
#include "ui/pixel_ops.h"
#include "ui/console.h"

/*
* NOTE:
Expand Down
1 change: 1 addition & 0 deletions hw/display/cirrus_vga.c
Expand Up @@ -45,6 +45,7 @@
#include "ui/pixel_ops.h"
#include "cirrus_vga_internal.h"
#include "qom/object.h"
#include "ui/console.h"

/*
* TODO:
Expand Down
1 change: 1 addition & 0 deletions hw/display/cirrus_vga_isa.c
Expand Up @@ -31,6 +31,7 @@
#include "hw/isa/isa.h"
#include "cirrus_vga_internal.h"
#include "qom/object.h"
#include "ui/console.h"

#define TYPE_ISA_CIRRUS_VGA "isa-cirrus-vga"
OBJECT_DECLARE_SIMPLE_TYPE(ISACirrusVGAState, ISA_CIRRUS_VGA)
Expand Down
1 change: 1 addition & 0 deletions hw/display/vga-isa.c
Expand Up @@ -32,6 +32,7 @@
#include "qemu/timer.h"
#include "hw/loader.h"
#include "hw/qdev-properties.h"
#include "ui/console.h"
#include "qom/object.h"

#define TYPE_ISA_VGA "isa-vga"
Expand Down
1 change: 1 addition & 0 deletions hw/display/vga-mmio.c
Expand Up @@ -27,6 +27,7 @@
#include "hw/sysbus.h"
#include "hw/display/vga.h"
#include "hw/qdev-properties.h"
#include "ui/console.h"
#include "vga_int.h"

/*
Expand Down
1 change: 1 addition & 0 deletions hw/display/vga-pci.c
Expand Up @@ -30,6 +30,7 @@
#include "migration/vmstate.h"
#include "vga_int.h"
#include "ui/pixel_ops.h"
#include "ui/console.h"
#include "qemu/module.h"
#include "qemu/timer.h"
#include "hw/loader.h"
Expand Down
1 change: 1 addition & 0 deletions hw/display/vga.c
Expand Up @@ -31,6 +31,7 @@
#include "vga_int.h"
#include "vga_regs.h"
#include "ui/pixel_ops.h"
#include "ui/console.h"
#include "qemu/timer.h"
#include "hw/xen/xen.h"
#include "migration/vmstate.h"
Expand Down
1 change: 0 additions & 1 deletion hw/display/vga_int.h
Expand Up @@ -27,7 +27,6 @@

#include "exec/ioport.h"
#include "exec/memory.h"
#include "ui/console.h"

#include "hw/display/bochs-vbe.h"
#include "hw/acpi/acpi_aml_interface.h"
Expand Down
1 change: 1 addition & 0 deletions hw/display/vmware_vga.c
Expand Up @@ -33,6 +33,7 @@
#include "hw/qdev-properties.h"
#include "migration/vmstate.h"
#include "qom/object.h"
#include "ui/console.h"

#undef VERBOSE
#define HW_RECT_ACCEL
Expand Down
2 changes: 2 additions & 0 deletions include/qemu/typedefs.h
Expand Up @@ -132,6 +132,8 @@ typedef struct Visitor Visitor;
typedef struct VMChangeStateEntry VMChangeStateEntry;
typedef struct VMStateDescription VMStateDescription;
typedef struct DumpState DumpState;
typedef struct GraphicHwOps GraphicHwOps;
typedef struct QEMUCursor QEMUCursor;

/*
* Pointer types
Expand Down

0 comments on commit 28cf396

Please sign in to comment.