Re: [PATCH] ASoC: SOF: Update correct LED status at the first time usage of update_mute_led()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Kai-Heng Feng <kai.heng.feng@xxxxxxxxxxxxx>
Subject: Re: [PATCH] ASoC: SOF: Update correct LED status at the first time usage of update_mute_led()
From: Kai Vehmanen <kai.vehmanen@xxxxxxxxxxxxxxx>
Date: Thu, 30 Apr 2020 20:40:32 +0300 (EEST)
Cc: "moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM..." <alsa-devel@xxxxxxxxxxxxxxxx>, kai.vehmanen@xxxxxxxxxxxxxxx, lgirdwood@xxxxxxxxx, open list <linux-kernel@xxxxxxxxxxxxxxx>, Takashi Iwai <tiwai@xxxxxxxx>, ranjani.sridharan@xxxxxxxxxxxxxxx, pierre-louis.bossart@xxxxxxxxxxxxxxx, Mark Brown <broonie@xxxxxxxxxx>, daniel.baluta@xxxxxxx, Jaska Uimonen <jaska.uimonen@xxxxxxxxxxxxxxx>, "moderated list:SOUND - SOUND OPEN FIRMWARE \(SOF\) DRIVERS" <sound-open-firmware@xxxxxxxxxxxxxxxx>
In-reply-to: <20200430091139.7003-1-kai.heng.feng@canonical.com>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7 02160 Espoo
References: <20200430091139.7003-1-kai.heng.feng@canonical.com>
User-agent: Alpine 2.21 (DEB 202 2017-01-01)

Hi,

On Thu, 30 Apr 2020, Kai-Heng Feng wrote:

> At the first time update_mute_led() gets called, if channels are already
> muted, the temp value equals to led_value as 0, skipping the following
> LED setting.

thanks, looks good! 

Acked-by: Kai Vehmanen <kai.vehmanen@xxxxxxxxxxxxxxx>

Br, Kai

References:
- [PATCH] ASoC: SOF: Update correct LED status at the first time usage of update_mute_led()
  - From: Kai-Heng Feng

Prev by Date: Re: [PATCH 0/1] Fiji GPU audio register timeout when in BACO state
Next by Date: Re: Pioneer DJ DJM-900NXS2 support patches
Previous by thread: [PATCH] ASoC: SOF: Update correct LED status at the first time usage of update_mute_led()
Next by thread: Re: [PATCH] ASoC: SOF: Update correct LED status at the first time usage of update_mute_led()
Index(es):
- Date
- Thread

[Index of Archives] [ALSA User] [Linux Audio Users] [Pulse Audio] [Kernel Archive] [Asterisk PBX] [Photo Sharing] [Linux Sound] [Video 4 Linux] [Gimp] [Yosemite News]