From owner-svn-src-all@FreeBSD.ORG Tue Jun 21 20:44:35 2011 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from [127.0.0.1] (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by hub.freebsd.org (Postfix) with ESMTP id CE02F1065689; Tue, 21 Jun 2011 20:44:34 +0000 (UTC) (envelope-from jkim@FreeBSD.org) From: Jung-uk Kim To: src-committers@FreeBSD.org Date: Tue, 21 Jun 2011 16:44:18 -0400 User-Agent: KMail/1.6.2 References: <201106211929.p5LJTRA5013589@svn.freebsd.org> In-Reply-To: <201106211929.p5LJTRA5013589@svn.freebsd.org> MIME-Version: 1.0 Content-Disposition: inline Content-Type: Multipart/Mixed; boundary="Boundary-00=_lKQAOuKuA7Ph73B" Message-Id: <201106211644.21026.jkim@FreeBSD.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, John Baldwin Subject: Re: svn commit: r223370 - head/sys/dev/acpica X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 21 Jun 2011 20:44:35 -0000 --Boundary-00=_lKQAOuKuA7Ph73B Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline On Tuesday 21 June 2011 03:29 pm, John Baldwin wrote: > Author: jhb > Date: Tue Jun 21 19:29:27 2011 > New Revision: 223370 > URL: http://svn.freebsd.org/changeset/base/223370 > > Log: > Use AcpiWalkResources() to parse the resource list from _CRS > rather than using a home-rolled loop. While here, add support for > 64-bit address range resources. This broke build with ACPI_DEBUG. The attached patch should fix it. > Silence on: acpi@ (older version) Sorry, I am quite busy at $work recently. :-( Jung-uk Kim --Boundary-00=_lKQAOuKuA7Ph73B Content-Type: text/plain; charset="iso-8859-1"; name="acpi_resource.diff" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="acpi_resource.diff" Index: sys/dev/acpica/acpi_resource.c =================================================================== --- sys/dev/acpica/acpi_resource.c (revision 223372) +++ sys/dev/acpica/acpi_resource.c (working copy) @@ -343,22 +343,23 @@ acpi_parse_resource(ACPI_RESOURCE *res, void *cont if (res->Data.Address.MinAddressFixed == ACPI_ADDRESS_FIXED && res->Data.Address.MaxAddressFixed == ACPI_ADDRESS_FIXED) { if (res->Data.Address.ResourceType == ACPI_MEMORY_RANGE) { - ACPI_DEBUG_PRINT((ACPI_DB_RESOURCES, "%s/Memory 0x%x/%d\n", - name, min, length)); + 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%x/%d\n", name, - min, length)); + 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, "%s/Memory 0x%x-0x%x/%d\n", - name, min, max, length)); + ACPI_DEBUG_PRINT((ACPI_DB_RESOURCES, + "%s/Memory 0x%jx-0x%jx/%ju\n", name, (uintmax_t)min, + (uintmax_t)max, (uintmax_t)length)); set->set_memoryrange(dev, arc->context, min, max, length, gran); } else { - ACPI_DEBUG_PRINT((ACPI_DB_RESOURCES, "%s/IO 0x%x-0x%x/%d\n", - name, min, max, length)); + ACPI_DEBUG_PRINT((ACPI_DB_RESOURCES, "%s/IO 0x%jx-0x%jx/%ju\n", + name, (uintmax_t)min, (uintmax_t)max, (uintmax_t)length)); set->set_iorange(dev, arc->context, min, max, length, gran); } } --Boundary-00=_lKQAOuKuA7Ph73B--