[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes
From: |
Peter Maydell |
Subject: |
Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes |
Date: |
Wed, 18 Nov 2020 14:30:16 +0000 |
On Wed, 18 Nov 2020 at 14:24, Markus Armbruster <armbru@redhat.com> wrote:
>
> 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().
> >> #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:
> > 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.
Yes: wdt_diag288.c -> include/hw/watchdog/wdt_diag288.h ->
include/qom/object.h -> include/qemu/module.h
thanks
-- PMM
- [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, 2020/11/18
- Re: [PATCH] hw/watchdog/wdt_diag288: Remove unnecessary includes,
Peter Maydell <=
- 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