Re: [PATCH v7 1/7] ASoC: qcom: qdsp6: add topology-driven Audio IF support
Cezary Rojewski <[email protected]> Tue, 4 Aug 2026 19:46:58 +0200
| Newsgroups | gmane.linux.drivers.devicetree,gmane.linux.ports.arm.msm,gmane.linux.sound,gmane.linux.kernel |
|---|---|
| Message-ID | <[email protected]> |
On 8/4/2026 9:03 AM, Prasad Kumpatla wrote: > Add topology parsing and media-format programming for Audio IF source and > sink modules. ... > include/uapi/sound/snd_ar_tokens.h | 93 ++++++++++++++++++++++++++ Meh, looks like QCOM took a "bad" example from Intel's past drivers and put the header directly within uapi/sound/ without any meaningful prefix. Even moving the header into vendor-specific subfolder e.g.: uapi/sound/qcom/ would be a good start. For "better" examples take a look at headers exposed for newer drivers: sound/intel and sound/sof. I'd suggest to move away from snd_ar_tokens.h (the name is as ambiguous is at can be) and start new header family within 'qcom' subfolder. At the same time I'll prepare a patch and remove the remaining "bad" examples so people don't propagate them. Kind regards, Czarek