[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 10/12] vt82c686: Remove unneeded includes and defines
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 10/12] vt82c686: Remove unneeded includes and defines |
Date: |
Sun, 27 Dec 2020 15:51:58 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.5.0 |
On 12/27/20 2:10 AM, BALATON Zoltan via wrote:
> These are not used or not needed.
>
> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
> ---
> hw/isa/vt82c686.c | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/hw/isa/vt82c686.c b/hw/isa/vt82c686.c
> index 789459bcae..6dff2bc67d 100644
> --- a/hw/isa/vt82c686.c
> +++ b/hw/isa/vt82c686.c
> @@ -12,22 +12,16 @@
>
> #include "qemu/osdep.h"
> #include "hw/isa/vt82c686.h"
> -#include "hw/i2c/i2c.h"
Maybe squash this one in patch 8 "Remove vt82c686b_pm_init"?
> #include "hw/pci/pci.h"
> #include "hw/qdev-properties.h"
> -#include "hw/isa/isa.h"
Used for isa_bus_new().
> #include "hw/isa/superio.h"
> -#include "hw/sysbus.h"
OK.
> #include "migration/vmstate.h"
> -#include "hw/mips/mips.h"
Indeed I can't see anything in commit edf79e66c02 ("Initial support
of vt82686b south bridge used by fulong mini pc") requiring it.
> #include "hw/isa/apm.h"
> #include "hw/acpi/acpi.h"
> #include "hw/i2c/pm_smbus.h"
> #include "qapi/error.h"
> -#include "qemu/module.h"
type_init().
> #include "qemu/timer.h"
> #include "exec/address-spaces.h"
> -#include "qom/object.h"
OK.
> #include "trace.h"
>
> typedef struct SuperIOConfig {
> @@ -137,8 +131,6 @@ static void vt82c686b_write_config(PCIDevice *d, uint32_t
> addr,
> }
> }
>
> -#define ACPI_DBG_IO_ADDR 0xb044
> -
> struct VT686PMState {
> PCIDevice dev;
> MemoryRegion io;
>
- [PATCH 09/12] vt82c686: Convert debug printf to trace points, (continued)
- [PATCH 09/12] vt82c686: Convert debug printf to trace points, BALATON Zoltan, 2020/12/26
- [PATCH 03/12] vt82c686: Remove unnecessary _DEVICE suffix from type macros, BALATON Zoltan, 2020/12/26
- [PATCH 05/12] vt82c686: Split off via-[am]c97 into separate file in hw/audio, BALATON Zoltan, 2020/12/26
- [PATCH 10/12] vt82c686: Remove unneeded includes and defines, BALATON Zoltan, 2020/12/26
- Re: [PATCH 10/12] vt82c686: Remove unneeded includes and defines,
Philippe Mathieu-Daudé <=
- [PATCH 12/12] vt82c686: Do not add floppy, BALATON Zoltan, 2020/12/26