Re: [PATCH i-g-t v3 0/5] tests/chamelium: Integrate the chamelium v3

[Vignesh Raman <vignesh.raman@collabora.com>]

Hi Louis,

On 22/10/24 22:01, Louis Chauvet wrote:
> Hi all,
> 
> This patch series introduces a new wrapper for the Chameleon v3 board, a
> testing tool used primarily for Chromebooks, but also by community members
> to increase test coverage of graphic devices. The Chameleon v3 is an
> upgrade from its predecessor, with the ability to emulate 2 HDMI and 2
> DisplayPort devices, including MST devices.
> 
> The board can read and analyze any stream on the DisplayPort and HDMI
> cables, and even take captures, making it an ideal tool for testing
> hardware, driver, and user space behavior in specific situations, such as
> screen hotplugging and multiple MST screens. However, the changes between
> the v2 and v3, such as the addition of MST support and the removal of CRC
> for screen captures, would require significant modifications to the
> existing tests and wrappers.
> 
> To avoid this complexity and ensure that the Chameleon v3 is fully
> supported and the v2 is not broken, I wrote a new wrapper with
> simplicity and ease of use in mind. The Chameleon v3 wrapper should only
> manage the Chameleon itself, without the need for DRM displays, EDID
> caching/modifications, or state machines. This approach will make it
> easier for developers to use the Chameleon v3 and contribute to its
> development.
> 
> This series depends on:
> -https://lore.kernel.org/igt-dev/20241022-b4-cv3-01-igt-kms-v2-0-8f654694b513@bootlin.com
> -https://lore.kernel.org/igt-dev/20241022-b4-cv3-02-monitor-edids-v2-0-7634786c21e6@bootlin.com
> -https://lore.kernel.org/igt-dev/20241022-b4-cv3-03-cv2-split-v1-0-9f7cc8591ff9@bootlin.com

I have tested this series and the dependent patches,
https://gitlab.freedesktop.org/vigneshraman/igt-gpu-tools/-/commits/test-chamelium-v3-fix/?ref_type=heads

kms_chamelium_v3_basic test passes in drm-ci,
https://gitlab.freedesktop.org/vigneshraman/linux/-/jobs/66445667


Regards,
Vignesh


> 
> In addition, I will send more series to:
> - add more methods to the chamelium wrapper
> - add new options in the configuration file to configure the chamelium
> - add tests for the cv3
> 
> ---
> Changes in v3:
> - Rebased and splitted the series
> - Fixed few build issues
> - Link to v2: https://lore.kernel.org/r/20240709-dev-remove-static-ports-v2-0-5adfc6985778@bootlin.com
> 
> Changes in v2:
> - Mainly splitting and cleaning work since the RFC,
> - Link to v1: https://lore.kernel.org/r/20240605-dev-remove-static-ports-v1-0-665f25b7a4db@bootlin.com
> 
> ---
> Louis Chauvet (5):
>        lib/chamelium/v3: Introduce the foundation for the Chamelium v3 wrapper
>        lib/chamelium/v3: Introduce initialization and cleanup of Chamelium-related structures
>        lib/chamelium/v3: Add method to discover Chamelium ports
>        lib/chamelium/v3: Implement method to retrieve Chamelium port names
>        tests/chamelium/v3: Implement a basic Chamelium v3 accessibility test
> 
>   lib/chamelium/v3/igt_chamelium.c            | 309 ++++++++++++++++++++++++++++
>   lib/chamelium/v3/igt_chamelium.h            |  52 +++++
>   lib/igt_kms.c                               |   8 +
>   lib/meson.build                             |  12 +-
>   lib/tests/meson.build                       |   4 +
>   meson.build                                 |  15 +-
>   meson_options.txt                           |   4 +
>   tests/chamelium/v3/kms_chamelium_v3_basic.c |  30 +++
>   tests/meson.build                           |  26 +++
>   9 files changed, 458 insertions(+), 2 deletions(-)
> ---
> base-commit: 9b8c0f6da8898f760bfaa2113455eb84b68a69f4
> change-id: 20240522-dev-remove-static-ports-9382ffbbf146
> prerequisite-message-id: 20241022-b4-cv3-01-igt-kms-v2-0-8f654694b513@bootlin.com
> prerequisite-patch-id: 0562bbe57a0b489e7f46a51717c54ea26296cb8c
> prerequisite-patch-id: a16d6fdd238eab5a91630d0934c9186669406fc7
> prerequisite-patch-id: c7fcc64d709f283da748a0f9bbe3bd6143d76300
> prerequisite-patch-id: 09fc5e2f569fa8287e767ceb353bc0a0f94ae2e1
> prerequisite-patch-id: 63cf2f6323333bb8cbd0dbbd883b5c9fe373dea1
> prerequisite-message-id: 20241022-b4-cv3-02-monitor-edids-v2-0-7634786c21e6@bootlin.com
> prerequisite-patch-id: 90ffdddeda6dd4cd2ae64e60fb31b0bbfee0250f
> prerequisite-patch-id: d62e9d4981a68ac6e71544ab2a3dfec37dfa3299
> prerequisite-patch-id: 3d898f58702cc303e18d3afc487e46f68641989b
> prerequisite-patch-id: 57f5446073d42a01c29e6ff8a24f6609eedb49a1
> prerequisite-patch-id: e6d6ee4ecd0e738a60091b19449c8622bb8bb6ae
> prerequisite-message-id: 20241022-b4-cv3-03-cv2-split-v1-0-9f7cc8591ff9@bootlin.com
> prerequisite-patch-id: 4ac6bf11d25e80e99d5607d4864c3c7b4273dde3
> prerequisite-patch-id: 2246b4289eca0b8e8b02519051af7564b0875a2a
> prerequisite-patch-id: 394057bc39308265a03e026ac924f88d3d933d49
> prerequisite-patch-id: ad1f0838e81fa620013313316afd35a3d5c146fa
> 
> Best regards,