Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754096AbYAWTtW (ORCPT ); Wed, 23 Jan 2008 14:49:22 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754070AbYAWTsw (ORCPT ); Wed, 23 Jan 2008 14:48:52 -0500 Received: from g4t0014.houston.hp.com ([15.201.24.17]:13253 "EHLO g4t0014.houston.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752892AbYAWTsu (ORCPT ); Wed, 23 Jan 2008 14:48:50 -0500 Subject: Re: [PATCH 3/4] ACPI: Check for any matching CID when walking namespace. From: Andrew Patterson To: Len Brown Cc: robert.moore@intel.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, gregkh@suse.de, matthew@wil.cx, tom.l.nguyen@intel.com, linux-acpi@vger.kernel.org In-Reply-To: <200801231439.42735.lenb@kernel.org> References: <20080123001807.28190.32648.stgit@bluto.andrew> <20080123001822.28190.3147.stgit@bluto.andrew> <200801231439.42735.lenb@kernel.org> Content-Type: text/plain Date: Wed, 23 Jan 2008 12:48:49 -0700 Message-Id: <1201117729.12916.132.camel@bluto.andrew> Mime-Version: 1.0 X-Mailer: Evolution 2.12.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2459 Lines: 79 On Wed, 2008-01-23 at 14:39 -0500, Len Brown wrote: > Right Greg, this is not only an ACPI patch, > but a patch that touches the ACPICA core. > Lets get Bob to see if it applies to the latest upstream ACPICA code. > > Andrew, > Do you give Intel permission to accept this patch under > both licenses on the top of the file, such that we can > re-distribute it in ACPICA to all the OSs that use ACPICA? > Yes, by all means. Andrew Patterson Hewlett-Packard > (hey, and check out the spiffy new acpica home page: http://acpica.org/ ) > > thanks, > -Len > > On Tuesday 22 January 2008 19:18, Andrew Patterson wrote: > > From: Andrew Patterson > > > > The callback function acpi_ns_get_device_callback called from > > acpi_get_devices() will check CID's if the HID does not match. This code > > has a bug where it requires that all CIDs match the HID. Changed the code > > so that any CID match will do. > > > > Signed-off-by: Andrew Patterson > > --- > > > > drivers/acpi/namespace/nsxfeval.c | 11 ++++++++--- > > 1 files changed, 8 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/acpi/namespace/nsxfeval.c b/drivers/acpi/namespace/nsxfeval.c > > index f39fbc6..e562b24 100644 > > --- a/drivers/acpi/namespace/nsxfeval.c > > +++ b/drivers/acpi/namespace/nsxfeval.c > > @@ -443,6 +443,7 @@ acpi_ns_get_device_callback(acpi_handle obj_handle, > > struct acpica_device_id hid; > > struct acpi_compatible_id_list *cid; > > acpi_native_uint i; > > + int found; > > > > status = acpi_ut_acquire_mutex(ACPI_MTX_NAMESPACE); > > if (ACPI_FAILURE(status)) { > > @@ -496,16 +497,20 @@ acpi_ns_get_device_callback(acpi_handle obj_handle, > > > > /* Walk the CID list */ > > > > + found = 0; > > for (i = 0; i < cid->count; i++) { > > if (ACPI_STRNCMP(cid->id[i].value, info->hid, > > sizeof(struct > > - acpi_compatible_id)) != > > + acpi_compatible_id)) == > > 0) { > > - ACPI_FREE(cid); > > - return (AE_OK); > > + found = 1; > > + break; > > } > > } > > ACPI_FREE(cid); > > + if (!found) { > > + return (AE_OK); > > + } > > } > > } > > > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/