linux-kernel - Re: [PATCH] thinkpad_acpi: Support micmute LED

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 14 Jul 2015 07:35:05 -0300
From:	Henrique de Moraes Holschuh <hmh@....eng.br>
To:	"Jason A. Donenfeld" <Jason@...c4.com>,
	ibm-acpi-devel@...ts.sourceforge.net, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] thinkpad_acpi: Support micmute LED

NACK.

This LED needs to be hooked internally (kernel-side) to the ALSA mixer
entry responsible for MIC muting.  It should NOT be exposed to userspace
directly.

The LED state should only be modified when the MIC mute gate state
changes.  This means a private trigger, directly in ALSA, like the
speaker mute led.

And it most definately is not supposed to be added to TPACPI_SAFE_LEDS
*ever*.

On Mon, Jul 13, 2015, at 19:46, Jason A. Donenfeld wrote:
> The micmute LED is currently unused. This patch allows it to be hooked
> up to various LED triggers.
> 
> Signed-off-by: Jason A. Donenfeld <Jason@...c4.com>
> ---
>  drivers/platform/x86/thinkpad_acpi.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/platform/x86/thinkpad_acpi.c
> b/drivers/platform/x86/thinkpad_acpi.c
> index 33e488c..ca4c435 100644
> --- a/drivers/platform/x86/thinkpad_acpi.c
> +++ b/drivers/platform/x86/thinkpad_acpi.c
> @@ -5281,8 +5281,10 @@ static const char * const
> tpacpi_led_names[TPACPI_LED_NUMLEDS] = {
>  	"tpacpi::unknown_led2",
>  	"tpacpi::unknown_led3",
>  	"tpacpi::thinkvantage",
> +       "tpacpi::unknown_led4",
> +       "tpacpi::micmute"
>  };
> -#define TPACPI_SAFE_LEDS       0x1081U
> +#define TPACPI_SAFE_LEDS       0x5081U
>  t
>  static inline bool tpacpi_is_led_restricted(const unsigned int led)
>  {
> @@ -5505,7 +5507,7 @@ static const struct tpacpi_quirk
> led_useful_qtable[] __initconst = {
>  	{ /* Lenovo */
>  	  .vendor = PCI_VENDOR_ID_LENOVO,
>  	  .bios = TPACPI_MATCH_ANY, .ec = TPACPI_MATCH_ANY,
> -         .quirks = 0x1fffU,
> +         .quirks = 0x5fffU,
>  	},
>  	{ /* IBM ThinkPads with no EC version string */
>  	  .vendor = PCI_VENDOR_ID_IBM,

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/