From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50356) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZNJHr-0006uE-BU for qemu-devel@nongnu.org; Thu, 06 Aug 2015 07:24:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZNJHo-00021D-3Y for qemu-devel@nongnu.org; Thu, 06 Aug 2015 07:24:39 -0400 Received: from mail-wi0-f169.google.com ([209.85.212.169]:35318) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZNJHn-000214-Tq for qemu-devel@nongnu.org; Thu, 06 Aug 2015 07:24:36 -0400 Received: by wibxm9 with SMTP id xm9so18258852wib.0 for ; Thu, 06 Aug 2015 04:24:34 -0700 (PDT) From: Leif Lindholm Date: Thu, 6 Aug 2015 12:24:27 +0100 Message-Id: <1438860267-3401-1-git-send-email-leif.lindholm@linaro.org> Subject: [Qemu-devel] [PATCH] hw/arm/virt-acpi-build: drop _ADR entry from SPCR List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: drjones@redhat.com, shannon.zhao@linaro.org The _ADR entry in SPCR is optional and redundant. The same information is already provided in _CRS (which is mandatory). Signed-off-by: Leif Lindholm --- So, this _ADR entry is only consumed by a set of not-widely-circulated patches for the Linux kernel. And while the ARM Server Base Boot Requirements specification mandates SPCR, it does not mandate this _ADR entry. In the interest of not propagating non-standard extensions, I would be really happy if we could consider dropping this from 2.4. I do realize that this is a completely unreasonable request this late in the release process, but I only spotted this yesterday, and it is a very isolated change with very quantifiable effects. The patch at https://git.linaro.org/leg/acpi/leg-kernel.git/commitdiff/46eeec7b7332bdd104941703696d3812afd934c8 converts the non-upstream kernel SPCR handling code to use the _CRS information instead. hw/arm/virt-acpi-build.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c index f365140..85eb48c 100644 --- a/hw/arm/virt-acpi-build.c +++ b/hw/arm/virt-acpi-build.c @@ -84,12 +84,6 @@ static void acpi_dsdt_add_uart(Aml *scope, const MemMapEntry *uart_memmap, aml_interrupt(AML_CONSUMER, AML_LEVEL, AML_ACTIVE_HIGH, AML_EXCLUSIVE, uart_irq)); aml_append(dev, aml_name_decl("_CRS", crs)); - - /* The _ADR entry is used to link this device to the UART described - * in the SPCR table, i.e. SPCR.base_address.address == _ADR. - */ - aml_append(dev, aml_name_decl("_ADR", aml_int(uart_memmap->base))); - aml_append(scope, dev); } -- 2.1.4