[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes
From: |
Markus Armbruster |
Subject: |
Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes |
Date: |
Wed, 18 Nov 2020 15:22:19 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Philippe Mathieu-Daudé <philmd@redhat.com> writes:
> On 11/18/20 10:03 AM, Thomas Huth wrote:
>> Both headers, sysbus.h and module.h, are not required to compile this file.
module.h is: it defines type_init().
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>> hw/watchdog/wdt_diag288.c | 2 --
>> 1 file changed, 2 deletions(-)
>>
>> diff --git a/hw/watchdog/wdt_diag288.c b/hw/watchdog/wdt_diag288.c
>> index 71a945f0bd..4c4b6a6ab7 100644
>> --- a/hw/watchdog/wdt_diag288.c
>> +++ b/hw/watchdog/wdt_diag288.c
>> @@ -14,12 +14,10 @@
>> #include "qemu/osdep.h"
>> #include "sysemu/reset.h"
>> #include "sysemu/watchdog.h"
>> -#include "hw/sysbus.h"
>
> OK
>
>> #include "qemu/timer.h"
>> #include "hw/watchdog/wdt_diag288.h"
>> #include "migration/vmstate.h"
>> #include "qemu/log.h"
>> -#include "qemu/module.h"
>
> Cc'ing Markus because of:
>
> commit 0b8fa32f551e863bb548a11394239239270dd3dc
> Author: Markus Armbruster <armbru@redhat.com>
> Date: Thu May 23 16:35:07 2019 +0200
>
> Include qemu/module.h where needed, drop it from qemu-common.h
If it still compiles and links, it must get it via some other header.
>>
>> static WatchdogTimerModel model = {
>> .wdt_name = TYPE_WDT_DIAG288,
>>
- [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes, Thomas Huth, 2020/11/18
- Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes, Christian Borntraeger, 2020/11/18
- Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes, Philippe Mathieu-Daudé, 2020/11/18
- Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes,
Markus Armbruster <=
- Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes, Peter Maydell, 2020/11/18
- Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes, Thomas Huth, 2020/11/23
- Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes, Markus Armbruster, 2020/11/23
- Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes, Cornelia Huck, 2020/11/23
- Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes, Thomas Huth, 2020/11/23
- Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes, Cornelia Huck, 2020/11/24
Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes, Cornelia Huck, 2020/11/24