From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Garrett Subject: Re: [PATCH 1/4] ACPI: EC: Add ec_get_handle() Date: Fri, 16 Dec 2011 00:33:33 +0000 Message-ID: <20111216003333.GA18489@srcf.ucam.org> References: <1323972371-13189-1-git-send-email-seth.forshee@canonical.com> <1323972371-13189-2-git-send-email-seth.forshee@canonical.com> <201112160122.36100.trenn@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from cavan.codon.org.uk ([93.93.128.6]:44082 "EHLO cavan.codon.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756418Ab1LPAdj (ORCPT ); Thu, 15 Dec 2011 19:33:39 -0500 Content-Disposition: inline In-Reply-To: <201112160122.36100.trenn@suse.de> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Thomas Renninger Cc: Seth Forshee , Len Brown , Azael Avalos , platform-driver-x86@vger.kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, Henrique de Moraes Holschuh On Fri, Dec 16, 2011 at 01:22:35AM +0100, Thomas Renninger wrote: > I think best is to move the thinkpad implementation of getting > ACPI handles based on HIDs to osl.c and make it global. > I'll send patches. > Please review them carefully, they are only compile tested. The ec driver is already finding the hardware on the basis of the HID - is there any reason to do this twice rather than just exporting the information the ec driver already has? -- Matthew Garrett | mjg59@srcf.ucam.org