git: 7bc993854162 - stable/13 - Support fixed size, variable location acpi resources

Andrew Turner andrew at FreeBSD.org
Fri Aug 6 12:51:20 UTC 2021


The branch stable/13 has been updated by andrew:

URL: https://cgit.FreeBSD.org/src/commit/?id=7bc993854162d38b37f52ce8c7dc2fce1d17a222

commit 7bc993854162d38b37f52ce8c7dc2fce1d17a222
Author:     Andrew Turner <andrew at FreeBSD.org>
AuthorDate: 2021-07-08 15:25:38 +0000
Commit:     Andrew Turner <andrew at FreeBSD.org>
CommitDate: 2021-08-05 20:50:20 +0000

    Support fixed size, variable location acpi resources
    
    These have been found in some Arm ACPI tables generated by edk2, e.g.
    when describing the pl011 uart on the Arm AEMv8 model.
    
    Reviewed by:    imp, jkim
    Sponsored by:   The FreeBSD Foundation
    Differential Revision: https://reviews.freebsd.org/D31110
    
    (cherry picked from commit 1472117a1e1c40f10b9c3a48788fbf8355588ee8)
---
 sys/dev/acpica/acpi_resource.c | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/sys/dev/acpica/acpi_resource.c b/sys/dev/acpica/acpi_resource.c
index fbd0baaa7d37..2e813276af8f 100644
--- a/sys/dev/acpica/acpi_resource.c
+++ b/sys/dev/acpica/acpi_resource.c
@@ -421,6 +421,27 @@ acpi_parse_resource(ACPI_RESOURCE *res, void *context)
 		    (uintmax_t)min, (uintmax_t)length));
 		set->set_ioport(dev, arc->context, min, length);
 	    }
+	} else if (res->Data.Address.MinAddressFixed != ACPI_ADDRESS_FIXED &&
+	    res->Data.Address.MaxAddressFixed != ACPI_ADDRESS_FIXED) {
+	    /* Fixed size, variable location resource descriptor */
+	    min = roundup(min, gran + 1);
+	    if ((min + length - 1) > max) {
+		device_printf(dev,
+		    "invalid memory range: start: %jx end: %jx max: %jx\n",
+		    (uintmax_t)min, (uintmax_t)(min + length - 1),
+		    (uintmax_t)max);
+	    } else {
+		if (res->Data.Address.ResourceType == ACPI_MEMORY_RANGE) {
+		    ACPI_DEBUG_PRINT((ACPI_DB_RESOURCES,
+			"%s/Memory 0x%jx/%ju\n", name, (uintmax_t)min,
+			(uintmax_t)length));
+		    set->set_memory(dev, arc->context, min, length);
+		} else {
+		    ACPI_DEBUG_PRINT((ACPI_DB_RESOURCES, "%s/IO 0x%jx/%ju\n",
+			name, (uintmax_t)min, (uintmax_t)length));
+		    set->set_ioport(dev, arc->context, min, length);
+		}
+	    }
 	} else {
 	    if (res->Data.Address32.ResourceType == ACPI_MEMORY_RANGE) {
 		ACPI_DEBUG_PRINT((ACPI_DB_RESOURCES,


More information about the dev-commits-src-branches mailing list