Skip to content

glymur-crd: fix display-gfx filename typo and add pre-merge exclusions#58

Merged
mwasilew merged 3 commits into
qualcomm-linux:masterfrom
qcom-anilyada:test/glymur-exclusions
May 6, 2026
Merged

glymur-crd: fix display-gfx filename typo and add pre-merge exclusions#58
mwasilew merged 3 commits into
qualcomm-linux:masterfrom
qcom-anilyada:test/glymur-exclusions

Conversation

@qcom-anilyada
Copy link
Copy Markdown

@qcom-anilyada qcom-anilyada commented May 5, 2026

Fix a typo in the pre-merge-display-gfx filename (dispaly → display) across the testcase and testplan files.

Add pre-merge testplan exclusions for glymur-crd, excluding pre-merge-camera, pre-merge-display-gfx, and pre-merge-gstr-video from the device's test suite.

For the meta-qcom project, exclude WiFi_Firmware_Driver, WiFi_OnOff, DSP_AudioPD, and fastrpc_test tests from the pre-merge test suite.

I'll be sharing the passed job, post that we can merge this.

Signed-off-by: Anil Yadav anilyada@qti.qualcomm.com

@qcom-anilyada
Copy link
Copy Markdown
Author

Here are the passed jobs on the generated yaml:

Testplan Job [qcom-distro] Status
boot https://lava.infra.foundries.io/scheduler/job/200014 PASS
audio https://lava.infra.foundries.io/scheduler/job/200015 PASS
basic https://lava.infra.foundries.io/scheduler/job/200016 PASS
bt https://lava.infra.foundries.io/scheduler/job/200017 PASS

Ready to merge. Please review.

@qcom-anilyada qcom-anilyada requested a review from mwasilew May 6, 2026 06:11
Copy link
Copy Markdown

@smuppand smuppand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only minor non-blocking cleanup

Comment thread lava_test_plans/devices/glymur-crd Outdated
@@ -9,6 +9,7 @@
]%}

{# Exclude camera testplan for this device #}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the comment says “Exclude camera testplan”, but the list now excludes camera, display-gfx, and gstr-video. It would be clearer to update it to “Exclude unsupported pre-merge testplans for this device”.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated for this file and created another commit to fix the same for other files

@qcom-anilyada qcom-anilyada force-pushed the test/glymur-exclusions branch from 655afae to dc2844e Compare May 6, 2026 06:33
Copy link
Copy Markdown

@smuppand smuppand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown

@mwasilew mwasilew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should merge #51 before this and then rebase.

'echo "STORAGE=nvme" >> $IMAGE_PATH/flash.settings'
]%}

{# Exclude camera testplan for this device #}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you move this to a project

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, will rebase and move it once #51 is merged

Copy link
Copy Markdown

@lumag lumag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The commit messages don't capture, why the tests are excluded. Is it a temporal measure until the kernel gets necessary patches or is it a permanent state, etc.

Rename pre-merge-dispaly-gfx to pre-merge-display-gfx to correct
the misspelling of "display" in both the testcase and testplan files.

Signed-off-by: Anil Yadav <anilyada@qti.qualcomm.com>
Temporarily exclude pre-merge-camera, pre-merge-display-gfx, and
pre-merge-gstr-video testplans for the glymur-crd device until the
kernel receives the necessary patches.

For the meta-qcom project, temporarily exclude WiFi_Firmware_Driver,
WiFi_OnOff, DSP_AudioPD, and fastrpc_test from the pre-merge test
suite pending the required kernel fixes.

Signed-off-by: Anil Yadav <anilyada@qti.qualcomm.com>
Replace the outdated "Exclude camera testplan" comment with
"Exclude unsupported testplans" across iq-8275-evk, iq-x7181-evk,
qcs615-ride, qcs8300-ride-sx, qcs9100-ride-sx, and rb1-core-kit
device files.

Signed-off-by: Anil Yadav <anilyada@qti.qualcomm.com>
@qcom-anilyada qcom-anilyada force-pushed the test/glymur-exclusions branch from dc2844e to 8145ad3 Compare May 6, 2026 10:29
@qcom-anilyada
Copy link
Copy Markdown
Author

The commit messages don't capture, why the tests are excluded. Is it a temporal measure until the kernel gets necessary patches or is it a permanent state, etc.

did rebase, resolved conflicts and updated the commit message mentioning that its a temporary exclusion until kernel patches arrive.

@mwasilew mwasilew merged commit b9de72b into qualcomm-linux:master May 6, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants