Hello Takashi Sakamoto, Takashi Iwai, Andrew Chant, Thank you for the discussion. I saw the drawbacks of my proposal, and it was nice to clarify them with you. Also, during our discussion I found, that the "description of playback switches" has a different solution. I will check my new idea. If it will work good enough, I will send a patch later. >From my perspective, we can consider this RFC to be closed. Thanks for your attention. Best Regards, Kirill On 03/26/18 23:53, Kirill Marinushkin wrote: > On 03/26/18 23:44, Takashi Sakamoto wrote: >> Hi, >> >> On Mar 27 2018 06:13, Kirill Marinushkin wrote: >>> On 03/26/18 22:41, Andrew Chant wrote: >>>> On Mon, Mar 26, 2018 at 11:32 AM, Kirill Marinushkin >>>> <k.marinushkin@xxxxxxxxx> wrote: >>>>> I faced 2 problems: >>>>> >>>>> * the clock source validity was displayed as [O]. I thought it is a zero, which is the opposite from the real state >>>>> * I couldn't find how to turn the processing feature on. The help doesn't say anything about it >>>> A recent patch of mine changed clock source validity from a mixer to a >>>> global control, so it should no longer show up in alsamixer. >>> Hello Andrew, >>> >>> Nice! One of the problems is solved. However, the second problem is still actual. >>> As you had a recent experience with alsamixer, what is your opinion? >> I think Kirill mentions a patch to change the type of 'iface' for such control elements to 'SNDRV_CTL_ELEM_IFACE_CARD' or 'SNDRV_CTL_ELEM_IFACE_PCM', but I cannot find such patches in Iwai-san's tree[1]. I guess they're not merged yet. > Andrew mentioned the patch [2] > >> In my humble opinion, if so, Kirill's patch is not better in a view of compatibility, because such smixer controls suddenly disappear via alsa-lib's mixer APIs when the patched kernel is released. In a point of backward compatibility, this kind of change should be avoided, as possible (of cource, depending on the cases). > The patch of Andrew and my RFC serve different purposes. They do not replace each other. > Patch of Andrew made the "clock valid" global. > My RFC is to add the string in the help, to clarify how to toggle the switches in alsamixer. > >> >> [1] https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/log/?h=for-next >> > [2] https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/commit/?h=for-next&id=568fa7e087ef98bc85b5aa31ea7c9252c1305c1f > >> Regards >> >> Takashi Sakamoto _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
