[PATCH v2 0/3] ALSA: usb-audio: add module param device_quirk_flags

Cryolitia PukNgae via B4 Relay
2025-09-12T06:49:11Z

As an implementation of what has been discussed previously[1].

> An open question is whether we may want yet a new module option or
> rather extend the existing quirk option to accept the strings
> instead. Basically, when the given argument has a colon, it's a new
> syntax. If it's only a number, it's an old syntax, and parse like
> before. But, I'm open for either way (a new option or extend the
> existing one).

I would like to add a new param. The existed param
`static unsigned int quirk_flags[SNDRV_CARDS]` seems to related to
some sequence the card probed. To be honest, I havn't fully understood
it. And it seems hard to improve it while keeping compatibility.

1. https://lore.kernel.org/all/87h5xm5g7f.wl-tiwai@suse.de/

Signed-off-by: Cryolitia PukNgae <cryolitia@uniontech.com>
---
Changes in v2:
- Cleaned up some internal rebase confusion, sorry for that
- Link to v1: https://lore.kernel.org/r/20250912-sound-v1-0-cc9cfd9f2d01@uniontech.com

---
Cryolitia PukNgae (3):
ALSA: usb-audio: add two-way convert between name and bit for QUIRK_FLAG_*
ALSA: usb-audio: add module param device_quirk_flags
ALSA: doc: add docs about device_device_quirk_flags in snd-usb-audio

Documentation/sound/alsa-configuration.rst | 105 ++++++++++++------
sound/usb/card.c | 165 ++++++++++++++++++++++++++++-
sound/usb/quirks.c | 110 ++++++++++++++++++-
sound/usb/quirks.h | 5 +
sound/usb/usbaudio.h | 14 +++
5 files changed, 362 insertions(+), 37 deletions(-)
---
base-commit: 82ad508a85dc64cb4bd648edec5a4ce741648426
change-id: 20250910-sound-a91c86c92dba

Best regards,
--
Cryolitia PukNgae <cryolitia@uniontech.com>