tests/kms_chamelium: add dp-audio test
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v5 posting 2/21 patch 1 replyIn review
- v4 posting 2/29 patches 4 repliesSuperseded
- v3 posting 2/29 patches 1 replySuperseded
Earlier versions · 7
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v7 Latest | 8 patches | 5 replies | In review | |
| v6 | 7 patches | 11 replies | Superseded | |
| v5 posting 1/2 | 1 patch | 0 replies | Superseded | |
| v4 posting 1/2 | 5 patches | 5 replies | Superseded | |
| v3 posting 1/2 | 4 patches | 1 reply | Superseded | |
| v2 | 1 patch | 0 replies | Superseded | |
| v1 | 1 patch | 1 reply | Superseded |
Patches in v7 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/8 | tests/kms_chamelium: add dp-audio testdocs/ docs/ lib/ +16 more | +1382 | 2 replies | |
| 2/8 | tests/kms_chamelium: capture audio data in real-timelib/ tests/ | +43 | 0 replies | |
| 3/8 | lib/igt_{alsa, audio}: improve logginglib/ lib/ | +23 | 1 reply | |
| 4/8 | lib/igt_alsa: use variables to improve readabilitylib/ | +4 | 1 reply | |
| 5/8 | tests/kms_chamelium: test we receive a signal from both audio channelslib/ lib/ tests/ | +111 | 0 replies | |
| 6/8 | tests/kms_chamelium: test audio channels are not mixed uplib/ lib/ tests/ | +73 | 0 replies | |
| 7/8 | tests/kms_chamelium: run audio test with multiple sampling rateslib/ lib/ lib/ +2 more | +89 | 0 replies | |
| 8/8 | HAX: add dp-audio test to fast-feedbacktests/ | +1 | 0 replies |
Reviews trailers per patch
3 of 8 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.
| Patch | Martin Peres |
|---|---|
| Series | R-b |
| 1/8 | |
| 2/8 | |
| 3/8 | R-b |
| 4/8 | R-b |
| 5/8 | |
| 6/8 | R-b |
| 7/8 | |
| 8/8 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [igt-dev] [PATCH i-g-t v7 0/8] tests/kms_chamelium: add dp-audio test
Changes in v7: addressed Martin's comments
Thanks for the detailed review! I've tried splitting some commits, let me know if more splitting is needed.
Simon Ser (8):
tests/kms_chamelium: add dp-audio test tests/kms_chamelium: capture audio data in real-time lib/igt_{alsa,audio}: improve logging
lib/igt_alsa: use variables to improve readability tests/kms_chamelium: test we receive a signal from both audio channels tests/kms_chamelium: test audio channels are not mixed up tests/kms_chamelium: run audio test with multiple sampling rates
HAX: add dp-audio test to fast-feedback
docs/audio.txt | 45 -- docs/chamelium.txt | 34 +- lib/igt.h | 1 + lib/igt_alsa.c | 83 +++- lib/igt_alsa.h | 1 + lib/igt_audio.c | 428 +++++++++++++------ lib/igt_audio.h | 19 +- lib/igt_aux.c | 31 ++ lib/igt_aux.h | 1 + lib/igt_chamelium.c | 161 ++++++++ lib/igt_chamelium.h | 17 + lib/igt_chamelium_stream.c | 572 ++++++++++++++++++++++++++ lib/igt_chamelium_stream.h | 50 +++ lib/meson.build | 5 +- meson.build | 52 +-- meson_options.txt | 6 - tests/audio.c | 193 --------- tests/intel-ci/fast-feedback.testlist | 1 + tests/kms_chamelium.c | 355 +++++++++++++++- tests/meson.build | 9 +- 20 files changed, 1614 insertions(+), 450 deletions(-) delete mode 100644 docs/audio.txt create mode 100644 lib/igt_chamelium_stream.c create mode 100644 lib/igt_chamelium_stream.h delete mode 100644 tests/audio.c
Signature · 6 lines
2.21.0 _______________________________________________ igt-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/igt-dev