[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-1.5 v2 08/20] mc146818rtc: QOM'ify
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH for-1.5 v2 08/20] mc146818rtc: QOM'ify |
Date: |
Mon, 29 Apr 2013 10:43:52 +0200 |
On Sat, 27 Apr 2013 22:18:43 +0200
Andreas Färber <address@hidden> wrote:
> Introduce type constant and cast macro to obsolete DO_UPCAST().
>
> Prepares for ISA realizefn.
>
> Signed-off-by: Andreas Färber <address@hidden>
> ---
> hw/timer/mc146818rtc.c | 36 ++++++++++++++++++++----------------
> include/hw/timer/mc146818rtc.h | 2 ++
> 2 files changed, 22 insertions(+), 16 deletions(-)
>
> diff --git a/hw/timer/mc146818rtc.c b/hw/timer/mc146818rtc.c
> index 69e6844..afbd0db 100644
> --- a/hw/timer/mc146818rtc.c
> +++ b/hw/timer/mc146818rtc.c
> @@ -57,8 +57,11 @@
> #define RTC_CLOCK_RATE 32768
> #define UIP_HOLD_LENGTH (8 * NSEC_PER_SEC / 32768)
>
> +#define MC146818_RTC(obj) OBJECT_CHECK(RTCState, (obj), TYPE_MC146818_RTC)
> +
> typedef struct RTCState {
> - ISADevice dev;
> + ISADevice parent_obj;
> +
> MemoryRegion io;
> uint8_t cmos_data[128];
> uint8_t cmos_index;
> @@ -672,14 +675,14 @@ static uint64_t cmos_ioport_read(void *opaque, hwaddr
> addr,
>
> void rtc_set_memory(ISADevice *dev, int addr, int val)
> {
> - RTCState *s = DO_UPCAST(RTCState, dev, dev);
> + RTCState *s = MC146818_RTC(dev);
> if (addr >= 0 && addr <= 127)
> s->cmos_data[addr] = val;
> }
>
> static void rtc_set_date_from_host(ISADevice *dev)
> {
> - RTCState *s = DO_UPCAST(RTCState, dev, dev);
> + RTCState *s = MC146818_RTC(dev);
> struct tm tm;
>
> qemu_get_timedate(&tm, 0);
> @@ -741,7 +744,7 @@ static void rtc_notify_clock_reset(Notifier *notifier,
> void *data)
> RTCState *s = container_of(notifier, RTCState, clock_reset_notifier);
> int64_t now = *(int64_t *)data;
>
> - rtc_set_date_from_host(&s->dev);
> + rtc_set_date_from_host(ISA_DEVICE(s));
> periodic_timer_update(s, now);
> check_update_timer(s);
> #ifdef TARGET_I386
> @@ -756,7 +759,7 @@ static void rtc_notify_clock_reset(Notifier *notifier,
> void *data)
> static void rtc_notify_suspend(Notifier *notifier, void *data)
> {
> RTCState *s = container_of(notifier, RTCState, suspend_notifier);
> - rtc_set_memory(&s->dev, 0xF, 0xFE);
> + rtc_set_memory(ISA_DEVICE(s), 0xF, 0xFE);
> }
>
> static void rtc_reset(void *opaque)
> @@ -789,8 +792,7 @@ static const MemoryRegionOps cmos_ops = {
> static void rtc_get_date(Object *obj, Visitor *v, void *opaque,
> const char *name, Error **errp)
> {
> - ISADevice *isa = ISA_DEVICE(obj);
> - RTCState *s = DO_UPCAST(RTCState, dev, isa);
> + RTCState *s = MC146818_RTC(obj);
> struct tm current_tm;
>
> rtc_update_time(s);
> @@ -807,7 +809,7 @@ static void rtc_get_date(Object *obj, Visitor *v, void
> *opaque,
>
> static int rtc_initfn(ISADevice *dev)
> {
> - RTCState *s = DO_UPCAST(RTCState, dev, dev);
> + RTCState *s = MC146818_RTC(dev);
> int base = 0x70;
>
> s->cmos_data[RTC_REG_A] = 0x26;
> @@ -866,19 +868,21 @@ static int rtc_initfn(ISADevice *dev)
>
> ISADevice *rtc_init(ISABus *bus, int base_year, qemu_irq intercept_irq)
> {
> - ISADevice *dev;
> + DeviceState *dev;
> + ISADevice *isadev;
> RTCState *s;
>
> - dev = isa_create(bus, "mc146818rtc");
> - s = DO_UPCAST(RTCState, dev, dev);
> - qdev_prop_set_int32(&dev->qdev, "base_year", base_year);
> - qdev_init_nofail(&dev->qdev);
> + isadev = isa_create(bus, TYPE_MC146818_RTC);
> + dev = DEVICE(isadev);
> + s = MC146818_RTC(isadev);
> + qdev_prop_set_int32(dev, "base_year", base_year);
> + qdev_init_nofail(dev);
> if (intercept_irq) {
> s->irq = intercept_irq;
> } else {
> - isa_init_irq(dev, &s->irq, RTC_ISA_IRQ);
> + isa_init_irq(isadev, &s->irq, RTC_ISA_IRQ);
> }
> - return dev;
> + return isadev;
> }
>
> static Property mc146818rtc_properties[] = {
> @@ -899,7 +903,7 @@ static void rtc_class_initfn(ObjectClass *klass, void
> *data)
> }
>
> static const TypeInfo mc146818rtc_info = {
> - .name = "mc146818rtc",
> + .name = TYPE_MC146818_RTC,
> .parent = TYPE_ISA_DEVICE,
> .instance_size = sizeof(RTCState),
> .class_init = rtc_class_initfn,
> diff --git a/include/hw/timer/mc146818rtc.h b/include/hw/timer/mc146818rtc.h
> index 854ea3f..425bd17 100644
> --- a/include/hw/timer/mc146818rtc.h
> +++ b/include/hw/timer/mc146818rtc.h
> @@ -4,6 +4,8 @@
> #include "hw/isa/isa.h"
> #include "hw/timer/mc146818rtc_regs.h"
>
> +#define TYPE_MC146818_RTC "mc146818rtc"
> +
> ISADevice *rtc_init(ISABus *bus, int base_year, qemu_irq intercept_irq);
> void rtc_set_memory(ISADevice *dev, int addr, int val);
> void rtc_set_date(ISADevice *dev, const struct tm *tm);
> --
> 1.8.1.4
>
>
Reviewed-By: Igor Mammedov <address@hidden>
--
Regards,
Igor
- [Qemu-devel] [PATCH for-1.5 v2 00/20] QOM realize for ISA, part 1, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 02/20] cirrus_vga: QOM'ify ISA Cirrus VGA, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 01/20] applesmc: QOM'ify, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 04/20] i82374: QOM'ify, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 03/20] fdc: QOM'ify ISA floppy controller, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 07/20] m48t59: QOM'ify ISA M48T59 NVRAM, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 06/20] ide: QOM'ify ISA IDE, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 05/20] i8259: QOM cleanups, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 10/20] parallel: QOM'ify, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 08/20] mc146818rtc: QOM'ify, Andreas Färber, 2013/04/27
- Re: [Qemu-devel] [PATCH for-1.5 v2 08/20] mc146818rtc: QOM'ify,
Igor Mammedov <=
- [Qemu-devel] [PATCH for-1.5 v2 09/20] ne2000-isa: QOM'ify, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 11/20] pc: QOM'ify port 92, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 12/20] pckbd: QOM'ify, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 13/20] pcspk: QOM'ify, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 14/20] sb16: QOM'ify, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 15/20] serial: QOM'ify ISA serial, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 16/20] sga: QOM'ify, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 17/20] vga-isa: QOM'ify ISA VGA, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 18/20] vmmouse: QOM'ify, Andreas Färber, 2013/04/27
- [Qemu-devel] [PATCH for-1.5 v2 19/20] vmport: QOM'ify, Andreas Färber, 2013/04/27