Re: [RFC PATCH] docs: sound: kernel-api: writing-an-alsa-driver.rst: add FIXMEs

[Oswald Buddenhagen <oswald.buddenhagen@gmx.de>]

On Thu, Apr 06, 2023 at 08:42:27AM +0200, Takashi Iwai wrote:
>On Wed, 05 Apr 2023 22:12:20 +0200, Oswald Buddenhagen wrote:
>>  The ``name`` is the name identifier string. Since ALSA 0.9.x, the
>>  control name is very important, because its role is classified from
>>  its name. There are pre-defined standard control names. The details
>>  are described in the `Control Names`_ subsection.
>> +// This is a questionable design, IMO. Why user-space heuristics when
>> +// the driver could set the roles/capabilities? This would avoid
>> +// problems like the Tone Control sliders (unlike the switch?!) being
>> +// misclassified as applying also to capture.
>
>Why this has to be discussed here and now...?
>
why not?

>It's the thing that was *defined* over two decades ago.
>
that may be so, but this doesn't explain anything.
it's a somewhat surprising choice, and it does in fact sometimes cause 
problems. so at least it should be thoroughly explained.

>> +// FIXME: explain.
>
>A patch please.
>
well, if i knew what to write there without doing deeper research first, 
i'd have already included it into the doc update. if you give me rough 
drafts (even just somewhat extensive bullet points), i can polish it for 
you (though i suspect that nowadays you may just dump it into chatgpt 
and get something reasonable out of it).

regards