Re: [PATCH] ASoC: tegra: Add master volume/mute control support

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

 





On 10/29/2021 8:56 PM, Jaroslav Kysela wrote:
External email: Use caution opening links or attachments


On 29. 10. 21 17:08, Sameer Pujar wrote:


On 10/26/2021 11:53 AM, Sameer Pujar wrote:


On 10/25/2021 6:28 PM, Jaroslav Kysela wrote:
On 25. 10. 21 13:06, Sameer Pujar wrote:

@@ -150,11 +186,22 @@ static int tegra210_mvc_put_mute(struct
snd_kcontrol *kcontrol,

...

       return 1;

It's a bit unrelated comment to this change, but it may be worth to
verify all
kcontrol put callbacks in the tegra code. Ensure that value 1 is
returned only
when something was really changed in hardware.

There are cases when the mixer control update is not immediately written
to HW, instead the update is ACKed (stored in variable) and writen to HW
at a later point of time. Do these cases qualify for "return 1" as well?

Yes - assuming that the get callback returns the cached value. The get/put
implementation should be consistent from the caller view. The driver
implementation (delayed write) is a separate thing.

Thanks Jaroslav and Mark. I have now sent a separate series to fix Tegra210 (and later) drivers.



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

  Powered by Linux