[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 03/15] acpi: rtc: use a single crs range
From: |
Igor Mammedov |
Subject: |
Re: [PATCH v5 03/15] acpi: rtc: use a single crs range |
Date: |
Mon, 11 May 2020 21:26:52 +0200 |
On Thu, 7 May 2020 15:16:28 +0200
Gerd Hoffmann <address@hidden> wrote:
> Use a single io range for _CRS instead of two,
> following what real hardware does.
>
> Signed-off-by: Gerd Hoffmann <address@hidden>
Reviewed-by: Igor Mammedov <address@hidden>
> ---
> hw/rtc/mc146818rtc.c | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/hw/rtc/mc146818rtc.c b/hw/rtc/mc146818rtc.c
> index 2104e0aa3b14..ab0cc59973b3 100644
> --- a/hw/rtc/mc146818rtc.c
> +++ b/hw/rtc/mc146818rtc.c
> @@ -1013,12 +1013,14 @@ static void rtc_build_aml(ISADevice *isadev, Aml
> *scope)
> Aml *dev;
> Aml *crs;
>
> + /*
> + * Reserving 8 io ports here, following what physical hardware
> + * does, even though qemu only responds to the first two ports.
> + */
> crs = aml_resource_template();
> aml_append(crs, aml_io(AML_DECODE16, RTC_ISA_BASE, RTC_ISA_BASE,
> - 0x10, 0x02));
> + 0x01, 0x08));
> aml_append(crs, aml_irq_no_flags(RTC_ISA_IRQ));
> - aml_append(crs, aml_io(AML_DECODE16, RTC_ISA_BASE + 2, RTC_ISA_BASE + 2,
> - 0x02, 0x06));
>
> dev = aml_device("RTC");
> aml_append(dev, aml_name_decl("_HID", aml_eisaid("PNP0B00")));
- [PATCH v5 00/15] acpi: i386 tweaks, Gerd Hoffmann, 2020/05/07
- [PATCH v5 01/15] qtest: allow DSDT acpi table changes, Gerd Hoffmann, 2020/05/07
- [PATCH v5 03/15] acpi: rtc: use a single crs range, Gerd Hoffmann, 2020/05/07
- Re: [PATCH v5 03/15] acpi: rtc: use a single crs range,
Igor Mammedov <=
- [PATCH v5 02/15] acpi: move aml builder code for rtc device, Gerd Hoffmann, 2020/05/07
- [PATCH v5 04/15] acpi: serial: don't use _STA method, Gerd Hoffmann, 2020/05/07
- [PATCH v5 07/15] acpi: move aml builder code for parallel device, Gerd Hoffmann, 2020/05/07
- [PATCH v5 10/15] acpi: factor out fw_cfg_add_acpi_dsdt(), Gerd Hoffmann, 2020/05/07
- [PATCH v5 08/15] acpi: move aml builder code for floppy device, Gerd Hoffmann, 2020/05/07