Mark asked for this in the es8316 thread [1]: have mixer-test check
for other controls changing when one is written. Patch 2 adds that.

Patch 1 fixes a bug that testing patch 2 turned up. With a driver that
notifies a neighbour with a value event, wait_for_event() returns on
the neighbour's event and the next test reports a spurious one.

The cost is two extra writes per control and a read of every other
control on the card after each. On the two cards here, 39 controls in
all, the run time does not move. I have no card with hundreds of
controls to measure against the 45 second kselftest timeout.

[1] 
https://lore.kernel.org/all/[email protected]/

HyeongJun An (2):
  selftests/alsa: Do not take another control's event as our own
  selftests/alsa: Check that a write does not silently change other
    controls

 tools/testing/selftests/alsa/mixer-test.c | 235 +++++++++++++++++++++-
 1 file changed, 234 insertions(+), 1 deletion(-)

-- 
2.43.0


Reply via email to