[Qemu-devel] [v2 PATCH] hw/arm/bcm2835_peripherals: add bcm283x sp804-alike timer

Mark posted 1 patch 1 week ago
Test docker-mingw@fedora failed
Test asan passed
Test checkpatch passed
Test docker-clang@ubuntu failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190209144918.GA14036@nyan
hw/arm/bcm2835_peripherals.c         |  17 ++
hw/timer/Makefile.objs               |   2 +
hw/timer/bcm283x_timer.c             | 299 +++++++++++++++++++++++++++
include/hw/arm/bcm2835_peripherals.h |   2 +
include/hw/timer/bcm283x_timer.h     |  38 ++++
5 files changed, 358 insertions(+)
create mode 100644 hw/timer/bcm283x_timer.c
create mode 100644 include/hw/timer/bcm283x_timer.h

[Qemu-devel] [v2 PATCH] hw/arm/bcm2835_peripherals: add bcm283x sp804-alike timer

Posted by Mark 1 week ago
Signed-off-by: Mark <alnyan@airmail.cc>
---
 hw/arm/bcm2835_peripherals.c         |  17 ++
 hw/timer/Makefile.objs               |   2 +
 hw/timer/bcm283x_timer.c             | 299 +++++++++++++++++++++++++++
 include/hw/arm/bcm2835_peripherals.h |   2 +
 include/hw/timer/bcm283x_timer.h     |  38 ++++
 5 files changed, 358 insertions(+)
 create mode 100644 hw/timer/bcm283x_timer.c
 create mode 100644 include/hw/timer/bcm283x_timer.h

diff --git a/hw/arm/bcm2835_peripherals.c b/hw/arm/bcm2835_peripherals.c
index 6be7660e8c..fccdb9509b 100644
--- a/hw/arm/bcm2835_peripherals.c
+++ b/hw/arm/bcm2835_peripherals.c
@@ -117,6 +117,13 @@ static void bcm2835_peripherals_init(Object *obj)
                                    OBJECT(&s->sdhci.sdbus), &error_abort);
     object_property_add_const_link(OBJECT(&s->gpio), "sdbus-sdhost",
                                    OBJECT(&s->sdhost.sdbus), &error_abort);
+
+    /* SP804-alike ARM Timer */
+    object_initialize(&s->bcm283xsp804, sizeof(s->bcm283xsp804),
+            TYPE_BCM283xSP804);
+    object_property_add_child(obj, "bcm283xsp804", OBJECT(&s->bcm283xsp804),
+            NULL);
+    qdev_set_parent_bus(DEVICE(&s->bcm283xsp804), sysbus_get_default());
 }

 static void bcm2835_peripherals_realize(DeviceState *dev, Error **errp)
@@ -334,6 +341,16 @@ static void bcm2835_peripherals_realize(DeviceState *dev, Error **errp)
         error_propagate(errp, err);
         return;
     }
+
+    /* SP804-alike ARM Timer */
+    object_property_set_bool(OBJECT(&s->bcm283xsp804), true, "realized", &err);
+    if (err) {
+        error_propagate(errp, err);
+        return;
+    }
+
+    memory_region_add_subregion(&s->peri_mr, ARMCTRL_TIMER0_1_OFFSET,
+                sysbus_mmio_get_region(SYS_BUS_DEVICE(&s->bcm283xsp804), 0));
 }

 static void bcm2835_peripherals_class_init(ObjectClass *oc, void *data)
diff --git a/hw/timer/Makefile.objs b/hw/timer/Makefile.objs
index 0e9a4530f8..09a3706701 100644
--- a/hw/timer/Makefile.objs
+++ b/hw/timer/Makefile.objs
@@ -47,3 +47,5 @@ common-obj-$(CONFIG_SUN4V_RTC) += sun4v-rtc.o
 common-obj-$(CONFIG_CMSDK_APB_TIMER) += cmsdk-apb-timer.o
 common-obj-$(CONFIG_CMSDK_APB_DUALTIMER) += cmsdk-apb-dualtimer.o
 common-obj-$(CONFIG_MSF2) += mss-timer.o
