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: Content-Disposition: inline In-Reply-To: <201112160122.36100.trenn@suse.de> Sender: linux-acpi-owner@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 List-Id: platform-driver-x86.vger.kernel.org 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