linux-kernel - RE: [PATCH v4 2/2] ASoC: rt715:add micmute led state control supports

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] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 22 Mar 2021 09:25:25 +0000
From:   "Yuan, Perry" <Perry.Yuan@...l.com>
To:     Mark Brown <broonie@...nel.org>,
        "pierre-louis.bossart@...ux.intel.com" 
        <pierre-louis.bossart@...ux.intel.com>,
        "Limonciello, Mario" <Mario.Limonciello@...l.com>,
        "hdegoede@...hat.com" <hdegoede@...hat.com>
CC:     "pobrn@...tonmail.com" <pobrn@...tonmail.com>,
        "oder_chiou@...ltek.com" <oder_chiou@...ltek.com>,
        "perex@...ex.cz" <perex@...ex.cz>,
        "tiwai@...e.com" <tiwai@...e.com>,
        "mgross@...ux.intel.com" <mgross@...ux.intel.com>,
        "lgirdwood@...il.com" <lgirdwood@...il.com>,
        "alsa-devel@...a-project.org" <alsa-devel@...a-project.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "platform-driver-x86@...r.kernel.org" 
        <platform-driver-x86@...r.kernel.org>
Subject: RE: [PATCH v4 2/2] ASoC: rt715:add micmute led state control supports

Hi Mark:

> -----Original Message-----
> From: Mark Brown <broonie@...nel.org>
> Sent: Tuesday, March 9, 2021 1:24 AM
> To: Yuan, Perry
> Cc: pobrn@...tonmail.com; pierre-louis.bossart@...ux.intel.com;
> oder_chiou@...ltek.com; perex@...ex.cz; tiwai@...e.com;
> hdegoede@...hat.com; mgross@...ux.intel.com; Limonciello, Mario;
> lgirdwood@...il.com; alsa-devel@...a-project.org; linux-
> kernel@...r.kernel.org; platform-driver-x86@...r.kernel.org
> Subject: Re: [PATCH v4 2/2] ASoC: rt715:add micmute led state control
> supports
> 
> On Mon, Mar 01, 2021 at 05:38:34PM +0800, Perry Yuan wrote:
> 
> > +	/* Micmute LED state changed by muted/unmute switch */
> > +	if (mc->invert) {
> > +		if (ucontrol->value.integer.value[0] || ucontrol-
> >value.integer.value[1]) {
> > +			micmute_led = LED_OFF;
> > +		} else {
> > +			micmute_led = LED_ON;
> > +		}
> > +		ledtrig_audio_set(LED_AUDIO_MICMUTE, micmute_led);
> > +	}
> 
> These conditionals on inversion seem weird and counterintuitive.  If we're
> going with this approach it would probably be clearer to define a custom
> operation for the affected controls that wraps the standard one and adds the
> LED setting rather than keying off invert like this.

Currently the sof soundwire driver has no generic led control yet.
This patch can handle the led control needs for MIC mute LED, definitely the patch is a short term solution.
There is a feature request discussion when we started to implement this solution.
https://github.com/thesofproject/linux/issues/2496#issuecomment-713892620

The workable way for now is that we put the LED mute control to the codec driver.
When there is new and full sound LED solution implemented, this part will be also optimized.
The Hardware privacy feature needs this patch to handle the Mic mute led state change.
Before that full solution ready in kernel, could we take this as short term solution?

Perry