+
+common-obj-$(CONFIG_RASPI) += bcm283x_timer.o
diff --git a/hw/timer/bcm283x_timer.c b/hw/timer/bcm283x_timer.c
new file mode 100644
index 0000000000..55e8a68807
--- /dev/null
+++ b/hw/timer/bcm283x_timer.c
@@ -0,0 +1,299 @@
+#include "qemu/osdep.h"
+#include "hw/sysbus.h"
+#include "qemu/timer.h"
+#include "qemu-common.h"
+#include "hw/qdev.h"
+#include "hw/ptimer.h"
+#include "hw/timer/bcm283x_timer.h"
+#include "qemu/main-loop.h"
+#include "qemu/log.h"
+
+/* TODO: implement free-running timer as per BCM283x peripheral specification */
+
+#define TIMER_CTRL_32BIT    (1 << 1)
+#define TIMER_CTRL_DIV1     (0 << 2)
+#define TIMER_CTRL_DIV16    (1 << 2)
+#define TIMER_CTRL_DIV256   (2 << 2)
+#define TIMER_CTRL_IE       (1 << 5)
+#define TIMER_CTRL_ENABLE   (1 << 7)
+
+struct bcm283x_timer_state {
+    ptimer_state *timer;
+    uint32_t control;
+    uint32_t limit;
+    int freq;
+    int int_level;
+    qemu_irq irq;
+    /* Not implemented */
+    int prev_div;
+    /* Not implemented */
+    int free_run_cnt;
+};
+
+static void bcm283x_timer_update(bcm283x_timer_state *s)
+{
+    if (s->int_level && (s->control & TIMER_CTRL_IE)) {
+        qemu_irq_raise(s->irq);
+    } else {
+        qemu_irq_lower(s->irq);
+    }
+}
+
+static uint32_t bcm283x_timer_read(void *opaque, hwaddr offset)
+{
+    bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
+
+    switch (offset >> 2) {
+    case 0: /* Load register */
+    case 6: /* Reload register */
+        return s->limit;
+    case 1: /* Value register */
+        return ptimer_get_count(s->timer);
+    case 2: /* Control register */
+        return s->control;
+    case 3: /* IRQ clear/ACK register */
+        /*
+         * The register is write-only,
+         * but returns reverse "ARMT" string bytes
+         */
+        return 0x544D5241;
+    case 4: /* RAW IRQ register */
+        return s->int_level;
+    case 5: /* Masked IRQ register */
+        if ((s->control & TIMER_CTRL_IE) == 0) {
+            return 0;
+        }
+        return s->int_level;
+    default:
+        qemu_log_mask(LOG_GUEST_ERROR,
+                      "%s: Bad offset %x\n", __func__, (int) offset);
+        return 0;
+    }
+}
+
+static void bcm283x_timer_recalibrate(bcm283x_timer_state *s, int reload)
+{
+    uint32_t limit;
+
+    /* Consider timer periodic */
+    limit = s->limit;
+
+    ptimer_set_limit(s->timer, limit, reload);
+}
+
+static void bcm283x_timer_write(void *opaque, hwaddr offset, uint32_t value)
+{
+    bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
+    int freq;
+
+    switch (offset >> 2) {
+    case 0: /* Load register */
+        s->limit = value;
+        bcm283x_timer_recalibrate(s, 1);
+        break;
+    case 1: /* Value register */
+        /* Read only */
+        break;
+    case 2: /* Control register */
+        if (s->control & TIMER_CTRL_ENABLE) {
+            ptimer_stop(s->timer);
+        }
+
+        s->control = value;
+        freq = s->freq;
+
+        /* Set pre-scaler */
+        switch ((value >> 2) & 3) {
+        case 1: /* 16 */
+            freq >>= 4;
+            break;
+        case 2: /* 256 */
+            freq >>= 8;
+            break;
+        }
+
+        bcm283x_timer_recalibrate(s, s->control & TIMER_CTRL_ENABLE);
+        ptimer_set_freq(s->timer, freq);
+        if (s->control & TIMER_CTRL_ENABLE) {
+            ptimer_run(s->timer, 0);
+        }
+        break;
+    case 3: /* IRQ clear/ACK register */
+        s->int_level = 0;
+        break;
+    case 6: /* Reload register */
+        s->limit = value;
+        bcm283x_timer_recalibrate(s, 0);
+        break;
+    default:
+        qemu_log_mask(LOG_GUEST_ERROR,
+                      "%s: Bad offset %x\n", __func__, (int) offset);
+        break;
+    }
+
+    bcm283x_timer_update(s);
+}
+
+static void bcm283x_timer_tick(void *opaque)
+{
+    bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
+    s->int_level = 1;
+    bcm283x_timer_update(s);
+}
+
+static const VMStateDescription vmstate_bcm283x_timer = {
+    .name = "bcm283x_timer",
+    .version_id = 1,
+    .minimum_version_id = 1,
+    .fields = (VMStateField[]) {
+        VMSTATE_UINT32(control, bcm283x_timer_state),
+        VMSTATE_UINT32(limit, bcm283x_timer_state),
+        VMSTATE_INT32(int_level, bcm283x_timer_state),
+        VMSTATE_PTIMER(timer, bcm283x_timer_state),
+        VMSTATE_END_OF_LIST()
+    }
+};
+
+static bcm283x_timer_state *bcm283x_timer_init(uint32_t freq)
+{
+    bcm283x_timer_state *s;
+    QEMUBH *bh;
+
+    s = g_new0(bcm283x_timer_state, 1);
+    s->freq = freq;
+    s->control = TIMER_CTRL_IE;
+
+    bh = qemu_bh_new(bcm283x_timer_tick, s);
+    s->timer = ptimer_init(bh, PTIMER_POLICY_DEFAULT);
+    vmstate_register(NULL, -1, &vmstate_bcm283x_timer, s);
+
+    return s;
+}
+
+/* BCM283x's implementation of SP804 ARM timer */
+
+/*
+ * XXX: BCM's datasheet does not seem to
+ * provide these values and they may differ
+ */
+static const uint8_t bcm283xsp803_ids[] = {
+    /* Timer ID */
+    0x04, 0x18, 0x14, 0x00,
+    /* PrimeCell ID */
+    0x0D, 0xF0, 0x05, 0xB1
+};
+
+static void bcm283xsp804_set_irq(void *opaque, int irq, int level)
+{
+    BCM283xSP804State *s = (BCM283xSP804State *) opaque;
+
+    s->level = level;
+    qemu_set_irq(s->irq, s->level);
+}
+
+static uint64_t bcm283xsp804_read(void *opaque, hwaddr offset, unsigned size)
+{
+    BCM283xSP804State *s = (BCM283xSP804State *) opaque;
+
+    if (offset < 0x20) {
+        return bcm283x_timer_read(s->timer, offset);
+    }
+    /* No second timer (0x20 < offset < 0x40) */
+
+    if (offset >= 0xFE0 && offset <= 0xFFC) {
+        return bcm283xsp803_ids[(offset - 0xFE0) >> 2];
+    }
+
+    switch (offset) {
+    /* Unimplemented: integration test control registers */
+    case 0xF00:
+    case 0xF04:
+        qemu_log_mask(LOG_UNIMP,
+                      "%s: integration test registers unimplemented\n",
+                      __func__);
+        return 0;
+    }
+
+    qemu_log_mask(LOG_GUEST_ERROR,
+                  "%s: Bad offset %x\n", __func__, (int) offset);
+    return 0;
+}
+
+static void bcm283xsp803_write(void *opaque, hwaddr offset, uint64_t value,
+        unsigned size)
+{
+    BCM283xSP804State *s = (BCM283xSP804State *) opaque;
+
+    if (offset < 0x20) {
+        bcm283x_timer_write(s->timer, offset, value);
+    }
+    /* No second timer (0x20 < offset < 0x40) */
+
+    qemu_log_mask(LOG_GUEST_ERROR,
+                  "%s: Bad offset %x\n", __func__, (int) offset);
+}
+
+static const MemoryRegionOps bcm283xsp804_ops = {
+    .read = bcm283xsp804_read,
+    .write = bcm283xsp803_write,
+    .endianness = DEVICE_NATIVE_ENDIAN
+};
+
+static const VMStateDescription vmstate_bcm283xsp804 = {
+    .name = "bcm283xsp804",
+    .version_id = 1,
+    .minimum_version_id = 1,
+    .fields = (VMStateField[]) {
+        VMSTATE_INT32(level, BCM283xSP804State),
+        VMSTATE_END_OF_LIST()
+    }
+};
+
+static void bcm283xsp804_init(Object *obj)
+{
+    BCM283xSP804State *s = BCM283xSP804(obj);
+    SysBusDevice *sbd = SYS_BUS_DEVICE(obj);
+
+    sysbus_init_irq(sbd, &s->irq);
+    memory_region_init_io(&s->iomem, obj, &bcm283xsp804_ops, s,
+            "bcm283xsp804", 0x1000);
+
+    sysbus_init_mmio(sbd, &s->iomem);
+}
+
+static void bcm283xsp804_realize(DeviceState *dev, Error **errp)
+{
+    BCM283xSP804State *s = BCM283xSP804(dev);
+
+    s->timer = bcm283x_timer_init(s->freq0);
+    s->timer->irq = qemu_allocate_irq(bcm283xsp804_set_irq, s, 0);
+}
+
+static Property bcm283xsp804_properties[] = {
+    DEFINE_PROP_UINT32("freq0", BCM283xSP804State, freq0, 1000000),
+    DEFINE_PROP_END_OF_LIST()
+};
+
+static void bcm283xsp804_class_init(ObjectClass *klass, void *data)
+{
+    DeviceClass *k = DEVICE_CLASS(klass);
+
+    k->realize = bcm283xsp804_realize;
+    k->props = bcm283xsp804_properties;
+    k->vmsd = &vmstate_bcm283xsp804;
+}
+
+static const TypeInfo bcm283xsp804_info = {
+    .name           = TYPE_BCM283xSP804,
+    .parent         = TYPE_SYS_BUS_DEVICE,
+    .instance_size  = sizeof(BCM283xSP804State),
+    .instance_init  = bcm283xsp804_init,
+    .class_init     = bcm283xsp804_class_init
+};
+
+static void bcm283x_timer_register_types(void)
+{
+    type_register_static(&bcm283xsp804_info);
+}
+
+type_init(bcm283x_timer_register_types)
diff --git a/include/hw/arm/bcm2835_peripherals.h b/include/hw/arm/bcm2835_peripherals.h
index f5b193f670..d13aad43cb 100644
--- a/include/hw/arm/bcm2835_peripherals.h
+++ b/include/hw/arm/bcm2835_peripherals.h
@@ -23,6 +23,7 @@
 #include "hw/sd/sdhci.h"
 #include "hw/sd/bcm2835_sdhost.h"
 #include "hw/gpio/bcm2835_gpio.h"
