[v2,0/6] ALSA: some driver fixes for control input validations

Message

Takashi Iwai June 14, 2024, 3:37 p.m. UTC

Hi,

this is a revised patch set as a follow up of the thread about the
errors reported by kselftest mixer-test.  It changes HD-audio and
vmaster control behavior to return -EINVAL for invalid input values.

There is a change in kselftest itself to skip the verification after
write tests for volatile controls, too.  It's for the channel map
controls that can't hold the stable values.

v1->v2:
* Skip only verification after write in kselftest
* Add sanity check to HDMI chmap write, too


Takashi

===

Takashi Iwai (6):
  ALSA: vmaster: Return error for invalid input values
  ALSA: hda: Return -EINVAL for invalid volume/switch inputs
  ALSA: control: Apply sanity check of input values for user elements
  kselftest/alsa: mixer-test: Skip write verification for volatile
    controls
  ALSA: chmap: Mark Channel Map controls as volatile
  ALSA: hda: Add input value sanity checks to HDMI channel map controls

 sound/core/control.c                      |  3 ++-
 sound/core/pcm_lib.c                      |  1 +
 sound/core/vmaster.c                      |  8 ++++++++
 sound/hda/hdmi_chmap.c                    | 18 ++++++++++++++++++
 sound/pci/hda/hda_codec.c                 | 23 ++++++++++++++++++-----
 tools/testing/selftests/alsa/mixer-test.c |  4 ++++
 6 files changed, 51 insertions(+), 6 deletions(-)