From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Renninger Subject: Re: [PATCH 5/6] X86 platform drivers: Remove EC dump from thinkpad_acpi Date: Fri, 2 Jul 2010 11:14:00 +0200 Message-ID: <201007021114.00982.trenn@suse.de> References: <1277996570-2686-1-git-send-email-trenn@suse.de> <20100701162741.GC4789@khazad-dum.debian.net> <201007012131.20031.trenn@suse.de> Mime-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from cantor2.suse.de ([195.135.220.15]:59151 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754200Ab0GBJOF (ORCPT ); Fri, 2 Jul 2010 05:14:05 -0400 In-Reply-To: <201007012131.20031.trenn@suse.de> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Henrique de Moraes Holschuh Cc: lenb@kernel.org, linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, platform-driver-x86@vger.kernel.org, astarikovskiy@suse.de, ibm-acpi-devel@lists.sourceforge.net On Thursday 01 July 2010 21:31:18 Thomas Renninger wrote: > On Thursday 01 July 2010 06:27:41 pm Henrique de Moraes Holschuh wrote: > > On Thu, 01 Jul 2010, Thomas Renninger wrote: > ... > > I'd prefer if the above text gets replaced by something that adds a pointer > > to the new facility, instead of being just outright removed. Also, the > > text about how to use the facility to detect tachometers and thermometers > > should be retained, if at all possible. > I agree. I'll repost this one in some days. Hmm, I take that back. This still should get removed completely from: Documentation/laptops/thinkpad-acpi.txt These problems apply to all laptops and instead of adding pointers to all Documentation/laptop/${vendor} files, a separate file should be added. This should then be more about thermal management problems in general and pointers to: Documentation/acpi/ec_sysfs and Documentation/acpi/debug.txt Documentation/acpi/* added there looks like the appropriate solution to me. If people agree: - This patch is still fine and should get applied as it is - I will send another patch adding some more documentation in some days Thanks, Thomas