+#include "hw/timer/bcm283x_timer.h"

 #define TYPE_BCM2835_PERIPHERALS "bcm2835-peripherals"
 #define BCM2835_PERIPHERALS(obj) \
@@ -48,6 +49,7 @@ typedef struct BCM2835PeripheralState {
     SDHCIState sdhci;
     BCM2835SDHostState sdhost;
     BCM2835GpioState gpio;
+    BCM283xSP804State bcm283xsp804;
 } BCM2835PeripheralState;

 #endif /* BCM2835_PERIPHERALS_H */
diff --git a/include/hw/timer/bcm283x_timer.h b/include/hw/timer/bcm283x_timer.h
new file mode 100644
index 0000000000..f5e8dcddfc
--- /dev/null
+++ b/include/hw/timer/bcm283x_timer.h
@@ -0,0 +1,38 @@
+/*
+ * Broadcom BCM283x ARM timer variant based on ARM SP804
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; either version
+ * 2 of the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, see <http://www.gnu.org/licenses/>.
+ */
+#ifndef HW_TIMER_BCM2835_TIMER_H
+#define HW_TIMER_BCM2835_TIMER_H
+
+#include "hw/sysbus.h"
+
+#define TYPE_BCM283xSP804 "bcm283xsp804"
+#define BCM283xSP804(obj)
+    OBJECT_CHECK(BCM283xSP804State, (obj), TYPE_BCM283xSP804)
+
+typedef struct bcm283x_timer_state bcm283x_timer_state;
+
+typedef struct {
+    SysBusDevice parent_obj;
+
+    MemoryRegion iomem;
+    bcm283x_timer_state *timer;
+    uint32_t freq0;
+    int level;
+    qemu_irq irq;
+} BCM283xSP804State;
+
+#endif
--
2.20.1


