Re: [PATCH v3 2/2] drm/ci: add sc7180-trogdor-wormdingler-rev1-boe

[Dmitry Baryshkov]

On Tue, Jun 16, 2026 at 05:24:05PM +0530, Vignesh Raman wrote:
> Add job that executes the IGT test suite for
> sc7180-trogdor-wormdingler-rev1-boe.
> 
> Signed-off-by: Vignesh Raman <vignesh.raman@xxxxxxxxxxxxx>
> ---
> 
> diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
> index fdaa26ad79d6..61ac2d043ce3 100644
> --- a/drivers/gpu/drm/ci/build.sh
> +++ b/drivers/gpu/drm/ci/build.sh
> @@ -28,6 +28,7 @@ if [[ "$KERNEL_ARCH" = "arm64" ]]; then
>      DEVICE_TREES+=" arch/arm64/boot/dts/mediatek/mt8192-asurada-spherion-r0.dtb"
>      DEVICE_TREES+=" arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown.dtb"
>      DEVICE_TREES+=" arch/arm64/boot/dts/qcom/sm8350-hdk.dtb"
> +    DEVICE_TREES+=" arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe.dtb"

Let's keep it at least minimally sorted, puting wormdingler between
kingoftown and sm8350

>  elif [[ "$KERNEL_ARCH" = "arm" ]]; then
>      GCC_ARCH="arm-linux-gnueabihf"
>      DEBIAN_ARCH="armhf"

[..]

> diff --git a/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-wormdingler-rev1-boe-fails.txt b/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-wormdingler-rev1-boe-fails.txt
> new file mode 100644
> index 000000000000..4af93eeeaf99
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-wormdingler-rev1-boe-fails.txt
> @@ -0,0 +1,23 @@

Most of the entries here are shared with kingoftown / limozeen, except
for:

> +kms_invalid_mode@overflow-vrefresh,Fail

Do you know why it fails? Why does it not on previous devices?

> diff --git a/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-wormdingler-rev1-boe-skips.txt b/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-wormdingler-rev1-boe-skips.txt
> new file mode 100644
> index 000000000000..8d1768de90cc
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-wormdingler-rev1-boe-skips.txt

This is mostly a copy of the existing lists, but kingoftown and limozeen
list .*suspend.* here. If we don't skip them for wormdingler, should we
unskip them for kingoftown too?

Also, I see that kingoftown lists several additional tests to be skipped
(kms_flip@2x-wf_vblank-ts-check, kms_cursor_crc@cursor-random-max-size,
kms_invalid_mode@overflow-vrefresh). Do you remember why?

Overall, I _think_ I have commented beforehand, but I can't find that
comment (so it might be my delusion or false memory). Is there a way to
unify the configuration for sc7180 trogdor devices?

-- 
With best wishes
Dmitry