From: "Philippe Mathieu-Daudé" <f4bug@amsat.org>
To: "Michael S. Tsirkin" <mst@redhat.com>, Michael Tokarev <mjt@tls.msk.ru>
Cc: "Philippe Mathieu-Daudé" <f4bug@amsat.org>,
qemu-devel@nongnu.org, "Peter Maydell" <peter.maydell@linaro.org>,
"Thomas Huth" <thuth@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Eduardo Habkost" <ehabkost@redhat.com>,
qemu-trivial@nongnu.org, "Richard Henderson" <rth@twiddle.net>,
"Hervé Poussineau" <hpoussin@reactos.org>,
"Yongbok Kim" <yongbok.kim@mips.com>,
"Aurelien Jarno" <aurelien@aurel32.net>,
"open list:PReP" <qemu-ppc@nongnu.org>
Subject: [Qemu-devel] [PATCH v4 31/43] hw/timer/i8254: rename pit_init() -> i8254_pit_init()
Date: Fri, 15 Dec 2017 00:30:03 -0300 [thread overview]
Message-ID: <20171215033015.738-32-f4bug@amsat.org> (raw)
In-Reply-To: <20171215033015.738-1-f4bug@amsat.org>
and remove the old i386/pc dependency
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Hervé Poussineau <hpoussin@reactos.org>
---
include/hw/timer/i8254.h | 5 +++--
include/hw/timer/i8254_internal.h | 2 +-
hw/alpha/dp264.c | 2 +-
hw/i386/pc.c | 2 +-
hw/isa/i82378.c | 2 +-
hw/mips/mips_fulong2e.c | 2 +-
hw/mips/mips_jazz.c | 2 +-
hw/mips/mips_malta.c | 2 +-
hw/mips/mips_r4k.c | 2 +-
hw/timer/i8254.c | 1 -
hw/timer/i8254_common.c | 1 -
11 files changed, 11 insertions(+), 12 deletions(-)
diff --git a/include/hw/timer/i8254.h b/include/hw/timer/i8254.h
index 5adae9fa44..5b12eb918e 100644
--- a/include/hw/timer/i8254.h
+++ b/include/hw/timer/i8254.h
@@ -26,6 +26,7 @@
#define HW_I8254_H
#include "hw/hw.h"
+#include "hw/qdev.h"
#include "hw/isa/isa.h"
#define PIT_FREQ 1193182
@@ -48,8 +49,8 @@ typedef struct PITChannelInfo {
#define TYPE_I8254 "isa-pit"
#define TYPE_KVM_I8254 "kvm-pit"
-static inline ISADevice *pit_init(ISABus *bus, int base, int isa_irq,
- qemu_irq alt_irq)
+static inline ISADevice *i8254_pit_init(ISABus *bus, int base, int isa_irq,
+ qemu_irq alt_irq)
{
DeviceState *dev;
ISADevice *d;
diff --git a/include/hw/timer/i8254_internal.h b/include/hw/timer/i8254_internal.h
index dc09cc0467..c37a438f82 100644
--- a/include/hw/timer/i8254_internal.h
+++ b/include/hw/timer/i8254_internal.h
@@ -26,8 +26,8 @@
#define QEMU_I8254_INTERNAL_H
#include "hw/hw.h"
-#include "hw/i386/pc.h"
#include "hw/isa/isa.h"
+#include "qemu/timer.h"
typedef struct PITChannelState {
int count; /* can be 65536 */
diff --git a/hw/alpha/dp264.c b/hw/alpha/dp264.c
index babd6ea514..887a7401f1 100644
--- a/hw/alpha/dp264.c
+++ b/hw/alpha/dp264.c
@@ -80,7 +80,7 @@ static void clipper_init(MachineState *machine)
/* Since we have an SRM-compatible PALcode, use the SRM epoch. */
rtc_init(isa_bus, 1900, rtc_irq);
- pit_init(isa_bus, 0x40, 0, NULL);
+ i8254_pit_init(isa_bus, 0x40, 0, NULL);
isa_create_simple(isa_bus, "i8042");
/* VGA setup. Don't bother loading the bios. */
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index 186545d2a4..6a6041573f 100644
--- a/hw/i386/pc.c
+++ b/hw/i386/pc.c
@@ -1573,7 +1573,7 @@ void pc_basic_device_init(ISABus *isa_bus, qemu_irq *gsi,
if (kvm_pit_in_kernel()) {
pit = kvm_pit_init(isa_bus, 0x40);
} else {
- pit = pit_init(isa_bus, 0x40, pit_isa_irq, pit_alt_irq);
+ pit = i8254_pit_init(isa_bus, 0x40, pit_isa_irq, pit_alt_irq);
}
if (hpet) {
/* connect PIT to output control line of the HPET */
diff --git a/hw/isa/i82378.c b/hw/isa/i82378.c
index d20ea4c2ee..a9c15f858d 100644
--- a/hw/isa/i82378.c
+++ b/hw/isa/i82378.c
@@ -97,7 +97,7 @@ static void i82378_realize(PCIDevice *pci, Error **errp)
isa_bus_irqs(isabus, s->i8259);
/* 1 82C54 (pit) */
- isa = pit_init(isabus, 0x40, 0, NULL);
+ isa = i8254_pit_init(isabus, 0x40, 0, NULL);
/* speaker */
pcspk_init(isabus, isa);
diff --git a/hw/mips/mips_fulong2e.c b/hw/mips/mips_fulong2e.c
index 146cf0fccd..2a2a09c9de 100644
--- a/hw/mips/mips_fulong2e.c
+++ b/hw/mips/mips_fulong2e.c
@@ -359,7 +359,7 @@ static void mips_fulong2e_init(MachineState *machine)
smbus_eeprom_init(smbus, 1, eeprom_spd, sizeof(eeprom_spd));
/* init other devices */
- pit = pit_init(isa_bus, 0x40, 0, NULL);
+ pit = i8254_pit_init(isa_bus, 0x40, 0, NULL);
DMA_init(isa_bus, 0);
/* Super I/O */
diff --git a/hw/mips/mips_jazz.c b/hw/mips/mips_jazz.c
index fe4f17389f..995419344d 100644
--- a/hw/mips/mips_jazz.c
+++ b/hw/mips/mips_jazz.c
@@ -218,7 +218,7 @@ static void mips_jazz_init(MachineState *machine,
i8259 = i8259_init(isa_bus, env->irq[4]);
isa_bus_irqs(isa_bus, i8259);
DMA_init(isa_bus, 0);
- pit = pit_init(isa_bus, 0x40, 0, NULL);
+ pit = i8254_pit_init(isa_bus, 0x40, 0, NULL);
pcspk_init(isa_bus, pit);
/* Video card */
diff --git a/hw/mips/mips_malta.c b/hw/mips/mips_malta.c
index ec6af4a277..88b4733743 100644
--- a/hw/mips/mips_malta.c
+++ b/hw/mips/mips_malta.c
@@ -1208,7 +1208,7 @@ void mips_malta_init(MachineState *machine)
isa_get_irq(NULL, 9), NULL, 0, NULL);
smbus_eeprom_init(smbus, 8, smbus_eeprom_buf, smbus_eeprom_size);
g_free(smbus_eeprom_buf);
- pit = pit_init(isa_bus, 0x40, 0, NULL);
+ pit = i8254_pit_init(isa_bus, 0x40, 0, NULL);
DMA_init(isa_bus, 0);
/* Super I/O */
diff --git a/hw/mips/mips_r4k.c b/hw/mips/mips_r4k.c
index 3bbb1827e1..58d7bac18b 100644
--- a/hw/mips/mips_r4k.c
+++ b/hw/mips/mips_r4k.c
@@ -272,7 +272,7 @@ void mips_r4k_init(MachineState *machine)
rtc_init(isa_bus, 2000, NULL);
- pit = pit_init(isa_bus, 0x40, 0, NULL);
+ pit = i8254_pit_init(isa_bus, 0x40, 0, NULL);
serial_hds_isa_init(isa_bus, 0, MAX_SERIAL_PORTS);
diff --git a/hw/timer/i8254.c b/hw/timer/i8254.c
index 5e61ad50a8..dbc4a0baec 100644
--- a/hw/timer/i8254.c
+++ b/hw/timer/i8254.c
@@ -23,7 +23,6 @@
*/
#include "qemu/osdep.h"
#include "hw/hw.h"
-#include "hw/i386/pc.h"
#include "hw/isa/isa.h"
#include "qemu/timer.h"
#include "hw/timer/i8254.h"
diff --git a/hw/timer/i8254_common.c b/hw/timer/i8254_common.c
index b623c96198..6190b6fc5d 100644
--- a/hw/timer/i8254_common.c
+++ b/hw/timer/i8254_common.c
@@ -24,7 +24,6 @@
*/
#include "qemu/osdep.h"
#include "hw/hw.h"
-#include "hw/i386/pc.h"
#include "hw/isa/isa.h"
#include "qemu/timer.h"
#include "hw/timer/i8254.h"
--
2.15.1
next prev parent reply other threads:[~2017-12-15 3:32 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-15 3:29 [Qemu-devel] [PATCH v4 00/43] remove i386/pc dependency from non-PC world (part 1) Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 01/43] MAINTAINERS: add entries for i2c/ppc4xx Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 02/43] MAINTAINERS: add an entry for the i82374 (southbridge) Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 03/43] MAINTAINERS: add an entry for the i8257 (DMA controller) Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 04/43] MAINTAINERS: add an entry for the i82378 (superio) Philippe Mathieu-Daudé
2017-12-15 10:17 ` Thomas Huth
2017-12-15 10:23 ` Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 05/43] MAINTAINERS: add an entry for watchdog/wdt_ib700 Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 06/43] MAINTAINERS: add an entry for input/pckbd.c Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 07/43] MAINTAINERS: add entries for timer/m48t59 Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 08/43] MAINTAINERS: add missing entry for include/hw/net/ Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 09/43] hw: use "qemu/osdep.h" as first #include in source files Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 10/43] hw: remove "qemu/osdep.h" from header files Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 11/43] block: remove "qemu/osdep.h" from header file Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 12/43] misc: remove headers implicitly included Philippe Mathieu-Daudé
2017-12-15 7:40 ` Ben Warren
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 13/43] misc: remove duplicated includes Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 14/43] ppc: " Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 15/43] i386/hax: " Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 16/43] i386/hax: remove duplicated include Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 17/43] nios2: remove duplicated includes (in code commented out) Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 18/43] misc: avoid "include/" in include path Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 19/43] numa: remove unused #include Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 20/43] hw/input/vmmouse: " Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 21/43] hw/misc/pvpanic: " Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 22/43] hw/ssi/aspeed_smc: " Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 23/43] amd_iommu: avoid needless includes in header file Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 24/43] misc: remove old i386 dependency Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 25/43] hw/ide: " Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 26/43] hw/ipmi: " Philippe Mathieu-Daudé
2017-12-15 3:29 ` [Qemu-devel] [PATCH v4 27/43] hw/i2c: " Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 28/43] hw/tpm: " Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 29/43] hw/virtio-balloon: " Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 30/43] hw/unicore32: restrict hw addr defines to source file Philippe Mathieu-Daudé
2017-12-15 3:30 ` Philippe Mathieu-Daudé [this message]
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 32/43] hw/timer/mc146818: rename rtc_init() -> mc146818_rtc_init() Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 33/43] hw/acpi: ACPI_PM_* defines are not restricted to i386 arch Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 34/43] hw/acpi/ich9: extract ACPI_PM_PROP_TCO_ENABLED from i386/pc Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 35/43] hw/display/vga: "vga.h" only contains registers defs, rename it "vga_regs.h" Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 36/43] hw/display/vga: "vga_int.h" requires "ui/console.h" Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 37/43] hw/display/vga: extract public API from i386/pc to "hw/display/vga.h" Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 38/43] hw/net/ne2000: extract ne2k-isa code from i386/pc to ne2000-isa.c Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 39/43] hw/misc/pvpanic: extract public API from i386/pc to "hw/misc/pvpanic.h" Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 40/43] i386/pc: move vmport.c to hw/i386/ Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 41/43] i386/pc: move vmmouse.c " Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 42/43] hw/i386/vmport: replace fprintf() by trace events Philippe Mathieu-Daudé
2017-12-15 3:37 ` Philippe Mathieu-Daudé
2017-12-15 3:30 ` [Qemu-devel] [PATCH v4 43/43] misc: drop old i386 dependency Philippe Mathieu-Daudé
2017-12-15 4:18 ` [Qemu-devel] [PATCH v4 00/43] remove i386/pc dependency from non-PC world (part 1) no-reply
2017-12-15 4:27 ` no-reply
2017-12-15 13:47 ` Philippe Mathieu-Daudé
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171215033015.738-32-f4bug@amsat.org \
--to=f4bug@amsat.org \
--cc=aurelien@aurel32.net \
--cc=ehabkost@redhat.com \
--cc=hpoussin@reactos.org \
--cc=mjt@tls.msk.ru \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=qemu-trivial@nongnu.org \
--cc=rth@twiddle.net \
--cc=thuth@redhat.com \
--cc=yongbok.kim@mips.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).