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

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

Hi,

On 02/10/26 18:16, Dmitry Baryshkov wrote:
> 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@collabora.com>
>> ---
>>
>> 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

Ack

> 
>>   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?

Will check this again.

> 
>> 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?

I will unskip for kingoftown and test.

> 
> 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?

These tests caused hang or timeout, so it was added to skip file.

> 
> 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?
> 

Let me check if we can use a unified configuration. I will regenerate 
the expectation files.

Regards,
Vignesh