Skip to content

tests/kms_chamelium: add pulse test

Simon Ser igt-dev 3 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest10 patches 10 repliesIn review
  2. v29 patches 12 repliesSuperseded
  3. v18 patches 0 repliesSuperseded

Changes in v3 from the cover letter

- Added one more patch to introduce CHAMELIUM_MAX_AUDIO_CHANNELS
- Addressed Martin's comments

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/10lib/igt_chamelium: introduce CHAMELIUM_MAX_AUDIO_CHANNELSlib/igt_chamelium.c lib/igt_chamelium.h+12 -41 reply
2/10tests/kms_chamelium: refactor audio testtests/kms_chamelium.c+195 -1410 replies
3/10tests/kms_chamelium: introduce audio_state_receivetests/kms_chamelium.c+36 -190 replies
4/10tests/kms_chamelium: rename do_test_display_audio and test_audio_configurationtests/kms_chamelium.c+9 -80 replies
5/10tests/kms_chamelium: explain why 8-channel tests aren't performedtests/kms_chamelium.c+3 -10 replies
6/10lib/igt_audio: introduce audio_convert_tolib/igt_audio.c lib/igt_audio.h tests/kms_chamelium.c+55 -660 replies
7/10tests/kms_chamelium: add name parameter to audio_state_starttests/kms_chamelium.c+14 -100 replies
8/10lib/igt_audio: make audio_extract_channel_s32_le support a NULL dstlib/igt_audio.c+10 -10 replies
9/10tests/kms_chamelium: add a flatline audio testtests/kms_chamelium.c+101 -09 replies
10/10tests/kms_chamelium: add audio channel alignment testtests/kms_chamelium.c+86 -140 replies

Reviews trailers per patch

All 10 patches have a Reviewed-by or Acked-by.

PatchMartin Peres
1/10R-b
2/10R-b
3/10R-b
4/10R-b
5/10R-b
6/10R-b
7/10R-b
8/10R-b
9/10R-b
10/10R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v3 thread

Cover letter [igt-dev] [PATCH i-g-t v3 00/10] tests/kms_chamelium: add pulse test

The series performs a refactoring of the current audio test to be able to introduce a new one.

The new test checks both the amplitude and the channel alignment.

Changes from v2 to v3:
- Added one more patch to introduce CHAMELIUM_MAX_AUDIO_CHANNELS
- Addressed Martin's comments

Simon Ser (10):
lib/igt_chamelium: introduce CHAMELIUM_MAX_AUDIO_CHANNELS tests/kms_chamelium: refactor audio test tests/kms_chamelium: introduce audio_state_receive tests/kms_chamelium: rename do_test_display_audio and
test_audio_configuration
tests/kms_chamelium: explain why 8-channel tests aren't performed lib/igt_audio: introduce audio_convert_to tests/kms_chamelium: add name parameter to audio_state_start lib/igt_audio: make audio_extract_channel_s32_le support a NULL dst tests/kms_chamelium: add a flatline audio test tests/kms_chamelium: add audio channel alignment test

 lib/igt_audio.c       |  98 ++++----
 lib/igt_audio.h       |  12 +-
 lib/igt_chamelium.c   |   8 +-
 lib/igt_chamelium.h   |   8 +-
 tests/kms_chamelium.c | 563 ++++++++++++++++++++++++++++++------------
 5 files changed, 473 insertions(+), 216 deletions(-)

--
2.21.0

_______________________________________________
igt-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/igt-dev