From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH v2 0/3] Enable mute LEDs and mic mute LEDs on Thinkpad Date: Thu, 17 Oct 2013 12:08:46 +0200 Message-ID: References: <1381957833-13788-1-git-send-email-david.henningsson@canonical.com> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Return-path: Received: from cantor2.suse.de ([195.135.220.15]:47566 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753922Ab3JQKFu (ORCPT ); Thu, 17 Oct 2013 06:05:50 -0400 In-Reply-To: <1381957833-13788-1-git-send-email-david.henningsson@canonical.com> Sender: platform-driver-x86-owner@vger.kernel.org List-ID: To: David Henningsson Cc: alsa-devel@alsa-project.org, platform-driver-x86@vger.kernel.org, ibm-acpi@hmh.eng.br, alex.hung@canonical.com, hui.wang@canonical.com At Wed, 16 Oct 2013 23:10:30 +0200, David Henningsson wrote: > > Encouraged by the quick and helpful feedback from both Takashi and Henrique (thanks!), > I went to work with v2 of the patch. > > Changes since v1: > > - thinkpad-acpi patch: rewritten to follow Takashi's ideas > - thinkpad-acpi patch: added documentation update > - alsa side: added a separate patch for the new fixup action > - alsa side: rewrote patch a little, mostly based on comments from Takashi > > David Henningsson (3): > thinkpad-acpi: Add mute and mic-mute LED functionality > ALSA: hda - add HDA_FIXUP_ACT_FREE action > ALSA: hda - add connection to thinkpad_acpi to control mute/micmute > LEDs The new patches look OK to me. If Henrique agrees, I can apply all to sound git tree. thanks, Takashi > > Documentation/laptops/thinkpad-acpi.txt | 7 ++- > drivers/platform/x86/thinkpad_acpi.c | 92 ++++++++++++++++++++++++++++++- > include/linux/thinkpad_acpi.h | 15 +++++ > sound/pci/hda/hda_local.h | 1 + > sound/pci/hda/patch_conexant.c | 92 ++++++++++++++++++++++++++++++- > 5 files changed, 202 insertions(+), 5 deletions(-) > create mode 100644 include/linux/thinkpad_acpi.h > > -- > 1.7.9.5 >