Re: [Qemu-devel] [v2 PATCH] hw/arm/bcm2835_peripherals: add bcm283x sp804-alike timer

Posted by no-reply@patchew.org 1 week ago
Patchew URL: https://patchew.org/QEMU/20190209144918.GA14036@nyan/



Hi,

This series failed the docker-mingw@fedora build test. Please find the testing commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.

=== TEST SCRIPT BEGIN ===
#!/bin/bash
time make docker-test-mingw@fedora SHOW_ENV=1 J=14
=== TEST SCRIPT END ===

                 from /tmp/qemu-test/src/include/hw/qdev.h:4,
                 from /tmp/qemu-test/src/include/hw/sysbus.h:6,
                 from /tmp/qemu-test/src/hw/timer/bcm283x_timer.c:2:
/tmp/qemu-test/src/include/qom/object.h:518:12: error: expected ')' before '*' token
     ((type *)object_dynamic_cast_assert(OBJECT(obj), (name), \
            ^
/tmp/qemu-test/src/include/hw/timer/bcm283x_timer.h:24:5: note: in expansion of macro 'OBJECT_CHECK'
     OBJECT_CHECK(BCM283xSP804State, (obj), TYPE_BCM283xSP804)
     ^~~~~~~~~~~~
/tmp/qemu-test/src/include/qom/object.h:518:14: error: expected ')' before 'object_dynamic_cast_assert'
     ((type *)object_dynamic_cast_assert(OBJECT(obj), (name), \
              ^~~~~~~~~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/include/hw/timer/bcm283x_timer.h:24:5: note: in expansion of macro 'OBJECT_CHECK'
     OBJECT_CHECK(BCM283xSP804State, (obj), TYPE_BCM283xSP804)
     ^~~~~~~~~~~~
In file included from /tmp/qemu-test/src/hw/timer/bcm283x_timer.c:7:
/tmp/qemu-test/src/include/hw/timer/bcm283x_timer.h:32:5: error: unknown type name 'bcm283x_timer_state'
     bcm283x_timer_state *timer;
     ^~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:33:34: error: unknown type name 'bcm283x_timer_state'; did you mean 'ptimer_state'?
 static void bcm283x_timer_update(bcm283x_timer_state *s)
                                  ^~~~~~~~~~~~~~~~~~~
                                  ptimer_state
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c: In function 'bcm283x_timer_read':
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:44:5: error: unknown type name 'bcm283x_timer_state'; use 'struct' keyword to refer to the type
     bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
     ^~~~~~~~~~~~~~~~~~~
     struct 
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:44:31: error: 'bcm283x_timer_state' undeclared (first use in this function)
     bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
                               ^~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:44:31: note: each undeclared identifier is reported only once for each function it appears in
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:44:52: error: expected expression before ')' token
     bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
                                                    ^
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:49:17: error: request for member 'limit' in something not a structure or union
         return s->limit;
                 ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:51:34: error: request for member 'timer' in something not a structure or union
         return ptimer_get_count(s->timer);
                                  ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:53:17: error: request for member 'control' in something not a structure or union
         return s->control;
                 ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:61:17: error: request for member 'int_level' in something not a structure or union
         return s->int_level;
                 ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:63:15: error: request for member 'control' in something not a structure or union
         if ((s->control & TIMER_CTRL_IE) == 0) {
               ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:66:17: error: request for member 'int_level' in something not a structure or union
         return s->int_level;
                 ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c: At top level:
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:74:39: error: unknown type name 'bcm283x_timer_state'; did you mean 'ptimer_state'?
 static void bcm283x_timer_recalibrate(bcm283x_timer_state *s, int reload)
                                       ^~~~~~~~~~~~~~~~~~~
                                       ptimer_state
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c: In function 'bcm283x_timer_write':
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:86:5: error: unknown type name 'bcm283x_timer_state'; use 'struct' keyword to refer to the type
     bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
     ^~~~~~~~~~~~~~~~~~~
     struct 
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:86:31: error: 'bcm283x_timer_state' undeclared (first use in this function)
     bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
                               ^~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:86:52: error: expected expression before ')' token
     bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
                                                    ^
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:91:10: error: request for member 'limit' in something not a structure or union
         s->limit = value;
          ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:92:9: error: implicit declaration of function 'bcm283x_timer_recalibrate'; did you mean 'bcm283x_timer_write'? [-Werror=implicit-function-declaration]
         bcm283x_timer_recalibrate(s, 1);
         ^~~~~~~~~~~~~~~~~~~~~~~~~
         bcm283x_timer_write
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:92:9: error: nested extern declaration of 'bcm283x_timer_recalibrate' [-Werror=nested-externs]
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:98:14: error: request for member 'control' in something not a structure or union
         if (s->control & TIMER_CTRL_ENABLE) {
              ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:99:26: error: request for member 'timer' in something not a structure or union
             ptimer_stop(s->timer);
                          ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:102:10: error: request for member 'control' in something not a structure or union
         s->control = value;
          ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:103:17: error: request for member 'freq' in something not a structure or union
         freq = s->freq;
                 ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:115:39: error: request for member 'control' in something not a structure or union
         bcm283x_timer_recalibrate(s, s->control & TIMER_CTRL_ENABLE);
                                       ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:116:26: error: request for member 'timer' in something not a structure or union
         ptimer_set_freq(s->timer, freq);
                          ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:117:14: error: request for member 'control' in something not a structure or union
         if (s->control & TIMER_CTRL_ENABLE) {
              ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:118:25: error: request for member 'timer' in something not a structure or union
             ptimer_run(s->timer, 0);
                         ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:122:10: error: request for member 'int_level' in something not a structure or union
         s->int_level = 0;
          ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:125:10: error: request for member 'limit' in something not a structure or union
         s->limit = value;
          ^~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:134:5: error: implicit declaration of function 'bcm283x_timer_update'; did you mean 'bcm283x_timer_write'? [-Werror=implicit-function-declaration]
     bcm283x_timer_update(s);
     ^~~~~~~~~~~~~~~~~~~~
     bcm283x_timer_write
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:134:5: error: nested extern declaration of 'bcm283x_timer_update' [-Werror=nested-externs]
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c: In function 'bcm283x_timer_tick':
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:139:5: error: unknown type name 'bcm283x_timer_state'; use 'struct' keyword to refer to the type
     bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
     ^~~~~~~~~~~~~~~~~~~
     struct 
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:139:31: error: 'bcm283x_timer_state' undeclared (first use in this function)
     bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
                               ^~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:139:52: error: expected expression before ')' token
     bcm283x_timer_state *s = (bcm283x_timer_state *) opaque;
                                                    ^
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:140:6: error: request for member 'int_level' in something not a structure or union
     s->int_level = 1;
      ^~
In file included from /usr/lib/gcc/x86_64-w64-mingw32/8.2.0/include/stddef.h:1,
---
                 from /tmp/qemu-test/src/include/qemu/osdep.h:103,
                 from /tmp/qemu-test/src/hw/timer/bcm283x_timer.c:1:
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c: At top level:
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:149:33: error: unknown type name 'bcm283x_timer_state'
         VMSTATE_UINT32(control, bcm283x_timer_state),
                                 ^~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/include/migration/vmstate.h:282:21: note: in expansion of macro 'vmstate_offset_value'
---
         ^~~~~~~~~~~~~~
In file included from /tmp/qemu-test/src/include/qemu/osdep.h:51,
                 from /tmp/qemu-test/src/hw/timer/bcm283x_timer.c:1:
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:149:33: error: 'bcm283x_timer_state' undeclared here (not in a function)
         VMSTATE_UINT32(control, bcm283x_timer_state),
                                 ^~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/include/qemu/compiler.h:74:38: note: in definition of macro 'type_check'
---
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:149:9: note: in expansion of macro 'VMSTATE_UINT32'
         VMSTATE_UINT32(control, bcm283x_timer_state),
         ^~~~~~~~~~~~~~
/tmp/qemu-test/src/include/qemu/compiler.h:73:50: error: expected expression before ')' token
 #define typeof_field(type, field) typeof(((type *)0)->field)
                                                  ^
/tmp/qemu-test/src/include/qemu/compiler.h:74:38: note: in definition of macro 'type_check'
---
                 from /usr/x86_64-w64-mingw32/sys-root/mingw/include/inttypes.h:14,
                 from /tmp/qemu-test/src/include/qemu/osdep.h:103,
                 from /tmp/qemu-test/src/hw/timer/bcm283x_timer.c:1:
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:150:31: error: expected specifier-qualifier-list before 'bcm283x_timer_state'
         VMSTATE_UINT32(limit, bcm283x_timer_state),
                               ^~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/include/migration/vmstate.h:282:21: note: in expansion of macro 'vmstate_offset_value'
---
         ^~~~~~~~~~~~~~
In file included from /tmp/qemu-test/src/include/qemu/osdep.h:51,
                 from /tmp/qemu-test/src/hw/timer/bcm283x_timer.c:1:
/tmp/qemu-test/src/include/qemu/compiler.h:73:50: error: expected expression before ')' token
 #define typeof_field(type, field) typeof(((type *)0)->field)
                                                  ^
/tmp/qemu-test/src/include/qemu/compiler.h:74:38: note: in definition of macro 'type_check'
---
                 from /usr/x86_64-w64-mingw32/sys-root/mingw/include/inttypes.h:14,
                 from /tmp/qemu-test/src/include/qemu/osdep.h:103,
                 from /tmp/qemu-test/src/hw/timer/bcm283x_timer.c:1:
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:151:34: error: expected specifier-qualifier-list before 'bcm283x_timer_state'
         VMSTATE_INT32(int_level, bcm283x_timer_state),
                                  ^~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/include/migration/vmstate.h:282:21: note: in expansion of macro 'vmstate_offset_value'
---
         ^~~~~~~~~~~~~
In file included from /tmp/qemu-test/src/include/qemu/osdep.h:51,
                 from /tmp/qemu-test/src/hw/timer/bcm283x_timer.c:1:
/tmp/qemu-test/src/include/qemu/compiler.h:73:50: error: expected expression before ')' token
 #define typeof_field(type, field) typeof(((type *)0)->field)
                                                  ^
/tmp/qemu-test/src/include/qemu/compiler.h:74:38: note: in definition of macro 'type_check'
---
                 from /usr/x86_64-w64-mingw32/sys-root/mingw/include/inttypes.h:14,
                 from /tmp/qemu-test/src/include/qemu/osdep.h:103,
                 from /tmp/qemu-test/src/hw/timer/bcm283x_timer.c:1:
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:152:31: error: expected specifier-qualifier-list before 'bcm283x_timer_state'
         VMSTATE_PTIMER(timer, bcm283x_timer_state),
                               ^~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/include/migration/vmstate.h:449:21: note: in expansion of macro 'vmstate_offset_pointer'
---
                 from /tmp/qemu-test/src/include/hw/qdev.h:4,
                 from /tmp/qemu-test/src/include/hw/sysbus.h:6,
                 from /tmp/qemu-test/src/hw/timer/bcm283x_timer.c:2:
/tmp/qemu-test/src/include/qemu/compiler.h:73:50: error: expected expression before ')' token
 #define typeof_field(type, field) typeof(((type *)0)->field)
                                                  ^
/tmp/qemu-test/src/include/migration/vmstate.h:231:47: note: in definition of macro 'type_check_pointer'
---
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:152:9: note: in expansion of macro 'VMSTATE_PTIMER'
         VMSTATE_PTIMER(timer, bcm283x_timer_state),
         ^~~~~~~~~~~~~~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:157:28: error: expected '=', ',', ';', 'asm' or '__attribute__' before '*' token
 static bcm283x_timer_state *bcm283x_timer_init(uint32_t freq)
                            ^
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c: In function 'bcm283xsp804_init':
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:254:45: error: expected expression before ';' token
     BCM283xSP804State *s = BCM283xSP804(obj);
                                             ^
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c: In function 'bcm283xsp804_realize':
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:266:45: error: expected expression before ';' token
     BCM283xSP804State *s = BCM283xSP804(dev);
                                             ^
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:268:16: error: implicit declaration of function 'bcm283x_timer_init'; did you mean 'bcm283x_timer_write'? [-Werror=implicit-function-declaration]
     s->timer = bcm283x_timer_init(s->freq0);
                ^~~~~~~~~~~~~~~~~~
                bcm283x_timer_write
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:268:16: error: nested extern declaration of 'bcm283x_timer_init' [-Werror=nested-externs]
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:268:14: error: assignment to 'int *' from 'int' makes pointer from integer without a cast [-Werror=int-conversion]
     s->timer = bcm283x_timer_init(s->freq0);
              ^
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:269:13: error: request for member 'irq' in something not a structure or union
     s->timer->irq = qemu_allocate_irq(bcm283xsp804_set_irq, s, 0);
             ^~
At top level:
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:144:33: error: 'vmstate_bcm283x_timer' defined but not used [-Werror=unused-const-variable=]
 static const VMStateDescription vmstate_bcm283x_timer = {
                                 ^~~~~~~~~~~~~~~~~~~~~
/tmp/qemu-test/src/hw/timer/bcm283x_timer.c:137:13: error: 'bcm283x_timer_tick' defined but not used [-Werror=unused-function]
 static void bcm283x_timer_tick(void *opaque)
             ^~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors


The full log is available at
http://patchew.org/logs/20190209144918.GA14036@nyan/testing.docker-mingw@fedora/?type=message.
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@redhat.com