[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] rtc: initialize irq even if caller doesn't use
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH] rtc: initialize irq even if caller doesn't use rtc_init() function |
Date: |
Mon, 23 Jan 2012 12:20:42 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2012-01-23 10:18, Hervé Poussineau wrote:
> rtc instanciated with -device has now a working irq line
That makes no sense. The mc146818rtc is no_user - for a reason.
What is the use case you are trying to address?
Jan
>
> Signed-off-by: Hervé Poussineau <address@hidden>
> ---
> hw/mc146818rtc.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/mc146818rtc.c b/hw/mc146818rtc.c
> index 657fa10..3067022 100644
> --- a/hw/mc146818rtc.c
> +++ b/hw/mc146818rtc.c
> @@ -672,6 +672,7 @@ static int rtc_initfn(ISADevice *dev)
>
> memory_region_init_io(&s->io, &cmos_ops, s, "rtc", 2);
> isa_register_ioport(dev, &s->io, base);
> + isa_init_irq(dev, &s->irq, RTC_ISA_IRQ);
>
> qdev_set_legacy_instance_id(&dev->qdev, base, 2);
> qemu_register_reset(rtc_reset, s);
> @@ -692,9 +693,8 @@ ISADevice *rtc_init(ISABus *bus, int base_year, qemu_irq
> intercept_irq)
> qdev_prop_set_int32(&dev->qdev, "base_year", base_year);
> qdev_init_nofail(&dev->qdev);
> if (intercept_irq) {
> + /* Use provided IRQ instead of default one */
> s->irq = intercept_irq;
> - } else {
> - isa_init_irq(dev, &s->irq, RTC_ISA_IRQ);
> }
> return dev;
> }
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux